Message ID | 20120608113605.75BD5D43B26@localhost.localdomain (mailing list archive) |
---|---|
State | Superseded, archived |
Headers | show |
On 06/08/2012 06:36 AM, Michael Neuling wrote: > -/* General Purpose Registers (GPRs) */ > +/* > + * General Purpose Registers (GPRs) > + * > + * The lower case r0-r31 should be used in preference to the upper > + * case R0-R31 as they provide more error checking in the assembler. > + * Use R0-31 only when really nessesary. > + */ > > -#define r0 0 > -#define r1 1 > -#define r2 2 > -#define r3 3 > -#define r4 4 > -#define r5 5 > -#define r6 6 > -#define r7 7 > -#define r8 8 > -#define r9 9 > -#define r10 10 > -#define r11 11 > -#define r12 12 > -#define r13 13 > -#define r14 14 > -#define r15 15 > -#define r16 16 > -#define r17 17 > -#define r18 18 > -#define r19 19 > -#define r20 20 > -#define r21 21 > -#define r22 22 > -#define r23 23 > -#define r24 24 > -#define r25 25 > -#define r26 26 > -#define r27 27 > -#define r28 28 > -#define r29 29 > -#define r30 30 > -#define r31 31 > +#define r0 %r0 > +#define r1 %r1 > +#define r2 %r2 > +#define r3 %r3 > +#define r4 %r4 > +#define r5 %r5 > +#define r6 %r6 > +#define r7 %r7 > +#define r8 %r8 > +#define r9 %r9 > +#define r10 %r10 > +#define r11 %r11 > +#define r12 %r12 > +#define r13 %r13 > +#define r14 %r14 > +#define r15 %r15 > +#define r16 %r16 > +#define r17 %r17 > +#define r18 %r18 > +#define r19 %r19 > +#define r20 %r20 > +#define r21 %r21 > +#define r22 %r22 > +#define r23 %r23 > +#define r24 %r24 > +#define r25 %r25 > +#define r26 %r26 > +#define r27 %r27 > +#define r28 %r28 > +#define r29 %r29 > +#define r30 %r30 > +#define r31 %r31 > > #define R0 0 > #define R1 1 I think your patch was truncated somehow. > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev > Jesse Larrew Software Engineer, Linux on Power Kernel Team IBM Linux Technology Center Phone: (512) 973-2052 (T/L: 363-2052) jlarrew@linux.vnet.ibm.com
Jesse Larrew <jlarrew@linux.vnet.ibm.com> wrote: > On 06/08/2012 06:36 AM, Michael Neuling wrote: > > > > -/* General Purpose Registers (GPRs) */ > > +/* > > + * General Purpose Registers (GPRs) > > + * > > + * The lower case r0-r31 should be used in preference to the upper > > + * case R0-R31 as they provide more error checking in the assembler. > > + * Use R0-31 only when really nessesary. > > + */ > > > > -#define r0 0 > > -#define r1 1 > > -#define r2 2 > > -#define r3 3 > > -#define r4 4 > > -#define r5 5 > > -#define r6 6 > > -#define r7 7 > > -#define r8 8 > > -#define r9 9 > > -#define r10 10 > > -#define r11 11 > > -#define r12 12 > > -#define r13 13 > > -#define r14 14 > > -#define r15 15 > > -#define r16 16 > > -#define r17 17 > > -#define r18 18 > > -#define r19 19 > > -#define r20 20 > > -#define r21 21 > > -#define r22 22 > > -#define r23 23 > > -#define r24 24 > > -#define r25 25 > > -#define r26 26 > > -#define r27 27 > > -#define r28 28 > > -#define r29 29 > > -#define r30 30 > > -#define r31 31 > > +#define r0 %r0 > > +#define r1 %r1 > > +#define r2 %r2 > > +#define r3 %r3 > > +#define r4 %r4 > > +#define r5 %r5 > > +#define r6 %r6 > > +#define r7 %r7 > > +#define r8 %r8 > > +#define r9 %r9 > > +#define r10 %r10 > > +#define r11 %r11 > > +#define r12 %r12 > > +#define r13 %r13 > > +#define r14 %r14 > > +#define r15 %r15 > > +#define r16 %r16 > > +#define r17 %r17 > > +#define r18 %r18 > > +#define r19 %r19 > > +#define r20 %r20 > > +#define r21 %r21 > > +#define r22 %r22 > > +#define r23 %r23 > > +#define r24 %r24 > > +#define r25 %r25 > > +#define r26 %r26 > > +#define r27 %r27 > > +#define r28 %r28 > > +#define r29 %r29 > > +#define r30 %r30 > > +#define r31 %r31 > > > > #define R0 0 > > #define R1 1 > > > I think your patch was truncated somehow. How so? Mikey
On 06/13/2012 10:25 PM, Michael Neuling wrote: > Jesse Larrew <jlarrew@linux.vnet.ibm.com> wrote: >> On 06/08/2012 06:36 AM, Michael Neuling wrote: >> >> >>> -/* General Purpose Registers (GPRs) */ >>> +/* >>> + * General Purpose Registers (GPRs) >>> + * >>> + * The lower case r0-r31 should be used in preference to the upper >>> + * case R0-R31 as they provide more error checking in the assembler. >>> + * Use R0-31 only when really nessesary. >>> + */ >>> <snip> >>> >>> #define R0 0 >>> #define R1 1 >> >> >> I think your patch was truncated somehow. > > How so? > > Mikey > Oops. False alarm. I read the comment about the GPRs and expected to see #define statements for R2-31 as well. However, I just realized that your patch didn't actually add them. The R0 and R1 defines are just there for context. My mistake. Jesse Larrew Software Engineer, Linux on Power Kernel Team IBM Linux Technology Center Phone: (512) 973-2052 (T/L: 363-2052) jlarrew@linux.vnet.ibm.com
Index: clone3/arch/powerpc/include/asm/ppc_asm.h =================================================================== --- clone3.orig/arch/powerpc/include/asm/ppc_asm.h +++ clone3/arch/powerpc/include/asm/ppc_asm.h @@ -490,40 +490,46 @@ END_FTR_SECTION_IFCLR(CPU_FTR_601) #define cr7 7 -/* General Purpose Registers (GPRs) */ +/* + * General Purpose Registers (GPRs) + * + * The lower case r0-r31 should be used in preference to the upper + * case R0-R31 as they provide more error checking in the assembler. + * Use R0-31 only when really nessesary. + */ -#define r0 0 -#define r1 1 -#define r2 2 -#define r3 3 -#define r4 4 -#define r5 5 -#define r6 6 -#define r7 7 -#define r8 8 -#define r9 9 -#define r10 10 -#define r11 11 -#define r12 12 -#define r13 13 -#define r14 14 -#define r15 15 -#define r16 16 -#define r17 17 -#define r18 18 -#define r19 19 -#define r20 20 -#define r21 21 -#define r22 22 -#define r23 23 -#define r24 24 -#define r25 25 -#define r26 26 -#define r27 27 -#define r28 28 -#define r29 29 -#define r30 30 -#define r31 31 +#define r0 %r0 +#define r1 %r1 +#define r2 %r2 +#define r3 %r3 +#define r4 %r4 +#define r5 %r5 +#define r6 %r6 +#define r7 %r7 +#define r8 %r8 +#define r9 %r9 +#define r10 %r10 +#define r11 %r11 +#define r12 %r12 +#define r13 %r13 +#define r14 %r14 +#define r15 %r15 +#define r16 %r16 +#define r17 %r17 +#define r18 %r18 +#define r19 %r19 +#define r20 %r20 +#define r21 %r21 +#define r22 %r22 +#define r23 %r23 +#define r24 %r24 +#define r25 %r25 +#define r26 %r26 +#define r27 %r27 +#define r28 %r28 +#define r29 %r29 +#define r30 %r30 +#define r31 %r31 #define R0 0 #define R1 1
Now all the fixes are in place, let's rock-n-roll! Signed-off-by: Michael Neuling <mikey@neuling.org> --- arch/powerpc/include/asm/ppc_asm.h | 72 ++++++++++++++++++++----------------- 1 file changed, 39 insertions(+), 33 deletions(-)