[U-Boot] [PATCH 1/3] arm: spl: Fix SPL booting for OMAP3

[Stefan Roese]

Hi Tom,

On 07/15/2013 04:33 PM, Tom Rini wrote:
>> SPL already has GD set to the correct location (in s_init), we mustn't
>> move it around now since some data (clocks etc) is already present.
>>
>> This error was detected on the SPL port for the Compulab CM-T35 board
>> (OMAP3530).
>>
>> Signed-off-by: Stefan Roese <sr at denx.de>
>> Cc: Tom Rini <trini at ti.com>
>> Cc: Albert ARIBAUD <albert.u.boot at aribaud.net>
> 
> While we have a problem here, it's not a problem that's visible on
> current SPL using platforms, just the CM-T35, yes?

Thats what I noticed as well. I was a bit astonished that I didn't see
it on beagle. It should hit there too.

>  I just gave my
> beagleboard (classic and xM) a spin and they're OK.

Did you also power-cycle the board (cold boot)?

>  I've got a few more
> platforms I can dig out if needed, but I'm inclined to hold this until
> after v2013.07 and we can take one of the paths Albert outlined (change
> s_init to system_init, add to the function table, call that way).  Does
> that work or have I underestimated the impact of this issue?  Thanks!

I can definitely live with postponing this solution/fix to after this
release. Since your tests on the beagle boards are also working fine,
then lets just hold this patch and release v2013.07 now.

Thanks,
Stefan