Re: [PATCH 1/5] cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask

2015-10-13 Thread Saravana Kannan
On 10/12/2015 08:23 PM, Viresh Kumar wrote: On 12-10-15, 12:12, Saravana Kannan wrote: if (new_policy) { /* related_cpus should at least include policy->cpus. */ - cpumask_or(policy->related_cpus, policy->related_cpus, policy->cpus); +

Re: [PATCH 1/5] cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask

2015-10-13 Thread Saravana Kannan
On 10/12/2015 08:23 PM, Viresh Kumar wrote: On 12-10-15, 12:12, Saravana Kannan wrote: if (new_policy) { /* related_cpus should at least include policy->cpus. */ - cpumask_or(policy->related_cpus, policy->related_cpus, policy->cpus); +

Re: [PATCH 1/5] cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask

2015-10-12 Thread Viresh Kumar
On 12-10-15, 12:12, Saravana Kannan wrote: > > if (new_policy) { > > /* related_cpus should at least include policy->cpus. */ > >-cpumask_or(policy->related_cpus, policy->related_cpus, > >policy->cpus); > >+cpumask_copy(policy->related_cpus, policy->cpus);

Re: [PATCH 1/5] cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask

2015-10-12 Thread Saravana Kannan
On 10/11/2015 10:21 AM, Viresh Kumar wrote: Signed-off-by: Viresh Kumar The commit text should explain the why you are doing this. --- drivers/cpufreq/cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index

Re: [PATCH 1/5] cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask

2015-10-12 Thread Saravana Kannan
On 10/11/2015 10:21 AM, Viresh Kumar wrote: Signed-off-by: Viresh Kumar The commit text should explain the why you are doing this. --- drivers/cpufreq/cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/cpufreq.c

Re: [PATCH 1/5] cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask

2015-10-12 Thread Viresh Kumar
On 12-10-15, 12:12, Saravana Kannan wrote: > > if (new_policy) { > > /* related_cpus should at least include policy->cpus. */ > >-cpumask_or(policy->related_cpus, policy->related_cpus, > >policy->cpus); > >+cpumask_copy(policy->related_cpus, policy->cpus);

[PATCH 1/5] cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask

2015-10-11 Thread Viresh Kumar
Signed-off-by: Viresh Kumar --- drivers/cpufreq/cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 25c4c15103a0..b32521432db4 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1221,7

[PATCH 1/5] cpufreq: Use cpumask_copy instead of cpumask_or to copy a mask

2015-10-11 Thread Viresh Kumar
Signed-off-by: Viresh Kumar --- drivers/cpufreq/cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 25c4c15103a0..b32521432db4 100644 --- a/drivers/cpufreq/cpufreq.c +++