Commit e1fae554 authored by Viresh Kumar's avatar Viresh Kumar Committed by Eduardo Valentin

thermal: cpu_cooling: don't iterate over all allowed_cpus to update cpufreq policy

All CPUs present in 'allowed_cpus' share the same 'struct cpufreq_policy'
structure and so calling cpufreq_update_policy() for each of them doesn't make
sense.
Signed-off-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarEduardo Valentin <edubezval@gmail.com>
parent 405fb825
...@@ -285,11 +285,10 @@ static unsigned int get_cpu_frequency(unsigned int cpu, unsigned long level) ...@@ -285,11 +285,10 @@ static unsigned int get_cpu_frequency(unsigned int cpu, unsigned long level)
static int cpufreq_apply_cooling(struct cpufreq_cooling_device *cpufreq_device, static int cpufreq_apply_cooling(struct cpufreq_cooling_device *cpufreq_device,
unsigned long cooling_state) unsigned long cooling_state)
{ {
unsigned int cpuid, clip_freq; unsigned int clip_freq;
struct cpumask *mask = &cpufreq_device->allowed_cpus; struct cpumask *mask = &cpufreq_device->allowed_cpus;
unsigned int cpu = cpumask_any(mask); unsigned int cpu = cpumask_any(mask);
/* Check if the old cooling action is same as new cooling action */ /* Check if the old cooling action is same as new cooling action */
if (cpufreq_device->cpufreq_state == cooling_state) if (cpufreq_device->cpufreq_state == cooling_state)
return 0; return 0;
...@@ -301,10 +300,8 @@ static int cpufreq_apply_cooling(struct cpufreq_cooling_device *cpufreq_device, ...@@ -301,10 +300,8 @@ static int cpufreq_apply_cooling(struct cpufreq_cooling_device *cpufreq_device,
cpufreq_device->cpufreq_state = cooling_state; cpufreq_device->cpufreq_state = cooling_state;
cpufreq_device->cpufreq_val = clip_freq; cpufreq_device->cpufreq_val = clip_freq;
for_each_cpu(cpuid, mask) { if (is_cpufreq_valid(cpu))
if (is_cpufreq_valid(cpuid)) cpufreq_update_policy(cpu);
cpufreq_update_policy(cpuid);
}
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment