diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-02-13 07:44:57 -0500 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2017-02-15 18:57:42 -0500 |
commit | f451014692ae34e587b00de6745e16661cf734d8 (patch) | |
tree | 6f3e747e1a4c2de894d5bf7aa6e10a0a28170fed | |
parent | 051bd84bb45bc994b7d762467f60b3bd0c7aa6bc (diff) |
cpufreq: Do not clear real_cpus mask on policy init
If new_policy is set in cpufreq_online(), the policy object has just
been created and its real_cpus mask has been zeroed on allocation,
and the driver's ->init() callback should not touch it.
It doesn't need to be cleared again, so don't do that.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
-rw-r--r-- | drivers/cpufreq/cpufreq.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 408479540566..80a785ad17e8 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c | |||
@@ -1166,8 +1166,6 @@ static int cpufreq_online(unsigned int cpu) | |||
1166 | if (new_policy) { | 1166 | if (new_policy) { |
1167 | /* related_cpus should at least include policy->cpus. */ | 1167 | /* related_cpus should at least include policy->cpus. */ |
1168 | cpumask_copy(policy->related_cpus, policy->cpus); | 1168 | cpumask_copy(policy->related_cpus, policy->cpus); |
1169 | /* Clear mask of registered CPUs */ | ||
1170 | cpumask_clear(policy->real_cpus); | ||
1171 | } | 1169 | } |
1172 | 1170 | ||
1173 | /* | 1171 | /* |