Hello Ben,
Ben Gardiner wrote:
Commit 25ddd1fb0a2281b182529afbc8fda5de2dc16d96 broke the da850evm build. The introduction of GENERATED_GBL_DATA_SIZE led to:
$make mrproper ; make da850evm_config ; make -j9 all|grep -E '( error| warning)' Generating include/autoconf.mk Generating include/autoconf.mk.dep awk '(NF && $1 !~ /^#/) { print $1 ": " $1 "_config; $(MAKE)" }' boards.cfg > .boards.depend Configuring for da850evm board... start.S: Assembler messages: start.S:177: Error: missing ')' start.S:177: Error: missing ')' start.S:177: Error: garbage following instruction -- `ldr sp,=((0xc0000000+0x1000-#96))' make[1]: *** [start.o] Error 1 make: *** [arch/arm/cpu/arm926ejs/start.o] Error 2 make: *** Waiting for unfinished jobs....
This patch removes from the definition of CONFIG_SYS_INIT_SP_ADDR the GENERATED_GBL_DATA_SIZE offset -- it doesn't appear to be needed since the resulting u-boot works fine; tested with NAND, env.oob and tftp.
Not necessary, already fixed here:
http://lists.denx.de/pipermail/u-boot/2010-October/080455.html
Can you try instead this patch?
Thanks!
bye, Heiko