[U-Boot] [PATCH 1/2 v4] env: only build env_embedded and envcrc when needed
Wolfgang Denk
wd at denx.de
Sun Oct 25 22:55:07 CET 2009
Dear Mike Frysinger,
In message <200910182055.01744.vapier at gentoo.org> you wrote:
>
> 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.
Hm... even though I like patch #2 more than the other one (which is
why I applied #2), there is a strange phenomenon remaining:
With patch #2 applied, I get these errors for the acadia_nand,
bamboo_nand, canyonlands_nand, and kilauea_nand configurations:
-> ./MAKEALL kilauea_nand
Configuring for kilauea_nand board...
In file included from include/common.h:724:
/home/wd/git/u-boot/work/include/environment.h:117:1: warning: "CONFIG_ENV_IS_EMBEDDED" redefined
In file included from /home/wd/git/u-boot/work/include/config.h:4,
from include/common.h:37:
/home/wd/git/u-boot/work/include/configs/kilauea.h:127:1: warning: this is the location of the previous definition
But when I remove the CONFIG_ENV_IS_EMBEDDED definition from line 127
of include/configs/kilauea.h, then I get this:
./MAKEALL kilauea kilauea_nand
Configuring for kilauea board...
text data bss dec hex filename
287816 24656 46548 359020 57a6c /work/wd/tmp-ppc/u-boot
Configuring for kilauea_nand board...
ppc_6xx-ld: cannot find common/env_embedded.o
make: *** [/work/wd/tmp-ppc/u-boot] Error 1
There seems to be still a logic error ...
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"A child is a person who can't understand why someone would give away
a perfectly good kitten." - Doug Larson
More information about the U-Boot
mailing list