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

[Tom Rini]

On Tue, Jul 16, 2013 at 08:24:48AM +0200, Stefan Roese wrote:
> 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)?

Yes, both SD and NAND boot on classic and SD boot on xM were cold boots.

> >  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.

OK, thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130716/0059178f/attachment.pgp>