Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-18 Thread Rafael J. Wysocki
On Tuesday, June 18, 2013 10:20:33 AM Viresh Kumar wrote: > On 5 June 2013 18:38, Viresh Kumar wrote: > > To be honest with the amount of experience I have now, my log was > > poor :( > > > > I have used following log in the attached patch: > > > > Subjec

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-18 Thread Rafael J. Wysocki
On Tuesday, June 18, 2013 10:20:33 AM Viresh Kumar wrote: On 5 June 2013 18:38, Viresh Kumar viresh.ku...@linaro.org wrote: To be honest with the amount of experience I have now, my log was poor :( I have used following log in the attached patch: Subject: [PATCH] cpufreq: userspace

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-17 Thread Viresh Kumar
On 5 June 2013 18:38, Viresh Kumar wrote: > To be honest with the amount of experience I have now, my log was > poor :( > > I have used following log in the attached patch: > > Subject: [PATCH] cpufreq: userspace: Simplify governor > > Userspace governor has got more

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-17 Thread Viresh Kumar
On 5 June 2013 18:38, Viresh Kumar viresh.ku...@linaro.org wrote: To be honest with the amount of experience I have now, my log was poor :( I have used following log in the attached patch: Subject: [PATCH] cpufreq: userspace: Simplify governor Userspace governor has got more code than what

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-08 Thread Xiaoguang Chen
. >>> >>> Also, please apply it as a fix for 3.10 as it is broken recently in 3.9. >> >> I'd love to, but I need answers to the above questions before I do that. > > To be honest with the amount of experience I have now, my log was > poor :( > > I have

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-08 Thread Xiaoguang Chen
have now, my log was poor :( I have used following log in the attached patch: Subject: [PATCH] cpufreq: userspace: Simplify governor Userspace governor has got more code than what it needs for its functioning. Lets simplify it. Portions of code removed are: - Extra header files which aren't

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-05 Thread Viresh Kumar
gt; doing >> something stupid. >> >> Also, please apply it as a fix for 3.10 as it is broken recently in 3.9. > > I'd love to, but I need answers to the above questions before I do that. To be honest with the amount of experience I have now, my log was poor :( I have used

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-05 Thread Rafael J. Wysocki
On Wednesday, June 05, 2013 02:34:33 PM Viresh Kumar wrote: > Userspace governor has got more code than what it needs for its functioning. > Lets simplify it. OK Why exactly is the code you're removing unnecessary? > Over that it will fix issues in cpufreq_governor_userspace(), which isn't >

[PATCH] cpufreq: userspace: Simplify governor

2013-06-05 Thread Viresh Kumar
Userspace governor has got more code than what it needs for its functioning. Lets simplify it. Over that it will fix issues in cpufreq_governor_userspace(), which isn't doing right things in START/STOP. It is working per-cpu currently whereas it just required to manage policy->cpu. Reported-by:

[PATCH] cpufreq: userspace: Simplify governor

2013-06-05 Thread Viresh Kumar
Userspace governor has got more code than what it needs for its functioning. Lets simplify it. Over that it will fix issues in cpufreq_governor_userspace(), which isn't doing right things in START/STOP. It is working per-cpu currently whereas it just required to manage policy-cpu. Reported-by:

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-05 Thread Rafael J. Wysocki
On Wednesday, June 05, 2013 02:34:33 PM Viresh Kumar wrote: Userspace governor has got more code than what it needs for its functioning. Lets simplify it. OK Why exactly is the code you're removing unnecessary? Over that it will fix issues in cpufreq_governor_userspace(), which isn't doing

Re: [PATCH] cpufreq: userspace: Simplify governor

2013-06-05 Thread Viresh Kumar
in 3.9. I'd love to, but I need answers to the above questions before I do that. To be honest with the amount of experience I have now, my log was poor :( I have used following log in the attached patch: Subject: [PATCH] cpufreq: userspace: Simplify governor Userspace governor has got more code