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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx <mailto:daniel.lezcano@xxxxxxxxxx>> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux