Re: [PATCH] cpuidle: menu: allow state 0 to be disabled

2017-06-30 Thread Nicholas Piggin
On Thu, 29 Jun 2017 22:57:33 +0200 "Rafael J. Wysocki" wrote: > On Mon, Jun 26, 2017 at 7:38 AM, Nicholas Piggin wrote: > > The menu driver does not allow state0 to be disabled completely. > > If it is disabled but other enabled states don't meet latency > > requirements, it is still used. > > >

Re: [PATCH] cpuidle: menu: allow state 0 to be disabled

2017-06-29 Thread Rafael J. Wysocki
On Mon, Jun 26, 2017 at 7:38 AM, Nicholas Piggin wrote: > The menu driver does not allow state0 to be disabled completely. > If it is disabled but other enabled states don't meet latency > requirements, it is still used. > > Fix this by starting with the first enabled idle state. Fall back > to st

[PATCH] cpuidle: menu: allow state 0 to be disabled

2017-06-25 Thread Nicholas Piggin
The menu driver does not allow state0 to be disabled completely. If it is disabled but other enabled states don't meet latency requirements, it is still used. Fix this by starting with the first enabled idle state. Fall back to state 0 if no idle states are enabled (arguably this should be -EINVAL