[Skiboot] [PATCH] mambo: Update P9 PVR to reflect Scale out 24 core chips

[Michael Neuling]

On Wed, 2017-06-14 at 20:16 +0530, Vaidyanathan Srinivasan wrote:
> * Michael Neuling <mikey at neuling.org> [2017-06-14 17:34:11]:
> 
> > The P9 PVR bits 48:51 don't indicate a revision but instead different
> > configurations.  From BookIV we have:
> >  Bits:  Configuration
> >     0:  Scale out 12 cores
> >     1:  Scale out 24 cores
> >     2:  Scale up 12 cores
> >     3:  Scale up 24 cores
> > 
> > Skiboot will mostly the use "Scale out 24 core" configuration
> > (ie. SMT4 not SMT8) so reflect this in mambo.
> > 
> > Signed-off-by: Michael Neuling <mikey at neuling.org>
> > ---
> >  external/mambo/skiboot.tcl | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/external/mambo/skiboot.tcl b/external/mambo/skiboot.tcl
> > index 9ac765dbb6..6032b6e215 100644
> > --- a/external/mambo/skiboot.tcl
> > +++ b/external/mambo/skiboot.tcl
> > @@ -82,8 +82,8 @@ if { $default_config == "PEGASUS" } {
> >      myconf config processor/initial/PVR 0x4b0201
> >  }
> >  if { $default_config == "P9" } {
> > -    # make sure we look like a POWER9 DD2
> > -    myconf config processor/initial/PVR 0x4e0200
> > +    # PVR configured for POWER9 DD2.0 Scale out 24 Core (ie SMT4)
> > +    myconf config processor/initial/PVR 0x4e1200
> 
>                                         0x4e4200 for scale out 24 cores??
> 
>                                         0x4e1200 means bit 3 (51 is set that is scale-up 24?)

Bits are around the other way.  The patch is right.

Mikey