diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2015-10-15 12:05:21 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2015-10-28 04:21:11 -0400 |
commit | 0998a03a3a4d35290993f86ffed0e311e8dd78f5 (patch) | |
tree | 2efd150a8e4016fef7bee454f7156a1e81046657 | |
parent | 083701b13c79a908fb400dc24aaa80aad07ac462 (diff) |
cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask
->related_cpus is empty at this point of time and copying ->cpus to it
or orring ->related_cpus with ->cpus would result in the same value. But
cpumask_copy makes it rather clear.
Reviewed-by: Saravana Kannan <skannan@codeaurora.org>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r-- | drivers/cpufreq/cpufreq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 8701dc559850..16b9e811ff01 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c | |||
@@ -1214,7 +1214,7 @@ static int cpufreq_online(unsigned int cpu) | |||
1214 | 1214 | ||
1215 | if (new_policy) { | 1215 | if (new_policy) { |
1216 | /* related_cpus should at least include policy->cpus. */ | 1216 | /* related_cpus should at least include policy->cpus. */ |
1217 | cpumask_or(policy->related_cpus, policy->related_cpus, policy->cpus); | 1217 | cpumask_copy(policy->related_cpus, policy->cpus); |
1218 | /* Remember CPUs present at the policy creation time. */ | 1218 | /* Remember CPUs present at the policy creation time. */ |
1219 | cpumask_and(policy->real_cpus, policy->cpus, cpu_present_mask); | 1219 | cpumask_and(policy->real_cpus, policy->cpus, cpu_present_mask); |
1220 | } | 1220 | } |