[PATCH v1 1/1] intel_idle: Add CPU model 74 (Atom Z34xx series)

Add CPU ID for Atom Z34xx processors. Datasheets indicate support for this,
detailed information about potential quirks or limitations are missing, though.
So we just reuse the definition for the previous ATOM series. The quite similar
values (C1 is slightly deviated) are used in the official Intel Edison BSP.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/idle/intel_idle.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/idle/intel_idle.c b/drivers/idle/intel_idle.c
index c966492..bb52275 100644
--- a/drivers/idle/intel_idle.c
+++ b/drivers/idle/intel_idle.c

@@ -1033,6 +1033,7 @@ static const struct x86_cpu_id intel_idle_ids[] __initconst = {
 	ICPU(0x2d, idle_cpu_snb),
 	ICPU(0x36, idle_cpu_atom),
 	ICPU(0x37, idle_cpu_byt),
+	ICPU(0x4a, idle_cpu_atom),
 	ICPU(0x4c, idle_cpu_cht),
 	ICPU(0x3a, idle_cpu_ivb),
 	ICPU(0x3e, idle_cpu_ivt),

-- 
2.8.1

Hi Andy,

Is this a patch from the Android kernel tree?
Is there anybody testing the upstream kernel on this hardware?

Past efforts to support old hardware that nobody tests upstream
have resulted in all pain and no gain, and I'm not eager to
repeat that experience.

thanks,
-Len


On Mon, Jun 6, 2016 at 1:06 PM, Andy Shevchenko
[off-list ref] wrote:

-- 
Len Brown, Intel Open Source Technology Center

On Wed, 2016-06-15 at 21:14 -0400, Len Brown wrote:

Hi Andy,

Is this a patch from the Android kernel tree?

Nope. Though official Intel Yocto BSP containts almost same values for
this CPU, except for C1 (I decided no to take their since they also
provide different values for that state for BayTrail. And latter one is
in upstream). 

Is there anybody testing the upstream kernel on this hardware?

I did. It works and I see changing values under sysfs. I'm not sure what
should I get from turbostat. Do I need to patch it as well to be able to
get something useful?

Past efforts to support old hardware that nobody tests upstream
have resulted in all pain and no gain, and I'm not eager to
repeat that experience.

thanks,
-Len


On Mon, Jun 6, 2016 at 1:06 PM, Andy Shevchenko
[off-list ref] wrote:

quoted

Add CPU ID for Atom Z34xx processors. Datasheets indicate support
for this,
detailed information about potential quirks or limitations are
missing, though.
So we just reuse the definition for the previous ATOM series. The
quite similar
values (C1 is slightly deviated) are used in the official Intel
Edison BSP.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/idle/intel_idle.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/idle/intel_idle.c b/drivers/idle/intel_idle.c
index c966492..bb52275 100644
--- a/drivers/idle/intel_idle.c
+++ b/drivers/idle/intel_idle.c

@@ -1033,6 +1033,7 @@ static const struct x86_cpu_id

intel_idle_ids[] __initconst = {
        ICPU(0x2d, idle_cpu_snb),
        ICPU(0x36, idle_cpu_atom),
        ICPU(0x37, idle_cpu_byt),
+       ICPU(0x4a, idle_cpu_atom),
        ICPU(0x4c, idle_cpu_cht),
        ICPU(0x3a, idle_cpu_ivb),
        ICPU(0x3e, idle_cpu_ivt),
--
2.8.1

-- 

Andy Shevchenko [off-list ref]
Intel Finland Oy