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 <4cc7c628.70...@denx.de> 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. 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. I'm guessing that the PPC tools don't complain about the # symbol? Steve _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
