* Re: [PATCH 0/4][V2] cpuidle : multiple drivers support
[not found] <1350641449-22863-1-git-send-email-daniel.lezcano@linaro.org>
@ 2012-10-25 13:49 ` Peter De Schrijver
[not found] ` <20121025134933.GA1962-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
2012-10-25 20:29 ` Rafael J. Wysocki
0 siblings, 2 replies; 6+ messages in thread
From: Peter De Schrijver @ 2012-10-25 13:49 UTC (permalink / raw)
To: Daniel Lezcano; +Cc: rjw, linux-pm, lorenzo.pieralisi, patches, linaro-dev
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.
Cheers,
Peter.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/4][V2] cpuidle : multiple drivers support
[not found] ` <20121025134933.GA1962-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
@ 2012-10-25 14:11 ` Daniel Lezcano
0 siblings, 0 replies; 6+ messages in thread
From: Daniel Lezcano @ 2012-10-25 14:11 UTC (permalink / raw)
To: Peter De Schrijver
Cc: rjw-KKrjLPT3xs0, linaro-dev-cunTk1MwBs8s++Sfvej+rw,
patches-QSEj5FYQhm4dnm+yROfE0A, linux-pm-u79uwXL29TY76Z2rM5mHXA
On 10/25/2012 03:49 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.
Cool thanks !
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/4][V2] cpuidle : multiple drivers support
2012-10-25 13:49 ` [PATCH 0/4][V2] cpuidle : multiple drivers support Peter De Schrijver
[not found] ` <20121025134933.GA1962-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
@ 2012-10-25 20:29 ` Rafael J. Wysocki
[not found] ` <4190590.cE8oL2xMlM-sKB8Sp2ER+y1GS7QM15AGw@public.gmane.org>
1 sibling, 1 reply; 6+ messages in thread
From: Rafael J. Wysocki @ 2012-10-25 20:29 UTC (permalink / raw)
To: Peter De Schrijver
Cc: Daniel Lezcano, linux-pm, lorenzo.pieralisi, patches, linaro-dev
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?
Rafael
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/4][V2] cpuidle : multiple drivers support
[not found] ` <4190590.cE8oL2xMlM-sKB8Sp2ER+y1GS7QM15AGw@public.gmane.org>
@ 2012-10-26 8:23 ` Peter De Schrijver
2012-10-26 19:23 ` Daniel Lezcano
0 siblings, 1 reply; 6+ messages in thread
From: Peter De Schrijver @ 2012-10-26 8:23 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: linaro-dev-cunTk1MwBs8s++Sfvej+rw,
patches-QSEj5FYQhm4dnm+yROfE0A, linux-pm-u79uwXL29TY76Z2rM5mHXA
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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/4][V2] cpuidle : multiple drivers support
2012-10-26 8:23 ` Peter De Schrijver
@ 2012-10-26 19:23 ` Daniel Lezcano
2012-10-26 21:10 ` Rafael J. Wysocki
0 siblings, 1 reply; 6+ messages in thread
From: Daniel Lezcano @ 2012-10-26 19:23 UTC (permalink / raw)
To: Peter De Schrijver; +Cc: Rafael J. Wysocki, linaro-dev, patches, linux-pm
Rafael,
this patchset does not apply anymore on linux-pm-next.
Let me refresh it and resend a V3.
Thanks
-- Daniel
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/4][V2] cpuidle : multiple drivers support
2012-10-26 19:23 ` Daniel Lezcano
@ 2012-10-26 21:10 ` Rafael J. Wysocki
0 siblings, 0 replies; 6+ messages in thread
From: Rafael J. Wysocki @ 2012-10-26 21:10 UTC (permalink / raw)
To: Daniel Lezcano; +Cc: Peter De Schrijver, linaro-dev, patches, linux-pm
On Friday, October 26, 2012 09:23:45 PM Daniel Lezcano wrote:
>
> Rafael,
>
> this patchset does not apply anymore on linux-pm-next.
>
> Let me refresh it and resend a V3.
OK, thanks!
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-10-26 21:06 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1350641449-22863-1-git-send-email-daniel.lezcano@linaro.org>
2012-10-25 13:49 ` [PATCH 0/4][V2] cpuidle : multiple drivers support Peter De Schrijver
[not found] ` <20121025134933.GA1962-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
2012-10-25 14:11 ` Daniel Lezcano
2012-10-25 20:29 ` Rafael J. Wysocki
[not found] ` <4190590.cE8oL2xMlM-sKB8Sp2ER+y1GS7QM15AGw@public.gmane.org>
2012-10-26 8:23 ` Peter De Schrijver
2012-10-26 19:23 ` Daniel Lezcano
2012-10-26 21:10 ` Rafael J. Wysocki
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.