[U-Boot] [PATCH v6 6/6] ti: armv7: am33xx: Move SPL SDRAM into the correct place

[Simon Glass]

Hi Tom,

On 3 March 2015 at 08:15, Tom Rini <trini at konsulko.com> wrote:
> On Tue, Mar 03, 2015 at 08:03:02AM -0700, Simon Glass wrote:
>
>> Currently SDRAM is set up before global_data is available. This will soon
>> not work. Adjust the ordering to cope, using the new CONFIG_SPL_STACK_R
>> option to ensure that the stack is still in SDRAM once baord_init_r()
>> starts in SPL.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>
> It's good that we're on the same path :)  I think we should stick this
> into the config files as part of getting people used to that.  What I've
> got locally will also remove a bunch of CONFIG_SPL_STACK settings that
> don't do anything now (since they pointed back up to
> CONFIG_SYS_INIT_SP_ADDR).  I'm going to test the hard case (falcon mode
> poking at env for settings).

Agreed, I was nervous about doing that for things not in Kconfig.

>
> But I'm going to grab your patch as the base since keeping
> board_early_init_f() around will keep the am335x+NOR case closer to
> working.

Please do whatever suits and thanks for sorting this out.

Regards,
Simon