On 01/27/2015 12:36 AM, Viresh Kumar wrote:
@Saravana: Few patches might look very similar to what you have done, please add your SOB for those :)
@Viresh, thanks for taking up the patches and working on it. I wish I could have spent more time to get this in myself.
For the patches that are very similar to the ones I sent out, could you please leave the author as me? I spent quite some time figuring out the different corner cases and coding it up and don't want to completely lose authorship for the work I did. Would appreciate it if you could accommodate that.
I'll definitely review this. One thing that's not done in this patchset which I think should eventually be done (so, we should make sure we don't have any code that makes that harder to do in the future) is to allow setting the cpufreq parameters even when all the CPUs in the policy are offline. It's actually trivial for most of the parameters except a few governor specific tunables.
Thanks, Saravana