Re: [PATCH] The x86 cpu RAPL id was added in to support BXT core

[Jacob Pan <jacob.jun.pan@linux.intel.com>]

On Tue, 13 Oct 2015 23:07:21 +0000
"Pandruvada, Srinivas" <srinivas.pandruvada@intel.com> wrote:

> On Tue, 2015-10-13 at 08:59 -0700, amy.l.wiles@intel.com wrote:
> > From: Amy Wiles <amy.l.wiles@intel.com>
> > 
> > BUG=None
> > TEST=Navigate to /sys/class/powercap and check that the directory 
> > intel-rapl:0 is present
> Remove above lines and just say "Enabled Broxton RAPL support".
> Copied to Jacob.
> 
agreed, other than that looks good to me.
> Thanks,
> Srinivas
> 
> 
> > Signed-off-by: Amy Wiles <amy.l.wiles@intel.com>
> > ---
> >  drivers/powercap/intel_rapl.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/powercap/intel_rapl.c
> > b/drivers/powercap/intel_rapl.c
> > index 5efacd0..cc97f08 100644
> > --- a/drivers/powercap/intel_rapl.c
> > +++ b/drivers/powercap/intel_rapl.c
> > @@ -1102,6 +1102,7 @@ static const struct x86_cpu_id rapl_ids[]
> > __initconst = {
> >  	RAPL_CPU(0x4A, rapl_defaults_tng),/* Tangier */
> >  	RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
> >  	RAPL_CPU(0x5A, rapl_defaults_ann),/* Annidale */
> > +	RAPL_CPU(0X5C, rapl_defaults_core),/* Broxton */
> >  	RAPL_CPU(0x5E, rapl_defaults_core),/* Skylake-H/S */
> >  	RAPL_CPU(0x57, rapl_defaults_hsw_server),/* Knights Landing
> > */
> >  	{}
[Jacob Pan]