All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] Enabled Broxton SoC thermal reporting device
@ 2015-10-14 15:09 amy.l.wiles
  2015-10-14 15:09 ` [PATCH 2/2] Enabled Broxton RAPL support amy.l.wiles
  0 siblings, 1 reply; 5+ messages in thread
From: amy.l.wiles @ 2015-10-14 15:09 UTC (permalink / raw)
  To: rui.zhang, srinivas.pandruvada, linux-pm; +Cc: Amy Wiles

From: Amy Wiles <amy.l.wiles@intel.com>

Signed-off-by: Amy Wiles <amy.l.wiles@intel.com>
---
 drivers/thermal/int340x_thermal/processor_thermal_device.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/drivers/thermal/int340x_thermal/processor_thermal_device.c b/drivers/thermal/int340x_thermal/processor_thermal_device.c
index ccc0ad0..36fa724 100644
--- a/drivers/thermal/int340x_thermal/processor_thermal_device.c
+++ b/drivers/thermal/int340x_thermal/processor_thermal_device.c
@@ -33,6 +33,12 @@
 /* Braswell thermal reporting device */
 #define PCI_DEVICE_ID_PROC_BSW_THERMAL	0x22DC
 
+/* Broxton thermal reporting device */
+#define PCI_DEVICE_ID_PROC_BXT0_THERMAL  0x0A8C
+#define PCI_DEVICE_ID_PROC_BXT1_THERMAL  0x1A8C
+#define PCI_DEVICE_ID_PROC_BXTX_THERMAL  0x4A8C
+#define PCI_DEVICE_ID_PROC_BXTP_THERMAL  0x5A8C
+
 struct power_config {
 	u32	index;
 	u32	min_uw;
@@ -404,6 +410,10 @@ static const struct pci_device_id proc_thermal_pci_ids[] = {
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_PROC_HSB_THERMAL)},
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_PROC_SKL_THERMAL)},
 	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_PROC_BSW_THERMAL)},
+	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_PROC_BXT0_THERMAL)},
+	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_PROC_BXT1_THERMAL)},
+	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_PROC_BXTX_THERMAL)},
+	{ PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_PROC_BXTP_THERMAL)},
 	{ 0, },
 };
 
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread
* [PATCH 2/2] Enabled Broxton RAPL support
  2015-10-14 15:09 [PATCH 1/2] Enabled Broxton SoC thermal reporting device amy.l.wiles
@ 2015-10-14 15:09 ` amy.l.wiles
  2015-10-14 23:29   ` Rafael J. Wysocki
  0 siblings, 1 reply; 5+ messages in thread
From: amy.l.wiles @ 2015-10-14 15:09 UTC (permalink / raw)
  To: rui.zhang, srinivas.pandruvada, linux-pm; +Cc: Amy Wiles

From: Amy Wiles <amy.l.wiles@intel.com>

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 */
 	{}
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] Enabled Broxton RAPL support
  2015-10-14 15:09 ` [PATCH 2/2] Enabled Broxton RAPL support amy.l.wiles
@ 2015-10-14 23:29   ` Rafael J. Wysocki
  2015-10-15  3:11     ` Jacob Pan
  0 siblings, 1 reply; 5+ messages in thread
From: Rafael J. Wysocki @ 2015-10-14 23:29 UTC (permalink / raw)
  To: amy.l.wiles, Jacob Pan Jun; +Cc: rui.zhang, srinivas.pandruvada, linux-pm

On Wednesday, October 14, 2015 08:09:15 AM amy.l.wiles@intel.com wrote:
> From: Amy Wiles <amy.l.wiles@intel.com>
> 
> 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, any problems with this one?

Thanks,
Rafael


^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] Enabled Broxton RAPL support
  2015-10-14 23:29   ` Rafael J. Wysocki
@ 2015-10-15  3:11     ` Jacob Pan
  2015-10-19 21:52       ` Rafael J. Wysocki
  0 siblings, 1 reply; 5+ messages in thread
From: Jacob Pan @ 2015-10-15  3:11 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: amy.l.wiles, rui.zhang, srinivas.pandruvada, linux-pm, jacob.jun.pan

On Thu, 15 Oct 2015 01:29:04 +0200
"Rafael J. Wysocki" <rjw@rjwysocki.net> wrote:

> On Wednesday, October 14, 2015 08:09:15 AM amy.l.wiles@intel.com
> wrote:
> > From: Amy Wiles <amy.l.wiles@intel.com>
> > 
> > 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, any problems with this one?
> 
looks good to me. I also tested it.
> Thanks,
> Rafael
> 

[Jacob Pan]

^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] Enabled Broxton RAPL support
  2015-10-15  3:11     ` Jacob Pan
@ 2015-10-19 21:52       ` Rafael J. Wysocki
  0 siblings, 0 replies; 5+ messages in thread
From: Rafael J. Wysocki @ 2015-10-19 21:52 UTC (permalink / raw)
  To: Jacob Pan; +Cc: amy.l.wiles, rui.zhang, srinivas.pandruvada, linux-pm

On Wednesday, October 14, 2015 08:11:58 PM Jacob Pan wrote:
> On Thu, 15 Oct 2015 01:29:04 +0200
> "Rafael J. Wysocki" <rjw@rjwysocki.net> wrote:
> 
> > On Wednesday, October 14, 2015 08:09:15 AM amy.l.wiles@intel.com
> > wrote:
> > > From: Amy Wiles <amy.l.wiles@intel.com>
> > > 
> > > 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, any problems with this one?
> > 
> looks good to me. I also tested it.

OK, applied.

Thanks,
Rafael


^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-10-19 21:23 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-14 15:09 [PATCH 1/2] Enabled Broxton SoC thermal reporting device amy.l.wiles
2015-10-14 15:09 ` [PATCH 2/2] Enabled Broxton RAPL support amy.l.wiles
2015-10-14 23:29   ` Rafael J. Wysocki
2015-10-15  3:11     ` Jacob Pan
2015-10-19 21:52       ` Rafael J. Wysocki

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.