| Message ID | 1465232806-138619-1-git-send-email-andriy.shevchenko@linux.intel.com (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
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 <andriy.shevchenko@linux.intel.com> wrote: > 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 >
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 > <andriy.shevchenko@linux.intel.com> wrote: > > 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 > > > > >
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),
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(+)