Re: [PATCH 0/4][V2] cpuidle : multiple drivers support

Peter De Schrijver
2012-10-26T08:23:58Z

On Thu, Oct 25, 2012 at 10:29:43PM +0200, Rafael J. Wysocki wrote:
> On Thursday, October 25, 2012 04:49:33 PM Peter De Schrijver wrote:
> > On Fri, Oct 19, 2012 at 12:10:45PM +0200, Daniel Lezcano wrote:
> > > The discussion about having different cpus on the system with
> > > different latencies bring us to a first attemp by adding a
> > > pointer in the cpuidle_device to the states array.
> > >
> > > But as Rafael suggested, it would make more sense to create a
> > > driver per cpu [1].
> > >
> > > This patch adds support for multiple cpuidle drivers.
> > >
> > > It creates a per cpu cpuidle driver pointer.
> > >
> > > In order to not break the different drivers, the function cpuidle_register_driver
> > > assign for each cpu, the driver.
> > >
> > > The multiple driver support is optional and if it is not set, the cpuide driver
> > > core code remains the same (except some code reorganisation).
> > >
> > > I did the following tests compiled, booted, tested without/with CONFIG_CPU_IDLE,
> > > with/without CONFIG_CPU_IDLE_MULTIPLE_DRIVERS.
> > >
> > > Tested on Core2 Duo T9500 with acpi_idle [and intel_idle]
> > > Tested on ARM Dual Cortex-A9 U8500 (aka Snowball)
> > >
> > > V1 tested on Tegra3 and Vexpress TC2
> > >
> >
> > V2 tested on Tegra3.
>
> Do I assume correctly that Tested-by applies?
>

Yes.

Cheers,

Peter.