Hello Steve, Steve Sakoman wrote: > On Wed, 2010-10-27 at 14:23 +0200, Heiko Schocher wrote: >> Hello Steve, >> >> Steve Sakoman wrote: >>> On Wed, 2010-10-27 at 09:14 +0200, Wolfgang Denk wrote: >>>> Dear Heiko Schocher, >>>> >>>> In message <[email protected]> you wrote: >>>>> CONFIG_SYS_INIT_RAM_ADDR (SRAM_VECT_CODE) >>>>> ^ >>>>> 0x4020f800 >>>>> CONFIG_SYS_INIT_RAM_SIZE 0x7f0 >>>> Why that? From the current settings I would expect something like >>>> >>>> CONFIG_SYS_INIT_RAM_ADDR 0x4020f800 >>>> CONFIG_SYS_INIT_RAM_SIZE 0x800 >>>> >>>> I see no reason to substract the top 16 bytes? >>> Agreed. I saw this issue last night, but didn't have enough time to >>> prepare a fix. >>> >>> I'll generate and test a patch on the OMAP boards which have implemented >>> relocation. >> Ok, thanks! > > Now that I have some coffee, I think I see the actual reason that the > build was broken. > > If you look at the error output: > > start.S: Assembler messages: > start.S:175: Error: missing ')' > start.S:175: Error: missing ')' > start.S:175: Error: garbage following instruction -- `ldr > sp,=((0x4020f800+0x800-#96))' > > It is apparent that the assembler does not like that GENERATED_GBL_DATA_SIZE > is defined as "#96" > > I tried without the # symbol and the build completes successfully.
Already fixed here: http://lists.denx.de/pipermail/u-boot/2010-October/080455.html > I'll prepare a patch to use CONFIG_SYS_INIT_RAM_ADDR and > CONFIG_SYS_INIT_RAM_SIZE on the OMAP boards with relocation, because > that is still a good idea, but we also need to deal with the prepended # > symbol. Ok, thanks. bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
