From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Tue, 16 Jul 2013 08:24:48 +0200 Subject: [U-Boot] [PATCH 1/3] arm: spl: Fix SPL booting for OMAP3 In-Reply-To: <20130715143300.GB12301@bill-the-cat> References: <1371200101-11510-1-git-send-email-sr@denx.de> <20130715143300.GB12301@bill-the-cat> Message-ID: <51E4E730.6020705@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 >> Cc: Tom Rini >> Cc: Albert ARIBAUD > > 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