Messages in this thread | | | Date | Wed, 30 Jan 2013 17:53:21 +0100 | From | Fabio Baltieri <> | Subject | Re: [PATCH v7 3/4] cpufreq: conservative: call dbs_check_cpu only when necessary |
| |
On Wed, Jan 30, 2013 at 09:23:22PM +0530, Viresh Kumar wrote: > On 30 January 2013 18:30, Fabio Baltieri <fabio.baltieri@linaro.org> wrote: > > Modify conservative timer to not resample CPU utilization if recently > > sampled from another SW coordinated core. > > > > Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org> > > We are again doing the same mistake which i fixed with: > > commit 4471a34f9a1f2da220272e823bdb8e8fa83a7661 > Author: Viresh Kumar <viresh.kumar@linaro.org> > Date: Fri Oct 26 00:47:42 2012 +0200 > > cpufreq: governors: remove redundant code
Can't argue with this, but the two had some subdle differences (namely th two dbs_info structures) and I opted to not mess up forcing some non-obvious common code.
Feel free to suggest a strategy.
Thanks, Fabio
-- Fabio Baltieri
| |