On Mon 2014-06-09 10:54:17, Viresh Kumar wrote:
'copy_prev_load' was recently added by commit: 18b46ab (cpufreq: governor: Be friendly towards latency-sensitive bursty workloads).
It actually is a bit redundant as we also have 'prev_load' which can store any integer value and can be used instead of 'copy_prev_load' by setting it to zero when we don't want to use previous load.
So, drop 'copy_prev_load' and use 'prev_load' instead.
Update comments as well to make it more clear.
There is another change here which was probably missed by Srivatsa during the last version of updates he made. The unlikely in the 'if' statement was covering only half of the condition and the whole line should actually come under it.
Also checkpatch is made more silent as it was reporting this (--strict option):
CHECK: Alignment should match open parenthesis
if (unlikely(wall_time > (2 * sampling_rate) &&j_cdbs->prev_load)) {Signed-off-by: Viresh Kumar viresh.kumar@linaro.org
Acked-by: Pavel Machek pavel@ucw.cz