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
 |