[PATCH V3 15/17] ARM: exynos: cpuidle: Disable cpuidle for 5440

[Nicolas Pitre]

On Tue, 8 Apr 2014, Daniel Lezcano wrote:

> On 04/08/2014 02:51 PM, Amit Kucheria wrote:
> > On Tue, Apr 8, 2014 at 5:49 PM, Daniel Lezcano
> > <daniel.lezcano at linaro.org <mailto:daniel.lezcano at linaro.org>> wrote:
> >
> >     There is no point to register the cpuidle driver for the 5440 as it
> >     has only
> >     one WFI state which is the default idle function when the cpuidle
> >     driver is
> >     disabled.
> >
> >     By disabling cpuidle we prevent to enter to the governor computation for
> >     nothing, thus saving a lot of processing time.
> >
> >     The only drawback is the statistic via sysfs on this state which is
> >     lost but
> >     it is meaningless and it could be retrieved from the ftrace easily.
> >
> >
> > So for the future, you'll only merge platform drivers that enable
> > something more than WFI?
> 
> Well, I already picked up a driver with only WFI because I knew the next idle
> states were in preparation. But adding a cpuidle driver just for WFI is no
> sense and if, for a reason I missed, it is really needed, I guess a generic
> WFI cpuidle driver for all platforms would be make much more sense.

Better relegate WFI-only configurations to arch_cpu_idle().


Nicolas