Messages in this thread | | | Date | Mon, 7 Dec 2009 16:26:16 +0530 | From | Arun R Bharadwaj <> | Subject | Re: [v10 PATCH 2/9]: cpuidle: cleanup drivers/cpuidle/cpuidle.c |
| |
* Torsten Duwe <duwe@lst.de> [2009-12-07 11:17:57]:
> On Sunday 06 December 2009, Arun R Bharadwaj wrote: > > > Peter objected to the idea of integrating this with the old pm_idle > > because it has already caused a lot of problems on x86 and we wouldn't > > want to be doing the same mistake on POWER. The discussion related to > > that could be found here http://lkml.org/lkml/2009/8/26/233 > > And BenH has sketched how it should be done on ppc, in that thread: > http://lkml.org/lkml/2009/8/26/624 AFAIS this comment is still valid for v10. > > Not only I would like to understand what is the conceptual idea behind the > other changes. Nothing wrong with cleanups, but there's got to be a purpose > and benefits. > > Torsten
The reason for the cleanups is that we should have just one idle function manager instead of having one for each arch, which needs to be exported and hence really ugly. So thats why we decided to do away with pm_idle and make cpuidle as _the_ idle function manager. So in case of POWER, we have the ppc_md.power_save which is the pm_idle equivalent. We discussed that in this thread http://lkml.org/lkml/2009/9/2/20
thanks arun
| |