Subject: [PATCH] intel powerclamp: support Knights Landing

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 <[email protected]>
---
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


Subject: Re: [PATCH] intel powerclamp: support Knights Landing

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 <[email protected]>
> ---
> 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);

2015-04-24 21:58:39

by Jacob Pan

[permalink] [raw]
Subject: Re: [PATCH] intel powerclamp: support Knights Landing

On Fri, 24 Apr 2015 10:33:34 -0700
Dasaratharaman Chandramouli <[email protected]>
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 <[email protected]>

> > Signed-off-by: Dasaratharaman Chandramouli
> > <[email protected]> ---
> > 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

Subject: RE: [PATCH] intel powerclamp: support Knights Landing

Can we make this patch available for 4.2

> -----Original Message-----
> From: Jacob Pan [mailto:[email protected]]
> Sent: Friday, April 24, 2015 2:59 PM
> To: Chandramouli, Dasaratharaman
> Cc: Zhang, Rui; Eduardo Valentin; [email protected]; linux-
> [email protected]; Pan, Jacob jun; Rao, Nikhil
> Subject: Re: [PATCH] intel powerclamp: support Knights Landing
>
> On Fri, 24 Apr 2015 10:33:34 -0700
> Dasaratharaman Chandramouli <[email protected]>
> 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 <[email protected]>
>
> > > Signed-off-by: Dasaratharaman Chandramouli
> > > <[email protected]> ---
> > > 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

2015-06-22 23:13:13

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [PATCH] intel powerclamp: support Knights Landing

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.

Subject: Re: [PATCH] intel powerclamp: support Knights Landing

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

>
>

2015-06-23 19:51:42

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [PATCH] intel powerclamp: support Knights Landing

On Tue, Jun 23, 2015 at 8:46 PM, Dasaratharaman Chandramouli
<[email protected]> 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

2015-06-24 01:46:46

by Zhang, Rui

[permalink] [raw]
Subject: RE: [PATCH] intel powerclamp: support Knights Landing

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; [email protected];
> [email protected]; 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
>
> >
> >
>

????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?