Re: [U-Boot] [PATCH] arm926ejs: 16-byte align stack to avoid LDRD/STRD problems

2009-10-05 Thread Simon Kagstrom
On Mon, 5 Oct 2009 09:30:54 -0500 Andrew Dyer wrote: > >        sub     r0, r0, #(CONFIG_STACKSIZE_IRQ+CONFIG_STACKSIZE_FIQ) > >  #endif > > -       sub     sp, r0, #12             /* leave 3 words for abort-stack     > > */ > > +       sub     sp, r0, #16             /* leave 3 words for abort-s

Re: [U-Boot] [PATCH] arm926ejs: 16-byte align stack to avoid LDRD/STRD problems

2009-10-05 Thread Andrew Dyer
On Mon, Oct 5, 2009 at 8:23 AM, Simon Kagstrom wrote: > U-boot for Marvell Kirkwood boards no longer work after the EABI changes > introduced in commit f772acf8a584067033eff1e231fcd1fb3a00d3d9. This > turns out to be caused by a stack alignment issue. The armv5te > instructions ldrd/strd instructi

Re: [U-Boot] [PATCH] arm926ejs: 16-byte align stack to avoid LDRD/STRD problems

2009-10-05 Thread Stefan Roese
On Monday 05 October 2009 16:30:54 Andrew Dyer wrote: > > diff --git a/cpu/arm926ejs/start.S b/cpu/arm926ejs/start.S > > index 8043322..ca520eb 100644 > > --- a/cpu/arm926ejs/start.S > > +++ b/cpu/arm926ejs/start.S > > @@ -171,7 +171,8 @@ stack_setup: > > #ifdef CONFIG_USE_IRQ > >sub r

[U-Boot] [PATCH] arm926ejs: 16-byte align stack to avoid LDRD/STRD problems

2009-10-05 Thread Simon Kagstrom
U-boot for Marvell Kirkwood boards no longer work after the EABI changes introduced in commit f772acf8a584067033eff1e231fcd1fb3a00d3d9. This turns out to be caused by a stack alignment issue. The armv5te instructions ldrd/strd instructions require 8-byte alignment to work properly (otherwise undefi