Re: [PATCH] powercap: intel_rapl: Add support for RAPTORLAKE_P

["Sworo, George D" <george.d.sworo@intel.com>]

On Tue, 2022-06-21 at 14:49 +0000, Pawnikar, Sumeet R wrote:
> > -----Original Message-----
> > From: Sworo, George D <george.d.sworo@intel.com>
> > Sent: Friday, June 10, 2022 9:02 AM
> > To: Zhang, Rui <rui.zhang@intel.com>; 
> > srinivas.pandruvada@linux.intel.com;
> > rafael@kernel.org
> > Cc: linux-pm@vger.kernel.org
> > Subject: Re: [PATCH] powercap: intel_rapl: Add support for
> > RAPTORLAKE_P
> > 
> > On Fri, 2022-06-10 at 10:26 +0800, Zhang Rui wrote:
> > > On Thu, 2022-06-09 at 20:55 +0200, Rafael J. Wysocki wrote:
> > > > Rui, Srinivas, should I take this one?
> > > 
> > > Sorry that I missed this one.
> > > 
> > > George,
> > > 
> > > Have this patch been tested yet?
> > yes. This is tested on RPL SOC. Thanks
> > > I have the same patch internally, it should work but I haven't
> > > got
> > > chance to test it yet.
> > > 
> > > thanks,
> > > rui
> > > > On Wed, Jun 1, 2022 at 9:22 PM <george.d.sworo@intel.com>
> > > > wrote:
> > > > > From: George D Sworo <george.d.sworo@intel.com>
> > > > > 
> > > > > Add RAPTORLAKE_P to the list of supported processor model in
> > > > > the
> > > > > Intel
> > > > > 
> > > > > RAPL power capping driver.
> > > > > 
> > > > > Signed-off-by: George D Sworo <george.d.sworo@intel.com>
> 
> Hi George, 
> Thank you for this patch. 
> I have tested this on Raptor Lake system and it works. 
> 
> Tested-by: Sumeet Pawnikar <sumeet.r.pawnikar@intel.com> 
> 
> Thanks,
> Sumeet. 
> 
Thanks Sumeet for confirming!
@Rafael, let me know if you have further comments
> > > > > ---
> > > > >  drivers/powercap/intel_rapl_common.c | 1 +
> > > > >  1 file changed, 1 insertion(+)
> > > > > 
> > > > > diff --git a/drivers/powercap/intel_rapl_common.c
> > > > > b/drivers/powercap/intel_rapl_common.c
> > > > > index a9c99d9e8b42..21d624f9f5fb 100644
> > > > > --- a/drivers/powercap/intel_rapl_common.c
> > > > > +++ b/drivers/powercap/intel_rapl_common.c
> > > > > @@ -1109,6 +1109,7 @@ static const struct x86_cpu_id
> > > > > rapl_ids[]
> > > > > __initconst = {
> > > > >         X86_MATCH_INTEL_FAM6_MODEL(ALDERLAKE_L,         &rapl
> > > > > _def
> > > > > au
> > > > > lts_core),
> > > > >         X86_MATCH_INTEL_FAM6_MODEL(ALDERLAKE_N,         &rapl
> > > > > _def
> > > > > au
> > > > > lts_core),
> > > > >         X86_MATCH_INTEL_FAM6_MODEL(RAPTORLAKE,          &rapl
> > > > > _def
> > > > > au
> > > > > lts_core),
> > > > > +       X86_MATCH_INTEL_FAM6_MODEL(RAPTORLAKE_P,        &rapl
> > > > > _def
> > > > > au
> > > > > lts_core),
> > > > >         X86_MATCH_INTEL_FAM6_MODEL(SAPPHIRERAPIDS_X,    &rapl
> > > > > _def
> > > > > au
> > > > > lts_spr_server),
> > > > >         X86_MATCH_INTEL_FAM6_MODEL(LAKEFIELD,           &rapl
> > > > > _def
> > > > > au
> > > > > lts_core),
> > > > > 
> > > > > --
> > > > > 2.17.1
> > > > >