Re: [PATCH] powerpc: Fix /proc/cpuinfo revision for POWER9P processors
On Sat, 2020-02-15 at 03:23 -0300, Leonardo Bras wrote: > Mahesh Salgaonkar writes: > > Hello Mahesh, > > > POWER9P PVR bits are same as that of POWER9. Hence mask off only the > > relevant bits for the major revision similar to POWER9. > > > > Without this patch the cpuinfo output shows 17.0 as revision: > > > > $ cat /proc/cpuinfo > > processor : 0 > > cpu : POWER9P, altivec supported > > clock : 2950.00MHz > > revision : 17.0 (pvr 004f 1100) > > > > With this change it reflects the correct revision: > > > > $ cat /proc/cpuinfo > > processor : 0 > > cpu : POWER9P, altivec supported > > clock : 2633.00MHz > > revision : 1.0 (pvr 004f 1100) > > > > Signed-off-by: Mahesh Salgaonkar > > --- > > arch/powerpc/kernel/setup-common.c |1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/arch/powerpc/kernel/setup-common.c > > b/arch/powerpc/kernel/setup-common.c > > index 488f1eecc0de..37342ec42cb3 100644 > > --- a/arch/powerpc/kernel/setup-common.c > > +++ b/arch/powerpc/kernel/setup-common.c > > @@ -315,6 +315,7 @@ static int show_cpuinfo(struct seq_file *m, void *v) > > min = pvr & 0xFF; > > break; > > case 0x004e: /* POWER9 bits 12-15 give chip type */ > > + case 0x004f: /* POWER9P bits 12-15 give chip type */ > > maj = (pvr >> 8) & 0x0F; > > min = pvr & 0xFF; > > break; > > Seems fine to me. > > FWIW: > Reviwed-by: Leonardo Bras Sorry, typo. Reviewed-by: Leonardo Bras signature.asc Description: This is a digitally signed message part
Re: [PATCH] powerpc: Fix /proc/cpuinfo revision for POWER9P processors
Mahesh Salgaonkar writes: Hello Mahesh, > POWER9P PVR bits are same as that of POWER9. Hence mask off only the > relevant bits for the major revision similar to POWER9. > > Without this patch the cpuinfo output shows 17.0 as revision: > > $ cat /proc/cpuinfo > processor : 0 > cpu : POWER9P, altivec supported > clock : 2950.00MHz > revision : 17.0 (pvr 004f 1100) > > With this change it reflects the correct revision: > > $ cat /proc/cpuinfo > processor : 0 > cpu : POWER9P, altivec supported > clock : 2633.00MHz > revision : 1.0 (pvr 004f 1100) > > Signed-off-by: Mahesh Salgaonkar > --- > arch/powerpc/kernel/setup-common.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/powerpc/kernel/setup-common.c > b/arch/powerpc/kernel/setup-common.c > index 488f1eecc0de..37342ec42cb3 100644 > --- a/arch/powerpc/kernel/setup-common.c > +++ b/arch/powerpc/kernel/setup-common.c > @@ -315,6 +315,7 @@ static int show_cpuinfo(struct seq_file *m, void *v) > min = pvr & 0xFF; > break; > case 0x004e: /* POWER9 bits 12-15 give chip type */ > + case 0x004f: /* POWER9P bits 12-15 give chip type */ > maj = (pvr >> 8) & 0x0F; > min = pvr & 0xFF; > break; Seems fine to me. FWIW: Reviwed-by: Leonardo Bras signature.asc Description: This is a digitally signed message part
Re: [PATCH] powerpc: Fix /proc/cpuinfo revision for POWER9P processors
Mahesh Salgaonkar writes: Hello Mahesh, > POWER9P PVR bits are same as that of POWER9. Hence mask off only the > relevant bits for the major revision similar to POWER9. > > Without this patch the cpuinfo output shows 17.0 as revision: > > $ cat /proc/cpuinfo > processor : 0 > cpu : POWER9P, altivec supported > clock : 2950.00MHz > revision : 17.0 (pvr 004f 1100) > > With this change it reflects the correct revision: > > $ cat /proc/cpuinfo > processor : 0 > cpu : POWER9P, altivec supported > clock : 2633.00MHz > revision : 1.0 (pvr 004f 1100) > > Signed-off-by: Mahesh Salgaonkar > --- > arch/powerpc/kernel/setup-common.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/powerpc/kernel/setup-common.c > b/arch/powerpc/kernel/setup-common.c > index 488f1eecc0de..37342ec42cb3 100644 > --- a/arch/powerpc/kernel/setup-common.c > +++ b/arch/powerpc/kernel/setup-common.c > @@ -315,6 +315,7 @@ static int show_cpuinfo(struct seq_file *m, void *v) > min = pvr & 0xFF; > break; > case 0x004e: /* POWER9 bits 12-15 give chip type */ > + case 0x004f: /* POWER9P bits 12-15 give chip type */ > maj = (pvr >> 8) & 0x0F; > min = pvr & 0xFF; > break; Seems fine to me. FWIW: Reviwed-by: Leonardo Bras signature.asc Description: This is a digitally signed message part
[PATCH] powerpc: Fix /proc/cpuinfo revision for POWER9P processors
POWER9P PVR bits are same as that of POWER9. Hence mask off only the relevant bits for the major revision similar to POWER9. Without this patch the cpuinfo output shows 17.0 as revision: $ cat /proc/cpuinfo processor : 0 cpu : POWER9P, altivec supported clock : 2950.00MHz revision: 17.0 (pvr 004f 1100) With this change it reflects the correct revision: $ cat /proc/cpuinfo processor : 0 cpu : POWER9P, altivec supported clock : 2633.00MHz revision: 1.0 (pvr 004f 1100) Signed-off-by: Mahesh Salgaonkar --- arch/powerpc/kernel/setup-common.c |1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/kernel/setup-common.c b/arch/powerpc/kernel/setup-common.c index 488f1eecc0de..37342ec42cb3 100644 --- a/arch/powerpc/kernel/setup-common.c +++ b/arch/powerpc/kernel/setup-common.c @@ -315,6 +315,7 @@ static int show_cpuinfo(struct seq_file *m, void *v) min = pvr & 0xFF; break; case 0x004e: /* POWER9 bits 12-15 give chip type */ + case 0x004f: /* POWER9P bits 12-15 give chip type */ maj = (pvr >> 8) & 0x0F; min = pvr & 0xFF; break;
