[U-Boot] [PATCH 1/2] mx51evk: support new relocation scheme

[Wolfgang Denk]

Dear Shawn Guo,

In message <AANLkTiknA2QYu1KVnxqmEnqJ0ewKaXwgnAMNu19i7SgO at mail.gmail.com> you wrote:
> On Tue, Oct 26, 2010 at 4:14 PM, Stefano Babic <sbabic at denx.de> wrote:
> > On 10/26/2010 10:05 AM, Shawn Guo wrote:
> >> Hi Stefano,
> >>
> >> On Tue, Oct 26, 2010 at 3:59 PM, Stefano Babic <sbabic at denx.de> wrote:
> >>> On 10/25/2010 05:14 PM, Shawn Guo wrote:
> >>>
> >>>> +#define CONFIG_SYS_SDRAM_BASE =A0 =A0 =A0 =A0PHYS_SDRAM_1
> >>>> +#define CONFIG_SYS_INIT_SP_ADDR =A0 =A0 =A0(IRAM_BASE_ADDR + 0x1000)
> >>>
> >>> What is the meaning of the offset at 0x1000 ? Is there something at
> >>> IRAM_BASE_ADDR ?
> >>>
> >> I thought CONFIG_SYS_INIT_SP_ADDR is assigned to stack pointer (sp)
> >> which decrease the address. =A0Is it not the case?
> >>
> >
> > Yes, my doubt is due to the fact that we do not set the SP at the end of
> > the IRAM and we set it to an arbitrary address in between.
> >
> At this point, IRAM is totally free for u-boot to use, so I simply
> reserved 4 KB from start.

It is a bad idea when everybody implement this differently, just as he
feels.

I want to get rid of this. Patches following soon. As for now, please
focus on defining the proper address and size of the IRAM, the rest
will then be automatically derived (global data on top, then stack
below).

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: wd at denx.de
One difference between a man and a machine is that a machine is quiet
when well oiled.