On Sat, 2020-02-15 at 03:23 -0300, Leonardo Bras wrote: > Mahesh Salgaonkar <mah...@linux.vnet.ibm.com> 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.000000MHz > > 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.000000MHz > > revision : 1.0 (pvr 004f 1100) > > > > Signed-off-by: Mahesh Salgaonkar <mah...@linux.vnet.ibm.com> > > --- > > 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 <leona...@linux.ibm.com>
Sorry, typo. Reviewed-by: Leonardo Bras <leona...@linux.ibm.com>
signature.asc
Description: This is a digitally signed message part
