Dear Shawn Guo, In message <1288174358-14896-1-git-send-email-shawn....@gmail.com> you wrote: > This patch is to fix build breakage and support new relocatoin > scheme for mx51evk. > > - Correct IRAM base address and add size definition > > The IRAM starts from 0x1FFE0000 on final revsion i.mx51 than > 0x1FFE8000 which is for older revision. > > - Include imx-regs.h in mx51evk.h > > Definitions like CSD0_BASE_ADDR and IRAM_BASE_ADDR can be > referred to. > > - Define CONFIG_SYS_INIT_RAM_ADDR and CONFIG_SYS_INIT_RAM_SIZE > > This is something suggested by Wolfgang, and for automatic > calculation of macros like CONFIG_SYS_GBL_DATA_SIZE (done) and > CONFIG_SYS_INIT_SP_ADDR (later).
Um... this is NOT what we discussed. > +#define CONFIG_SYS_SDRAM_BASE (PHYS_SDRAM_1) > +#define CONFIG_SYS_INIT_RAM_ADDR (IRAM_BASE_ADDR) > +#define CONFIG_SYS_INIT_RAM_SIZE (IRAM_SIZE) > +#define CONFIG_SYS_INIT_SP_ADDR \ > + (CONFIG_SYS_INIT_RAM_ADDR + CONFIG_SYS_INIT_RAM_SIZE) > + No. This is not what I want to se. Memory allocation in the inital ram has the global data at the top. Please use something like #define CONFIG_SYS_INIT_SP_OFFSET (CONFIG_SYS_INIT_RAM_SIZE - GENERATED_GBL_DATA_SIZE) #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_INIT_RAM_ADDR + CONFIG_SYS_INIT_SP_OFFSET) 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: w...@denx.de "Whoever undertakes to set himself up as a judge of Truth and Know- ledge is shipwrecked by the laughter of the gods." - Albert Einstein _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
