Re: [PATCH] cpufreq: userspace: Simplify governor

On Tuesday, June 18, 2013 10:20:33 AM Viresh Kumar wrote:

On 5 June 2013 18:38, Viresh Kumar viresh.kumar@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 it needs for its functioning. Lets simplify it. Portions of code removed are:

• Extra header files which aren't required anymore (rearranged them as well)
• cpu_{max|min|cur|set}_freq, as they are always same as policy->{max|min|cur}
• userspace_cpufreq_notifier_block as we don't need to set cpu_cur_freq anymore
• cpus_using_userspace_governor as it was for notifier code

Signed-off-by: Viresh Kumar viresh.kumar@linaro.org

Hi Rafael,

Are you happy with the log & description of this patch now?

Yes, I am. I'm hoping to get to the pending patches later today.

Thanks, Rafael