Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/20 23:11:33: On 07/20/2012 04:20 AM, Joakim Tjernlund wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which is just get in the way when gdb is walking the stack

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Scott Wood
On 07/21/2012 10:06 AM, Joakim Tjernlund wrote: Scott Wood scottw...@freescale.com wrote on 2012/07/20 23:11:33: On 07/20/2012 04:20 AM, Joakim Tjernlund wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which is just get

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/23 18:52:28: From: Scott Wood scottw...@freescale.com To: Joakim Tjernlund joakim.tjernl...@transmode.se, Cc: u-boot@lists.denx.de Date: 2012/07/23 18:52 Subject: Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong. On 07/21

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Scott Wood
] mpc85xx: Initial SP alignment is wrong. On 07/21/2012 10:06 AM, Joakim Tjernlund wrote: Scott Wood scottw...@freescale.com wrote on 2012/07/20 23:11:33: On 07/20/2012 04:20 AM, Joakim Tjernlund wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-23 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/23 19:28:20: On 07/23/2012 12:11 PM, Joakim Tjernlund wrote: ...but I was a bit confused when I thought it would help terminate things. The NULL LR only helps prevent finding something worse, if something happens to do a backtrace

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-21 Thread Joakim Tjernlund
Scott Wood scottw...@freescale.com wrote on 2012/07/20 23:11:33: On 07/20/2012 04:20 AM, Joakim Tjernlund wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which is just get in the way when gdb is walking the stack because

[U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-20 Thread Joakim Tjernlund
PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which is just get in the way when gdb is walking the stack because the reset vector may not accessible depending on emulator settings. Also use a temp register so gdb doesn't pick up

Re: [U-Boot] [PATCH 2/2] mpc85xx: Initial SP alignment is wrong.

2012-07-20 Thread Scott Wood
On 07/20/2012 04:20 AM, Joakim Tjernlund wrote: PowerPC mandates SP to be 16 bytes aligned. Furthermore, a stack frame is added, pointing to the reset vector which is just get in the way when gdb is walking the stack because the reset vector may not accessible depending on emulator settings.