[U-Boot] [PATCH 1/2 v4] env: only build env_embedded and envcrc when needed

Mike Frysinger vapier at gentoo.org
Mon Oct 19 02:55:01 CEST 2009


On Sunday 18 October 2009 17:32:15 Wolfgang Denk wrote:
> Mike Frysinger wrote:
> > > Mike Frysinger wrote:
> > > > The env code is protected by the ENV_IS_EMBEDDED define, so
> > > > attempting to compile the code when this isn't defined is pointless. 
> > > > Now that the env headers have unified around CONFIG_ENV_IS_EMBEDDED,
> > > > convert the build system to only build the env objects when this is
> > > > enabled.  And now that the env code is conditionally compiled, we can
> > > > drop the source code checks.
> > > >
> > > > For people who want to extract the environment manually, add a new
> > > > option CONFIG_BUILD_ENVCRC that only enables the envcrc utility.
> > > >
> > > > Signed-off-by: Mike Frysinger <vapier at gentoo.org>
> > > > ---
> > > > v4
> > > > 	- rename config option to CONFIG_BUILD_ENVCRC
> > > >
> > > >  README                |    5 +++++
> > > >  common/Makefile       |    3 ---
> > > >  common/env_embedded.c |    7 -------
> > > >  include/common.h      |    5 +++++
> > > >  tools/Makefile        |    9 ++-------
> > > >  tools/envcrc.c        |   11 +----------
> > > >  6 files changed, 13 insertions(+), 27 deletions(-)
> > >
> > > Applied, thanks.
> 
> Arghhh...
> 
> This patch seems to break a *lot* of boards:

i'm attaching two patches here.  since we're past the merge window but before 
rc1, i dont know how invasive you want to get.

the first one restores env_embedded.o building for certain config options 
(even though it'll only produce a 0 byte file).  if you want to be cautious 
for this release, then i guess we can merge just this patch.

the second one attempts to clean up env_embedded.o in all linker scripts where 
the board would only end up with a 0 byte file.  obviously i cant test any of 
these since i dont have the hardware, but the logic seems straight forward.  
if you want to stay cautious, this would go into the next branch for start of 
next merge window.

or just merge the 2nd patch only and assume that people who dont test the rc1+ 
are dead boards anyways.  i got some build errors even after these fixes, but 
they seem unrelated to my env_embedded changes as they have to do with 
sections filling up & overflowing with my gcc-4.1.1 compiler.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-env-restore-limited-env_embedded-building.patch
Type: text/x-patch
Size: 1897 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091018/c09d17e6/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-env_embedded-remove-unused-code-from-linker-scripts.patch
Type: text/x-patch
Size: 34007 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091018/c09d17e6/attachment-0001.bin 


More information about the U-Boot mailing list