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@denx.de Cc: Tom Rini trini@ti.com Cc: Albert ARIBAUD albert.u.boot@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