On Wed, Oct 27, 2010 at 7:46 PM, Wolfgang Denk <w...@denx.de> wrote: > 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. > Are you saying that automatic calculation of CONFIG_SYS_INIT_SP_ADDR will not happen?
> > 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) > If the automatic calculation of CONFIG_SYS_INIT_SP_ADDR will not happen, I can send another patch to fix it, since it's been applied by Stefano already. -- Regards, Shawn _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
