lkml.org 
[lkml]   [2012]   [Dec]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] cpuidle - remove the power_specified field in the driver
Date
On Wednesday, December 12, 2012 09:00:53 PM Daniel Lezcano wrote:
> On 12/12/2012 07:50 PM, Julius Werner wrote:
> > Thanks again for making this happen, Daniel. I like this version,
> > except for the small nitpick that I still think it would make sense to
> > also turn the loop in menu.c around. How about something like this:
> >
> > for (i = drv->state_count - 1; i >= CPUIDLE_DRIVER_STATE_START; i++) {
> > struct cpuidle_state *s = &drv->states[i];
> > if (!s->disable && s->exit_latency <= latency_req &&
> > s->target_residency <= data->predicted_us &&
> > s->exit_latency * multiplier <= data->predicted_us) {
> > data->last_state_idx = i;
> > data->exit_us = s->exit_latency;
> > break;
> > }
> > }
>
> Actually I was planning to do that in a separate patch.

Can you submit that second patch too, please, so that people don't have to
wonder?

Rafael


--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.


\
 
 \ /
  Last update: 2012-12-13 00:01    [W:0.038 / U:4.852 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site