Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-24 Thread Stephen Warren
On 02/23/2014 11:43 PM, Viresh Kumar wrote: > On 20 February 2014 23:10, Stephen Warren wrote: >> Well, except that still leaves a bunch of errors in the kernel log, and >> I have to remember to ignore them:-/ > > Just for few releases, before this patchset goes in. > >> It'd be nice if the

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-24 Thread Stephen Warren
On 02/23/2014 11:43 PM, Viresh Kumar wrote: On 20 February 2014 23:10, Stephen Warren swar...@wwwdotorg.org wrote: Well, except that still leaves a bunch of errors in the kernel log, and I have to remember to ignore them:-/ Just for few releases, before this patchset goes in. It'd be nice

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-23 Thread Viresh Kumar
On 20 February 2014 23:10, Stephen Warren wrote: > Well, except that still leaves a bunch of errors in the kernel log, and > I have to remember to ignore them:-/ Just for few releases, before this patchset goes in. > It'd be nice if the cpufreq core didn't keep changing its behaviour and >

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-23 Thread Viresh Kumar
On 20 February 2014 23:10, Stephen Warren swar...@wwwdotorg.org wrote: Well, except that still leaves a bunch of errors in the kernel log, and I have to remember to ignore them:-/ Just for few releases, before this patchset goes in. It'd be nice if the cpufreq core didn't keep changing its

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-20 Thread Stephen Warren
On 02/19/2014 06:50 PM, Linaro wrote: > > >> On 20-Feb-2014, at 7:19 am, Linaro wrote: >> >> >> On 19-Feb-2014, at 10:56 pm, Stephen Warren wrote: > On 02/18/2014 09:15 PM, Viresh Kumar wrote: >> On 19-Feb-2014 1:48 AM, "Stephen Warren" wrote: >> >>> On 02/17/2014

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-20 Thread Stephen Warren
On 02/19/2014 06:50 PM, Linaro wrote: On 20-Feb-2014, at 7:19 am, Linaro viresh.ku...@linaro.org wrote: On 19-Feb-2014, at 10:56 pm, Stephen Warren swar...@wwwdotorg.org wrote: On 02/18/2014 09:15 PM, Viresh Kumar wrote: On 19-Feb-2014 1:48 AM, Stephen Warren swar...@wwwdotorg.org

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-19 Thread Linaro
> On 20-Feb-2014, at 7:19 am, Linaro wrote: > > > >>> On 19-Feb-2014, at 10:56 pm, Stephen Warren wrote: >>> On 02/18/2014 09:15 PM, Viresh Kumar wrote: > On 19-Feb-2014 1:48 AM, "Stephen Warren" wrote: > >> On 02/17/2014 02:20 AM, Viresh Kumar wrote: >> On 15

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-19 Thread Linaro
> On 19-Feb-2014, at 10:56 pm, Stephen Warren wrote: > >> On 02/18/2014 09:15 PM, Viresh Kumar wrote: >>> On 19-Feb-2014 1:48 AM, "Stephen Warren" wrote: >>> On 02/17/2014 02:20 AM, Viresh Kumar wrote: > On 15 February 2014 05:33, Stephen Warren wrote: > On 02/14/2014 03:23 PM,

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-19 Thread Stephen Warren
On 02/18/2014 09:15 PM, Viresh Kumar wrote: > On 19-Feb-2014 1:48 AM, "Stephen Warren" wrote: >> >> On 02/17/2014 02:20 AM, Viresh Kumar wrote: >>> On 15 February 2014 05:33, Stephen Warren wrote: On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: >>> > Well, it would be good to verify

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-19 Thread Stephen Warren
On 02/18/2014 09:15 PM, Viresh Kumar wrote: On 19-Feb-2014 1:48 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 02/17/2014 02:20 AM, Viresh Kumar wrote: On 15 February 2014 05:33, Stephen Warren swar...@wwwdotorg.org wrote: On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: Well, it would

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-19 Thread Linaro
On 19-Feb-2014, at 10:56 pm, Stephen Warren swar...@wwwdotorg.org wrote: On 02/18/2014 09:15 PM, Viresh Kumar wrote: On 19-Feb-2014 1:48 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 02/17/2014 02:20 AM, Viresh Kumar wrote: On 15 February 2014 05:33, Stephen Warren

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-19 Thread Linaro
On 20-Feb-2014, at 7:19 am, Linaro viresh.ku...@linaro.org wrote: On 19-Feb-2014, at 10:56 pm, Stephen Warren swar...@wwwdotorg.org wrote: On 02/18/2014 09:15 PM, Viresh Kumar wrote: On 19-Feb-2014 1:48 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 02/17/2014 02:20 AM, Viresh

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-18 Thread Viresh Kumar
On 19-Feb-2014 1:48 AM, "Stephen Warren" wrote: > > On 02/17/2014 02:20 AM, Viresh Kumar wrote: > > On 15 February 2014 05:33, Stephen Warren wrote: > >> On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: > > > >>> Well, it would be good to verify which part, then. > >> > >> Patch 2/7 appears to

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-18 Thread Stephen Warren
On 02/16/2014 11:02 PM, Viresh Kumar wrote: > On 15 February 2014 01:52, Stephen Warren wrote: >> BTW, I also happened to test these on a Tegra114 system, on which there >> is no cpufreq driver, and this series (applied on top of commit >> 9398a10cd964 Merge tag 'regulator-v3.14-rc2') causes the

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-18 Thread Stephen Warren
On 02/17/2014 02:20 AM, Viresh Kumar wrote: > On 15 February 2014 05:33, Stephen Warren wrote: >> On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: > >>> Well, it would be good to verify which part, then. >> >> Patch 2/7 appears to stop that message from being printed during >> suspend, and

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-18 Thread Stephen Warren
On 02/17/2014 02:20 AM, Viresh Kumar wrote: On 15 February 2014 05:33, Stephen Warren swar...@wwwdotorg.org wrote: On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: Well, it would be good to verify which part, then. Patch 2/7 appears to stop that message from being printed during suspend,

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-18 Thread Stephen Warren
On 02/16/2014 11:02 PM, Viresh Kumar wrote: On 15 February 2014 01:52, Stephen Warren swar...@wwwdotorg.org wrote: BTW, I also happened to test these on a Tegra114 system, on which there is no cpufreq driver, and this series (applied on top of commit 9398a10cd964 Merge tag

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-18 Thread Viresh Kumar
On 19-Feb-2014 1:48 AM, Stephen Warren swar...@wwwdotorg.org wrote: On 02/17/2014 02:20 AM, Viresh Kumar wrote: On 15 February 2014 05:33, Stephen Warren swar...@wwwdotorg.org wrote: On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: Well, it would be good to verify which part, then.

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-17 Thread Viresh Kumar
On 15 February 2014 05:33, Stephen Warren wrote: > On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: >> Well, it would be good to verify which part, then. > > Patch 2/7 appears to stop that message from being printed during > suspend, and perhaps reduce the number of times it's printed during >

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-17 Thread Viresh Kumar
On 15 February 2014 05:33, Stephen Warren swar...@wwwdotorg.org wrote: On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: Well, it would be good to verify which part, then. Patch 2/7 appears to stop that message from being printed during suspend, and perhaps reduce the number of times it's

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-16 Thread Viresh Kumar
On 15 February 2014 01:12, Stephen Warren wrote: > On 02/12/2014 11:50 PM, Viresh Kumar wrote: >> This patchset creates/calls cpufreq suspend/resume callbacks from >> dpm_{suspend|resume}() >> for handling suspend/resume of cpufreq governors and core. > > Are these patches for 3.14 or 3.15?

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-16 Thread Viresh Kumar
On 15 February 2014 01:52, Stephen Warren wrote: > BTW, I also happened to test these on a Tegra114 system, on which there > is no cpufreq driver, and this series (applied on top of commit > 9398a10cd964 Merge tag 'regulator-v3.14-rc2') causes the following > during suspend: Ahh, that's a

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-16 Thread Viresh Kumar
On 15 February 2014 01:52, Stephen Warren swar...@wwwdotorg.org wrote: BTW, I also happened to test these on a Tegra114 system, on which there is no cpufreq driver, and this series (applied on top of commit 9398a10cd964 Merge tag 'regulator-v3.14-rc2') causes the following during suspend:

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-16 Thread Viresh Kumar
On 15 February 2014 01:12, Stephen Warren swar...@wwwdotorg.org wrote: On 02/12/2014 11:50 PM, Viresh Kumar wrote: This patchset creates/calls cpufreq suspend/resume callbacks from dpm_{suspend|resume}() for handling suspend/resume of cpufreq governors and core. Are these patches for 3.14

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-14 Thread Stephen Warren
On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: > On Friday, February 14, 2014 12:42:53 PM Stephen Warren wrote: >> On 02/12/2014 11:50 PM, Viresh Kumar wrote: >>> This patchset creates/calls cpufreq suspend/resume callbacks from >>> dpm_{suspend|resume}() >>> for handling suspend/resume of

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-14 Thread Rafael J. Wysocki
On Friday, February 14, 2014 12:42:53 PM Stephen Warren wrote: > On 02/12/2014 11:50 PM, Viresh Kumar wrote: > > This patchset creates/calls cpufreq suspend/resume callbacks from > > dpm_{suspend|resume}() > > for handling suspend/resume of cpufreq governors and core. > > Are these patches for

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-14 Thread Stephen Warren
On 02/14/2014 12:42 PM, Stephen Warren wrote: > On 02/12/2014 11:50 PM, Viresh Kumar wrote: >> This patchset creates/calls cpufreq suspend/resume callbacks from >> dpm_{suspend|resume}() >> for handling suspend/resume of cpufreq governors and core. BTW, I also happened to test these on a

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-14 Thread Stephen Warren
On 02/12/2014 11:50 PM, Viresh Kumar wrote: > This patchset creates/calls cpufreq suspend/resume callbacks from > dpm_{suspend|resume}() > for handling suspend/resume of cpufreq governors and core. Are these patches for 3.14 or 3.15? I ask because I just tested Linus's master from a few days

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-14 Thread Stephen Warren
On 02/12/2014 11:50 PM, Viresh Kumar wrote: This patchset creates/calls cpufreq suspend/resume callbacks from dpm_{suspend|resume}() for handling suspend/resume of cpufreq governors and core. Are these patches for 3.14 or 3.15? I ask because I just tested Linus's master from a few days back

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-14 Thread Stephen Warren
On 02/14/2014 12:42 PM, Stephen Warren wrote: On 02/12/2014 11:50 PM, Viresh Kumar wrote: This patchset creates/calls cpufreq suspend/resume callbacks from dpm_{suspend|resume}() for handling suspend/resume of cpufreq governors and core. BTW, I also happened to test these on a Tegra114

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-14 Thread Rafael J. Wysocki
On Friday, February 14, 2014 12:42:53 PM Stephen Warren wrote: On 02/12/2014 11:50 PM, Viresh Kumar wrote: This patchset creates/calls cpufreq suspend/resume callbacks from dpm_{suspend|resume}() for handling suspend/resume of cpufreq governors and core. Are these patches for 3.14 or

Re: [PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-14 Thread Stephen Warren
On 02/14/2014 03:23 PM, Rafael J. Wysocki wrote: On Friday, February 14, 2014 12:42:53 PM Stephen Warren wrote: On 02/12/2014 11:50 PM, Viresh Kumar wrote: This patchset creates/calls cpufreq suspend/resume callbacks from dpm_{suspend|resume}() for handling suspend/resume of cpufreq

[PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-12 Thread Viresh Kumar
This patchset creates/calls cpufreq suspend/resume callbacks from dpm_{suspend|resume}() for handling suspend/resume of cpufreq governors and core. There are multiple problems that are fixed by this patch: - Nishanth Menon (TI) found an interesting problem on his platform, OMAP. His board

[PATCH V5 0/7] cpufreq: suspend early/resume late: dpm_{suspend|resume}()

2014-02-12 Thread Viresh Kumar
This patchset creates/calls cpufreq suspend/resume callbacks from dpm_{suspend|resume}() for handling suspend/resume of cpufreq governors and core. There are multiple problems that are fixed by this patch: - Nishanth Menon (TI) found an interesting problem on his platform, OMAP. His board