Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-11 Thread Rafael J. Wysocki
On Sunday, December 08, 2013 01:41:08 PM Borislav Petkov wrote: > On Sun, Dec 08, 2013 at 01:34:36AM +0100, Rafael J. Wysocki wrote: > > On Saturday, December 07, 2013 04:24:09 PM Paul Bolle wrote: > > > On Sat, 2013-12-07 at 12:01 +0100, Bjørn Mork wrote: > > > > Sorry to be making noise here

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-11 Thread Rafael J. Wysocki
On Sunday, December 08, 2013 01:41:08 PM Borislav Petkov wrote: On Sun, Dec 08, 2013 at 01:34:36AM +0100, Rafael J. Wysocki wrote: On Saturday, December 07, 2013 04:24:09 PM Paul Bolle wrote: On Sat, 2013-12-07 at 12:01 +0100, Bjørn Mork wrote: Sorry to be making noise here again, but I

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-08 Thread Borislav Petkov
On Sun, Dec 08, 2013 at 01:34:36AM +0100, Rafael J. Wysocki wrote: > On Saturday, December 07, 2013 04:24:09 PM Paul Bolle wrote: > > On Sat, 2013-12-07 at 12:01 +0100, Bjørn Mork wrote: > > > Sorry to be making noise here again, but I was eager to verify that the > > > fixes in v3.13-rc3 were OK

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-08 Thread Borislav Petkov
On Sun, Dec 08, 2013 at 01:34:36AM +0100, Rafael J. Wysocki wrote: On Saturday, December 07, 2013 04:24:09 PM Paul Bolle wrote: On Sat, 2013-12-07 at 12:01 +0100, Bjørn Mork wrote: Sorry to be making noise here again, but I was eager to verify that the fixes in v3.13-rc3 were OK on my

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-07 Thread Rafael J. Wysocki
On Saturday, December 07, 2013 04:24:09 PM Paul Bolle wrote: > On Sat, 2013-12-07 at 12:01 +0100, Bjørn Mork wrote: > > Sorry to be making noise here again, but I was eager to verify that the > > fixes in v3.13-rc3 were OK on my system. Unfortunately it seems this > > patch completely broke

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-07 Thread Paul Bolle
On Sat, 2013-12-07 at 12:01 +0100, Bjørn Mork wrote: > Sorry to be making noise here again, but I was eager to verify that the > fixes in v3.13-rc3 were OK on my system. Unfortunately it seems this > patch completely broke suspend for me. Hibernete ended up with a blank > console and no visible

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-07 Thread Bjørn Mork
Viresh Kumar writes: > On 29 November 2013 01:59, Rafael J. Wysocki wrote: >> So I'm reading this as "the version of the patch in bleeding-edge is OK". >> >> I hope that really is the case. :-) > > Yeah, that's correct. Sorry to be making noise here again, but I was eager to verify that the

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-07 Thread Bjørn Mork
Viresh Kumar viresh.ku...@linaro.org writes: On 29 November 2013 01:59, Rafael J. Wysocki r...@rjwysocki.net wrote: So I'm reading this as the version of the patch in bleeding-edge is OK. I hope that really is the case. :-) Yeah, that's correct. Sorry to be making noise here again, but I

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-07 Thread Paul Bolle
On Sat, 2013-12-07 at 12:01 +0100, Bjørn Mork wrote: Sorry to be making noise here again, but I was eager to verify that the fixes in v3.13-rc3 were OK on my system. Unfortunately it seems this patch completely broke suspend for me. Hibernete ended up with a blank console and no visible

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-12-07 Thread Rafael J. Wysocki
On Saturday, December 07, 2013 04:24:09 PM Paul Bolle wrote: On Sat, 2013-12-07 at 12:01 +0100, Bjørn Mork wrote: Sorry to be making noise here again, but I was eager to verify that the fixes in v3.13-rc3 were OK on my system. Unfortunately it seems this patch completely broke suspend for

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-28 Thread Viresh Kumar
On 29 November 2013 01:59, Rafael J. Wysocki wrote: > So I'm reading this as "the version of the patch in bleeding-edge is OK". > > I hope that really is the case. :-) Yeah, that's correct. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-28 Thread Rafael J. Wysocki
On Thursday, November 28, 2013 07:49:29 PM Viresh Kumar wrote: > On 28 November 2013 19:53, Rafael J. Wysocki wrote: > > I don't think that the Nishanth's issue is fixed by this particular version > > of > > the patch, so I modified the changelog and removed a the comment above > >

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-28 Thread Viresh Kumar
On 28 November 2013 19:53, Rafael J. Wysocki wrote: > I don't think that the Nishanth's issue is fixed by this particular version of > the patch, so I modified the changelog and removed a the comment above > cpufreq_suspend() No, it does fix his issue as we are just stopping the governors from

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-28 Thread Rafael J. Wysocki
On Wednesday, November 27, 2013 09:09:42 AM Viresh Kumar wrote: > This patch adds cpufreq callbacks to dpm_{suspend|resume}_noirq() for handling > suspend/resume of cpufreq governors. > > There are multiple problems that are fixed by this patch: > - Nishanth Menon (TI) found an interesting

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-28 Thread Rafael J. Wysocki
On Wednesday, November 27, 2013 09:09:42 AM Viresh Kumar wrote: This patch adds cpufreq callbacks to dpm_{suspend|resume}_noirq() for handling suspend/resume of cpufreq governors. There are multiple problems that are fixed by this patch: - Nishanth Menon (TI) found an interesting problem on

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-28 Thread Viresh Kumar
On 28 November 2013 19:53, Rafael J. Wysocki r...@rjwysocki.net wrote: I don't think that the Nishanth's issue is fixed by this particular version of the patch, so I modified the changelog and removed a the comment above cpufreq_suspend() No, it does fix his issue as we are just stopping the

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-28 Thread Rafael J. Wysocki
On Thursday, November 28, 2013 07:49:29 PM Viresh Kumar wrote: On 28 November 2013 19:53, Rafael J. Wysocki r...@rjwysocki.net wrote: I don't think that the Nishanth's issue is fixed by this particular version of the patch, so I modified the changelog and removed a the comment above

Re: [PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-28 Thread Viresh Kumar
On 29 November 2013 01:59, Rafael J. Wysocki r...@rjwysocki.net wrote: So I'm reading this as the version of the patch in bleeding-edge is OK. I hope that really is the case. :-) Yeah, that's correct. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a

[PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-26 Thread Viresh Kumar
This patch adds cpufreq callbacks to dpm_{suspend|resume}_noirq() for handling suspend/resume of cpufreq governors. There are multiple problems that are fixed by this patch: - Nishanth Menon (TI) found an interesting problem on his platform, OMAP. His board wasn't working well with

[PATCH V4] cpufreq: suspend governors on system suspend/hibernate

2013-11-26 Thread Viresh Kumar
This patch adds cpufreq callbacks to dpm_{suspend|resume}_noirq() for handling suspend/resume of cpufreq governors. There are multiple problems that are fixed by this patch: - Nishanth Menon (TI) found an interesting problem on his platform, OMAP. His board wasn't working well with