Re: [PATCH V2 1/4] cpufreq: Add per policy governor-init/exit infrastructure

2013-02-21 Thread Viresh Kumar
On 22 February 2013 05:05, Rafael J. Wysocki wrote: > Why don't you use different values here? > > If you need only one value, one #define should be sufficient. This is the fixup i have for this, I will push all patches again to cpufreq-for-3.10 branch: --

Re: [PATCH V2 1/4] cpufreq: Add per policy governor-init/exit infrastructure

2013-02-21 Thread Viresh Kumar
On 22 February 2013 07:51, Rafael J. Wysocki wrote: > On Friday, February 22, 2013 07:38:12 AM Viresh Kumar wrote: >> No. These are used atleast for ondemand & conservative. > > They will be after the next patch, you mean? :-) > Well, it appeared so from the next patch ... Yes :) __

Re: [PATCH V2 1/4] cpufreq: Add per policy governor-init/exit infrastructure

2013-02-21 Thread Rafael J. Wysocki
On Friday, February 22, 2013 07:38:12 AM Viresh Kumar wrote: > On 22 February 2013 05:05, Rafael J. Wysocki wrote: > > On Monday, February 11, 2013 01:20:00 PM Viresh Kumar wrote: > > >> This patch is inclined towards providing this infrastructure. Because we > >> are > >> required to allocate g

Re: [PATCH V2 1/4] cpufreq: Add per policy governor-init/exit infrastructure

2013-02-21 Thread Viresh Kumar
On 22 February 2013 05:05, Rafael J. Wysocki wrote: > On Monday, February 11, 2013 01:20:00 PM Viresh Kumar wrote: >> This patch is inclined towards providing this infrastructure. Because we are >> required to allocate governor's resources dynamically now, we must do it at >> policy creation and

Re: [PATCH V2 1/4] cpufreq: Add per policy governor-init/exit infrastructure

2013-02-21 Thread Rafael J. Wysocki
On Monday, February 11, 2013 01:20:00 PM Viresh Kumar wrote: > Currently, there can't be multiple instances of single governor_type. If we > have > a multi-package system, where we have multiple instances of struct policy (per > package), we can't have multiple instances of same governor. i.e. We

[PATCH V2 1/4] cpufreq: Add per policy governor-init/exit infrastructure

2013-02-10 Thread Viresh Kumar
Currently, there can't be multiple instances of single governor_type. If we have a multi-package system, where we have multiple instances of struct policy (per package), we can't have multiple instances of same governor. i.e. We can't have multiple instances of ondemand governor for multiple packag