Re: [PATCH 1/2] cpufreq: governors: Get rid of dbs_data->enable field

2013-01-31 Thread Viresh Kumar
On 1 February 2013 09:22, Viresh Kumar wrote: > On 1 February 2013 00:14, Fabio Baltieri wrote: >> As a sidenote, I noticed just now that since: >> >> bc92bea cpufreq: Notify governors when cpus are hot-[un]plugged >> >> governor's sampling_rate gets reset to default every time you hotplug a >>

Re: [PATCH 1/2] cpufreq: governors: Get rid of dbs_data->enable field

2013-01-31 Thread Viresh Kumar
On 1 February 2013 00:14, Fabio Baltieri wrote: > Hello Viresh, thanks for getting this done... looks much cleaner now! > > I tested both patches on my ux500 setup (dual Cortex-A9) and it seems to > run correctly on both CPU load changes and CPU hotplug, so: > > Tested-by: Fabio Baltieri Thanks.

Re: [PATCH 1/2] cpufreq: governors: Get rid of dbs_data->enable field

2013-01-31 Thread Fabio Baltieri
On Thu, Jan 31, 2013 at 10:58:01PM +0530, Viresh Kumar wrote: > CPUFREQ_GOV_START/STOP are called only once for all policy->cpus and hence we > don't need to adapt cpufreq_governor_dbs() routine for multiple calls. > > So, this patch removes dbs_data->enable field entirely. And rearrange code a >

Re: [PATCH 1/2] cpufreq: governors: Get rid of dbs_data->enable field

2013-01-31 Thread Viresh Kumar
On 31 January 2013 22:58, Viresh Kumar wrote: > CPUFREQ_GOV_START/STOP are called only once for all policy->cpus and hence we > don't need to adapt cpufreq_governor_dbs() routine for multiple calls. > > So, this patch removes dbs_data->enable field entirely. And rearrange code a > bit. > > Signed-