Re: [U-Boot] [PATCH] powerpc: mpc83xx: Enable pre-relocation malloc

2017-01-16 Thread Mario Six
I'll take a look at the places where r1 is modified and send a v2. On Mon, Jan 16, 2017 at 11:00 PM, york sun wrote: > On 01/06/2017 08:32 AM, Joakim Tjernlund wrote: >> On Fri, 2017-01-06 at 14:56 +0100, Mario Six wrote: >>> To enable DM on MPC83xx, we need pre-relocation

Re: [U-Boot] [PATCH] powerpc: mpc83xx: Enable pre-relocation malloc

2017-01-16 Thread york sun
On 01/06/2017 08:32 AM, Joakim Tjernlund wrote: > On Fri, 2017-01-06 at 14:56 +0100, Mario Six wrote: >> To enable DM on MPC83xx, we need pre-relocation malloc, which is >> implemented in this patch. >> > > Would be nice if you could avoid using r1, each time you modify r1 gdb will be >

Re: [U-Boot] [PATCH] powerpc: mpc83xx: Enable pre-relocation malloc

2017-01-06 Thread Joakim Tjernlund
On Fri, 2017-01-06 at 14:56 +0100, Mario Six wrote: > To enable DM on MPC83xx, we need pre-relocation malloc, which is > implemented in this patch. > Would be nice if you could avoid using r1, each time you modify r1 gdb will be upset/confused if you ever try to debug start.S with gdb. I guess

Re: [U-Boot] [PATCH] powerpc: mpc83xx: Enable pre-relocation malloc

2017-01-06 Thread york sun
On 01/06/2017 05:57 AM, Mario Six wrote: > To enable DM on MPC83xx, we need pre-relocation malloc, which is > implemented in this patch. > > Signed-off-by: Mario Six > --- > arch/powerpc/cpu/mpc83xx/cpu_init.c| 3 +-- > arch/powerpc/cpu/mpc83xx/spl_minimal.c | 4 +--- >

[U-Boot] [PATCH] powerpc: mpc83xx: Enable pre-relocation malloc

2017-01-06 Thread Mario Six
To enable DM on MPC83xx, we need pre-relocation malloc, which is implemented in this patch. Signed-off-by: Mario Six --- arch/powerpc/cpu/mpc83xx/cpu_init.c| 3 +-- arch/powerpc/cpu/mpc83xx/spl_minimal.c | 4 +--- arch/powerpc/cpu/mpc83xx/start.S | 23