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

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to