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

2013-07-16 Thread 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

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

2013-07-15 Thread 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-o

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

2013-07-15 Thread Tom Rini
On Fri, Jun 14, 2013 at 10:54:59AM +0200, Stefan Roese 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). > >

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

2013-06-20 Thread Albert ARIBAUD
Hi Stefan, On Thu, 20 Jun 2013 20:28:01 +0200, Stefan Roese wrote: > Hi Albert, > > On 20.06.2013 19:51, Albert ARIBAUD wrote: > >>> The correct fix (read: the one I won't NAK) is thus to add a #else > >>> clause in the code above, in which r8 will be set to =gdata, and to > >>> remove the corr

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

2013-06-20 Thread Stefan Roese
Hi Albert, On 20.06.2013 19:51, Albert ARIBAUD wrote: >>> The correct fix (read: the one I won't NAK) is thus to add a #else >>> clause in the code above, in which r8 will be set to =gdata, and to >>> remove the corresponding assignments in the various places where they >>> reside. >> >> Here's th

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

2013-06-20 Thread Albert ARIBAUD
Hi Stefan, On Thu, 20 Jun 2013 19:01:22 +0200, Stefan Roese wrote: > Hi Albert, > > On 20.06.2013 18:42, Albert ARIBAUD wrote: > >> diff --git a/arch/arm/lib/crt0.S b/arch/arm/lib/crt0.S > >> index a9657d1..b05f66a 100644 > >> --- a/arch/arm/lib/crt0.S > >> +++ b/arch/arm/lib/crt0.S > >> @@ -85

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

2013-06-20 Thread Stefan Roese
Hi Albert, On 20.06.2013 18:42, Albert ARIBAUD wrote: >> diff --git a/arch/arm/lib/crt0.S b/arch/arm/lib/crt0.S >> index a9657d1..b05f66a 100644 >> --- a/arch/arm/lib/crt0.S >> +++ b/arch/arm/lib/crt0.S >> @@ -85,7 +85,13 @@ ENTRY(_main) >> bic sp, sp, #7 /* 8-byte alignment for ABI

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

2013-06-20 Thread Albert ARIBAUD
Hi Stefan, On Fri, 14 Jun 2013 10:54:59 +0200, Stefan Roese 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