linux-pm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] powercap/rapl: add ids for future xeon cpus
@ 2014-12-15 18:14 Jacob Pan
  2014-12-16  0:36 ` Guenter Roeck
  0 siblings, 1 reply; 4+ messages in thread
From: Jacob Pan @ 2014-12-15 18:14 UTC (permalink / raw)
  To: LKML, Linux PM, Rafael Wysocki; +Cc: Srinivas Pandruvada, Jacob Pan

Enable RAPL driver on Xeon cpu id 0x56.

Signed-off-by: Jacob Pan <jacob.jun.pan@linux.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 c71443c..ee00ffb 100644
--- a/drivers/powercap/intel_rapl.c
+++ b/drivers/powercap/intel_rapl.c
@@ -1041,6 +1041,7 @@ static const struct x86_cpu_id rapl_ids[] = {
 	RAPL_CPU(0x45, rapl_defaults_core),/* Haswell ULT */
 	RAPL_CPU(0x4C, rapl_defaults_atom),/* Braswell */
 	RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
+	RAPL_CPU(0x56, rapl_defaults_atom),/* Future Xeon */
 	RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
 	{}
 };
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/2] powercap/rapl: add ids for future xeon cpus
  2014-12-15 18:14 [PATCH 1/2] powercap/rapl: add ids for future xeon cpus Jacob Pan
@ 2014-12-16  0:36 ` Guenter Roeck
  2014-12-16 17:41   ` Jacob Pan
  0 siblings, 1 reply; 4+ messages in thread
From: Guenter Roeck @ 2014-12-16  0:36 UTC (permalink / raw)
  To: Jacob Pan; +Cc: LKML, Linux PM, Rafael Wysocki, Srinivas Pandruvada

On Mon, Dec 15, 2014 at 10:14:15AM -0800, Jacob Pan wrote:
> Enable RAPL driver on Xeon cpu id 0x56.
> 
> Signed-off-by: Jacob Pan <jacob.jun.pan@linux.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 c71443c..ee00ffb 100644
> --- a/drivers/powercap/intel_rapl.c
> +++ b/drivers/powercap/intel_rapl.c
> @@ -1041,6 +1041,7 @@ static const struct x86_cpu_id rapl_ids[] = {
>  	RAPL_CPU(0x45, rapl_defaults_core),/* Haswell ULT */
>  	RAPL_CPU(0x4C, rapl_defaults_atom),/* Braswell */
>  	RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
> +	RAPL_CPU(0x56, rapl_defaults_atom),/* Future Xeon */

Is rapl_defaults_atom correct, or should this be rapl_defaults_core ?

Guenter

>  	RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
>  	{}
>  };
> -- 
> 1.9.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/2] powercap/rapl: add ids for future xeon cpus
  2014-12-16  0:36 ` Guenter Roeck
@ 2014-12-16 17:41   ` Jacob Pan
  0 siblings, 0 replies; 4+ messages in thread
From: Jacob Pan @ 2014-12-16 17:41 UTC (permalink / raw)
  To: Guenter Roeck; +Cc: LKML, Linux PM, Rafael Wysocki, Srinivas Pandruvada

On Mon, 15 Dec 2014 16:36:48 -0800
Guenter Roeck <linux@roeck-us.net> wrote:

> On Mon, Dec 15, 2014 at 10:14:15AM -0800, Jacob Pan wrote:
> > Enable RAPL driver on Xeon cpu id 0x56.
> > 
> > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.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 c71443c..ee00ffb 100644
> > --- a/drivers/powercap/intel_rapl.c
> > +++ b/drivers/powercap/intel_rapl.c
> > @@ -1041,6 +1041,7 @@ static const struct x86_cpu_id rapl_ids[] = {
> >  	RAPL_CPU(0x45, rapl_defaults_core),/* Haswell ULT */
> >  	RAPL_CPU(0x4C, rapl_defaults_atom),/* Braswell */
> >  	RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
> > +	RAPL_CPU(0x56, rapl_defaults_atom),/* Future Xeon */
> 
> Is rapl_defaults_atom correct, or should this be rapl_defaults_core ?
> 
you are right should be core, will fix.

thanks
> Guenter
> 
> >  	RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
> >  	{}
> >  };
> > -- 
> > 1.9.1
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe
> > linux-kernel" in the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at  http://www.tux.org/lkml/
> > 
> > 

[Jacob Pan]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] powercap/rapl: add ids for future xeon cpus
@ 2014-12-16 17:52 Jacob Pan
  0 siblings, 0 replies; 4+ messages in thread
From: Jacob Pan @ 2014-12-16 17:52 UTC (permalink / raw)
  To: LKML, Linux PM, Rafael Wysocki; +Cc: Srinivas Pandruvada, Jacob Pan

Enable RAPL driver on Xeon cpu id 0x56.

Signed-off-by: Jacob Pan <jacob.jun.pan@linux.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 c71443c..97b5e4e 100644
--- a/drivers/powercap/intel_rapl.c
+++ b/drivers/powercap/intel_rapl.c
@@ -1041,6 +1041,7 @@ static const struct x86_cpu_id rapl_ids[] = {
 	RAPL_CPU(0x45, rapl_defaults_core),/* Haswell ULT */
 	RAPL_CPU(0x4C, rapl_defaults_atom),/* Braswell */
 	RAPL_CPU(0x4A, rapl_defaults_atom),/* Tangier */
+	RAPL_CPU(0x56, rapl_defaults_core),/* Future Xeon */
 	RAPL_CPU(0x5A, rapl_defaults_atom),/* Annidale */
 	{}
 };
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-12-16 18:23 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-15 18:14 [PATCH 1/2] powercap/rapl: add ids for future xeon cpus Jacob Pan
2014-12-16  0:36 ` Guenter Roeck
2014-12-16 17:41   ` Jacob Pan
  -- strict thread matches above, loose matches on Subject: below --
2014-12-16 17:52 Jacob Pan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).