On Fri, Aug 31, 2012 at 11:19:15PM +0200, Daniel Lezcano wrote: > On 07/27/2012 01:28 PM, Peter De Schrijver wrote: > > On Wed, Jul 25, 2012 at 12:45:57PM +0200, Daniel Lezcano wrote: > >> Since commit 46bcfad7a819bd17ac4e831b04405152d59784ab, > >> cpuidle: Single/Global registration of idle states > >> > >> we have a single registration for the cpuidle states which makes > >> sense. But now two new architectures are coming: tegra3 and big.LITTLE. > >> > >> These architectures have different cpus with different caracteristics > >> for power saving. High load => powerfull processors, idle => small > >> processors. > >> > >> That implies different cpu latencies. > >> > >> This patchset present a simple way to keep the current behavior as > >> introduced > >> by Deepthi without breaking the drivers and add the possibility to specify > >> a > >> per cpu states. > >> > > Looks good! I will do some tests with this next week. > > Hi Peter, > > shall I consider adding your acked-by ? >
You can. I don't see any problem with the code as such (besides the race). > Did you have time to test the patchset ? > I haven't had time no :( I was on holiday most of august and had some other things to take care of last week :(. Working on it now. Cheers, Peter. _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev
