[U-Boot] pine64 and "sunxi: Move the SPL stack top to 0x1A000 on Allwinner A64/A80"

[Tom Rini <trini@konsulko.com>]

On Thu, Aug 04, 2016 at 11:36:01AM -0400, Tom Rini wrote:
> On Thu, Aug 04, 2016 at 04:14:21PM +0100, Andre Przywara wrote:
> > Hi,
> > 
> > On 04/08/16 16:01, Tom Rini wrote:
> > > Hey guys,
> > > 
> > > I just started trying out my Pine64 1GB and mainline U-Boot and I've
> > > found that:
> > > commit 1a83fb4a17d959d7b037999ab7ed7e62429abe34
> > > Author: Siarhei Siamashka <siarhei.siamashka@gmail.com>
> > > Date:   Tue May 31 01:48:06 2016 +0300
> > > 
> > >     sunxi: Move the SPL stack top to 0x1A000 on Allwinner A64/A80
> > > 
> > > is breaking boot for me.  boot0 seems to run fine and then the last bit
> > > of output that I get is:
> > > boot0: start load other image
> > > boot0: Loading BL3-1
> > > Loading file 0 at address 0x40000000,size 0x00000200 success
> > > boot0: Loading scp
> > > Loading file 2 at address 0x00040000,size 0x0000c200 success
> > > set arisc reset to de-assert state
> > > Ready to disable icache.
> > > Jump to secend Boot.
> > > NOTICE:  BL3-1: Running in SRAM A2 (@0x44000)
> > > NOTICE:  Configuring SPC Controller
> > > NOTICE:  BL3-1: v1.0(debug):d697594
> > > NOTICE:  BL3-1: Built : 09:15:57, Aug  4 2016
> > > NOTICE:  Configuring AXP PMIC
> > > NOTICE:  PMIC: already configured for RSB
> > > NOTICE:  PMIC: setup successful
> > > INFO:    BL3-1: Initializing runtime services
> > > INFO:    BL3-1: Preparing for EL3 exit to normal world
> > > INFO:    BL3-1: Next image address: 0x4a000000, SPSR: 0x3c9
> > > 
> > > I'm using d697594 from
> > > https://github.com/apritzel/arm-trusted-firmware.git for ATF and
> > > boot0.img from the 20160701 Debian "longsleep" image.  Any ideas?
> > 
> > Yes, we were experiencing similar issues, basically it's stuck in
> > start.S, as far as I could quickly check.
> > Siarhei and I were doubtful that this commit (which we eyed at before)
> > could be the culprit, as it _should_ affect only SPL code, which we
> > don't use atm.
> 
> It's not touching SPL code tho.  You're changing CONFIG_SYS_INIT_SP_ADDR
> which is the start of _main in arch/arm/lib/crt0_64.S
> 

Or to be extra clear:
@@ -100,7 +100,7 @@
  * the 1 actually activates the mapping of the first 32 KiB to 0x00000000.
  */
 #define CONFIG_SYS_INIT_RAM_ADDR       0x10000
-#define CONFIG_SYS_INIT_RAM_SIZE       0x08000 /* FIXME: 40 KiB ? */
+#define CONFIG_SYS_INIT_RAM_SIZE       0xA000  /* 40 KiB */
 #else
 #define CONFIG_SYS_INIT_RAM_ADDR       0x0
 #define CONFIG_SYS_INIT_RAM_SIZE       0x8000  /* 32 KiB */

is changing CONFIG_SYS_INIT_SP_ADDR which is in U-Boot proper.  To
change the SPL stack pointer you use CONFIG_SPL_STACK which is what the
second hunk tweaks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160804/0ae90d86/attachment.sig>