On Mon, Sep 03, 2012 at 01:05:11PM +0200, Peter De Schrijver wrote:
> 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.
> 

I did some preliminary testing, besides the issues I posted earlier. It looks
all good here.

Cheers,

Peter.

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to