[PATCH] intel powerclamp: support Knights Landing

[PATCH] intel powerclamp: support Knights Landing

[Dasaratharaman Chandramouli]

This patch enables intel_powerclamp driver to run on the
next-generation Intel(R) Xeon Phi Microarchitecture
code named "Knights Landing"

Signed-off-by: Dasaratharaman Chandramouli <dasaratharaman.chandramouli@intel.com>
---
 drivers/thermal/intel_powerclamp.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal/intel_powerclamp.c
index 12623bc..e34ccd5 100644
--- a/drivers/thermal/intel_powerclamp.c
+++ b/drivers/thermal/intel_powerclamp.c
@@ -690,6 +690,7 @@ static const struct x86_cpu_id intel_powerclamp_ids[] = {
 	{ X86_VENDOR_INTEL, 6, 0x4c},
 	{ X86_VENDOR_INTEL, 6, 0x4d},
 	{ X86_VENDOR_INTEL, 6, 0x56},
+	{ X86_VENDOR_INTEL, 6, 0x57},
 	{}
 };
 MODULE_DEVICE_TABLE(x86cpu, intel_powerclamp_ids);
-- 
1.8.1.5

Re: [PATCH] intel powerclamp: support Knights Landing

[Dasaratharaman Chandramouli]

Adding Jacob Pan

On Fri, 2015-04-17 at 15:31 -0700, Dasaratharaman Chandramouli wrote:
> This patch enables intel_powerclamp driver to run on the
> next-generation Intel(R) Xeon Phi Microarchitecture
> code named "Knights Landing"
> 
> Signed-off-by: Dasaratharaman Chandramouli <dasaratharaman.chandramouli@intel.com>
> ---
>  drivers/thermal/intel_powerclamp.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal/intel_powerclamp.c
> index 12623bc..e34ccd5 100644
> --- a/drivers/thermal/intel_powerclamp.c
> +++ b/drivers/thermal/intel_powerclamp.c
> @@ -690,6 +690,7 @@ static const struct x86_cpu_id intel_powerclamp_ids[] = {
>  	{ X86_VENDOR_INTEL, 6, 0x4c},
>  	{ X86_VENDOR_INTEL, 6, 0x4d},
>  	{ X86_VENDOR_INTEL, 6, 0x56},
> +	{ X86_VENDOR_INTEL, 6, 0x57},
>  	{}
>  };
>  MODULE_DEVICE_TABLE(x86cpu, intel_powerclamp_ids);

Re: [PATCH] intel powerclamp: support Knights Landing

[Jacob Pan]

On Fri, 24 Apr 2015 10:33:34 -0700
Dasaratharaman Chandramouli <dasaratharaman.chandramouli@intel.com>
wrote:

> Adding Jacob Pan
> 
> On Fri, 2015-04-17 at 15:31 -0700, Dasaratharaman Chandramouli wrote:
> > This patch enables intel_powerclamp driver to run on the
> > next-generation Intel(R) Xeon Phi Microarchitecture
> > code named "Knights Landing"
> > 
looks good.

Acked-by: Jacob Pan <jacob.jun.pan@linux.intel.com>

> > Signed-off-by: Dasaratharaman Chandramouli
> > <dasaratharaman.chandramouli@intel.com> ---
> >  drivers/thermal/intel_powerclamp.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/thermal/intel_powerclamp.c
> > b/drivers/thermal/intel_powerclamp.c index 12623bc..e34ccd5 100644
> > --- a/drivers/thermal/intel_powerclamp.c
> > +++ b/drivers/thermal/intel_powerclamp.c
> > @@ -690,6 +690,7 @@ static const struct x86_cpu_id
> > intel_powerclamp_ids[] = { { X86_VENDOR_INTEL, 6, 0x4c},
> >  	{ X86_VENDOR_INTEL, 6, 0x4d},
> >  	{ X86_VENDOR_INTEL, 6, 0x56},
> > +	{ X86_VENDOR_INTEL, 6, 0x57},
> >  	{}
> >  };
> >  MODULE_DEVICE_TABLE(x86cpu, intel_powerclamp_ids);
> 
> 




-- 
Jacob Pan

RE: [PATCH] intel powerclamp: support Knights Landing

[Chandramouli, Dasaratharaman]

Can we make this patch available for 4.2

> -----Original Message-----
> From: Jacob Pan [mailto:jacob.jun.pan@linux.intel.com]
> Sent: Friday, April 24, 2015 2:59 PM
> To: Chandramouli, Dasaratharaman
> Cc: Zhang, Rui; Eduardo Valentin; linux-pm@vger.kernel.org; linux-
> kernel@vger.kernel.org; Pan, Jacob jun; Rao, Nikhil
> Subject: Re: [PATCH] intel powerclamp: support Knights Landing
> 
> On Fri, 24 Apr 2015 10:33:34 -0700
> Dasaratharaman Chandramouli <dasaratharaman.chandramouli@intel.com>
> wrote:
> 
> > Adding Jacob Pan
> >
> > On Fri, 2015-04-17 at 15:31 -0700, Dasaratharaman Chandramouli wrote:
> > > This patch enables intel_powerclamp driver to run on the
> > > next-generation Intel(R) Xeon Phi Microarchitecture code named
> > > "Knights Landing"
> > >
> looks good.
> 
> Acked-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
> 
> > > Signed-off-by: Dasaratharaman Chandramouli
> > > <dasaratharaman.chandramouli@intel.com> ---
> > > drivers/thermal/intel_powerclamp.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/drivers/thermal/intel_powerclamp.c
> > > b/drivers/thermal/intel_powerclamp.c index 12623bc..e34ccd5 100644
> > > --- a/drivers/thermal/intel_powerclamp.c
> > > +++ b/drivers/thermal/intel_powerclamp.c
> > > @@ -690,6 +690,7 @@ static const struct x86_cpu_id
> > > intel_powerclamp_ids[] = { { X86_VENDOR_INTEL, 6, 0x4c},
> > >  	{ X86_VENDOR_INTEL, 6, 0x4d},
> > >  	{ X86_VENDOR_INTEL, 6, 0x56},
> > > +	{ X86_VENDOR_INTEL, 6, 0x57},
> > >  	{}
> > >  };
> > >  MODULE_DEVICE_TABLE(x86cpu, intel_powerclamp_ids);
> >
> >
> 
> 
> 
> 
> --
> Jacob Pan

Re: [PATCH] intel powerclamp: support Knights Landing

[Rafael J. Wysocki]

On Monday, June 22, 2015 04:51:13 PM Chandramouli, Dasaratharaman wrote:
> Can we make this patch available for 4.2

It is included in the power management and ACPI pull request for 4.1-rc1
I've just sent.


-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-pm" in

Re: [PATCH] intel powerclamp: support Knights Landing

[Dasaratharaman Chandramouli]

On Tue, 2015-06-23 at 01:39 +0200, Rafael J. Wysocki wrote:
> On Monday, June 22, 2015 04:51:13 PM Chandramouli, Dasaratharaman wrote:
> > Can we make this patch available for 4.2
> 
> It is included in the power management and ACPI pull request for 4.1-rc1
> I've just sent.

I just checked the branch pm+acpi-4.2-rc1 and i don't see the Knights
Landing patch for the powerclamp driver in that. Kindly verify that it
was added.  

https://lkml.org/lkml/2015/4/17/795

> 
> 

Re: [PATCH] intel powerclamp: support Knights Landing

[Rafael J. Wysocki]

On Tue, Jun 23, 2015 at 8:46 PM, Dasaratharaman Chandramouli
<dasaratharaman.chandramouli@intel.com> wrote:
> On Tue, 2015-06-23 at 01:39 +0200, Rafael J. Wysocki wrote:
>> On Monday, June 22, 2015 04:51:13 PM Chandramouli, Dasaratharaman wrote:
>> > Can we make this patch available for 4.2
>>
>> It is included in the power management and ACPI pull request for 4.1-rc1
>> I've just sent.
>
> I just checked the branch pm+acpi-4.2-rc1 and i don't see the Knights
> Landing patch for the powerclamp driver in that. Kindly verify that it
> was added.
>
> https://lkml.org/lkml/2015/4/17/795

I was talking about the RAPL driver, sorry for the noise.

The powerclamp one is the Rui's area.  Rui, what's the status of this one?

Rafael

RE: [PATCH] intel powerclamp: support Knights Landing

[Zhang, Rui]

It has been in my  -next branch and I will send it out for 4.2-rc1.

Thanks,
rui

> -----Original Message-----
> From: Chandramouli, Dasaratharaman
> Sent: Wednesday, June 24, 2015 2:46 AM
> To: Rafael J. Wysocki
> Cc: Jacob Pan; Zhang, Rui; Eduardo Valentin; linux-pm@vger.kernel.org;
> linux-kernel@vger.kernel.org; Pan, Jacob jun; Rao, Nikhil
> Subject: Re: [PATCH] intel powerclamp: support Knights Landing
> Importance: High
> 
> On Tue, 2015-06-23 at 01:39 +0200, Rafael J. Wysocki wrote:
> > On Monday, June 22, 2015 04:51:13 PM Chandramouli, Dasaratharaman
> wrote:
> > > Can we make this patch available for 4.2
> >
> > It is included in the power management and ACPI pull request for
> > 4.1-rc1 I've just sent.
> 
> I just checked the branch pm+acpi-4.2-rc1 and i don't see the Knights Landing
> patch for the powerclamp driver in that. Kindly verify that it was added.
> 
> https://lkml.org/lkml/2015/4/17/795
> 
> >
> >
>