Messages in this thread | | | Date | Wed, 27 May 2009 10:03:59 +0900 | From | Paul Mundt <> | Subject | Re: [PATCH 1/3] mtd/maps: uclinux: allow systems to override map addr/size |
| |
On Tue, May 26, 2009 at 08:46:24PM -0400, Mike Frysinger wrote: > On Tuesday 26 May 2009 19:46:16 Paul Mundt wrote: > > On Tue, May 26, 2009 at 07:33:16PM -0400, Mike Frysinger wrote: > > > +extern char _ebss; > > > + > > > > Also, it would be nice if the remaining architectures that define _ebss > > could shove this in their asm/sections.h so we can kill this off from the > > driver itself. > > well, everyone would still have to define the _ebss symbol otherwise they'd > get an undefined error due to how the map is initialized.
Yes, but that is not the issue. The issue is that the section extern has no place in a driver, this is precisely what asm/sections.h is for. Only microblaze and sh define it there today, and there is already inconsistency. Having the others define it for themselves and agreeing on a type would permit us to kill it off from the driver.
| |