From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Wed, 27 Oct 2010 16:35:47 +0200
Subject: [U-Boot] [PATCH] da850evm: remove GENERATED_GBL_DATA_SIZE from
 initial stack
In-Reply-To: <1288189635-15414-1-git-send-email-bengardiner@nanometrics.ca>
References: <1288189635-15414-1-git-send-email-bengardiner@nanometrics.ca>
Message-ID: <4CC838C3.7090105@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany