* 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@sisk.pl, linux-pm@vger.kernel.org, lorenzo.pieralisi@arm.com, patches@linaro.org, linaro-dev@lists.linaro.org 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
[parent not found: <20121025134933.GA1962-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>]
* 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@public.gmane.org, linaro-dev-cunTk1MwBs8s++Sfvej+rw@public.gmane.org, patches-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org 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@vger.kernel.org, lorenzo.pieralisi@arm.com, patches@linaro.org, linaro-dev@lists.linaro.org 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
[parent not found: <4190590.cE8oL2xMlM-sKB8Sp2ER+y1GS7QM15AGw@public.gmane.org>]
* 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@public.gmane.org, patches-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org 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@lists.linaro.org, patches@linaro.org, linux-pm@vger.kernel.org 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@lists.linaro.org, patches@linaro.org, linux-pm@vger.kernel.org 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 a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox