Re: [PATCH 5/15] powerpc: convert to %r for all GPR usage
On 06/13/2012 10:25 PM, Michael Neuling wrote: Jesse Larrew jlar...@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 #defineR0 0 #defineR1 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) jlar...@linux.vnet.ibm.com ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 5/15] powerpc: convert to %r for all GPR usage
Jesse Larrew jlar...@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. + */ -#definer0 0 -#definer1 1 -#definer2 2 -#definer3 3 -#definer4 4 -#definer5 5 -#definer6 6 -#definer7 7 -#definer8 8 -#definer9 9 -#definer10 10 -#definer11 11 -#definer12 12 -#definer13 13 -#definer14 14 -#definer15 15 -#definer16 16 -#definer17 17 -#definer18 18 -#definer19 19 -#definer20 20 -#definer21 21 -#definer22 22 -#definer23 23 -#definer24 24 -#definer25 25 -#definer26 26 -#definer27 27 -#definer28 28 -#definer29 29 -#definer30 30 -#definer31 31 +#definer0 %r0 +#definer1 %r1 +#definer2 %r2 +#definer3 %r3 +#definer4 %r4 +#definer5 %r5 +#definer6 %r6 +#definer7 %r7 +#definer8 %r8 +#definer9 %r9 +#definer10 %r10 +#definer11 %r11 +#definer12 %r12 +#definer13 %r13 +#definer14 %r14 +#definer15 %r15 +#definer16 %r16 +#definer17 %r17 +#definer18 %r18 +#definer19 %r19 +#definer20 %r20 +#definer21 %r21 +#definer22 %r22 +#definer23 %r23 +#definer24 %r24 +#definer25 %r25 +#definer26 %r26 +#definer27 %r27 +#definer28 %r28 +#definer29 %r29 +#definer30 %r30 +#definer31 %r31 #defineR0 0 #defineR1 1 I think your patch was truncated somehow. How so? Mikey ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
[PATCH 5/15] powerpc: convert to %r for all GPR usage
Now all the fixes are in place, let's rock-n-roll! Signed-off-by: Michael Neuling mi...@neuling.org --- arch/powerpc/include/asm/ppc_asm.h | 72 - 1 file changed, 39 insertions(+), 33 deletions(-) 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) #definecr7 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. + */ -#definer0 0 -#definer1 1 -#definer2 2 -#definer3 3 -#definer4 4 -#definer5 5 -#definer6 6 -#definer7 7 -#definer8 8 -#definer9 9 -#definer10 10 -#definer11 11 -#definer12 12 -#definer13 13 -#definer14 14 -#definer15 15 -#definer16 16 -#definer17 17 -#definer18 18 -#definer19 19 -#definer20 20 -#definer21 21 -#definer22 22 -#definer23 23 -#definer24 24 -#definer25 25 -#definer26 26 -#definer27 27 -#definer28 28 -#definer29 29 -#definer30 30 -#definer31 31 +#definer0 %r0 +#definer1 %r1 +#definer2 %r2 +#definer3 %r3 +#definer4 %r4 +#definer5 %r5 +#definer6 %r6 +#definer7 %r7 +#definer8 %r8 +#definer9 %r9 +#definer10 %r10 +#definer11 %r11 +#definer12 %r12 +#definer13 %r13 +#definer14 %r14 +#definer15 %r15 +#definer16 %r16 +#definer17 %r17 +#definer18 %r18 +#definer19 %r19 +#definer20 %r20 +#definer21 %r21 +#definer22 %r22 +#definer23 %r23 +#definer24 %r24 +#definer25 %r25 +#definer26 %r26 +#definer27 %r27 +#definer28 %r28 +#definer29 %r29 +#definer30 %r30 +#definer31 %r31 #defineR0 0 #defineR1 1 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 5/15] powerpc: convert to %r for all GPR usage
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) jlar...@linux.vnet.ibm.com ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
