| Message ID | 1363085270-22886-1-git-send-email-laurent.navet@gmail.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Delegated to: | Zhang Rui |
| Headers | show |
On Tue, 2013-03-12 at 11:47 +0100, Laurent Navet wrote: > - WARNING: Avoid CamelCase: <maskPtr> > > Signed-off-by: Laurent Navet <laurent.navet@gmail.com> applied to thermal -next. thanks, rui > --- > drivers/thermal/cpu_cooling.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c > index 8dc44cb..be2e6b0 100644 > --- a/drivers/thermal/cpu_cooling.c > +++ b/drivers/thermal/cpu_cooling.c > @@ -168,8 +168,8 @@ static int cpufreq_apply_cooling(struct cpufreq_cooling_device *cpufreq_device, > unsigned long cooling_state) > { > unsigned int cpuid, clip_freq; > - struct cpumask *maskPtr = &cpufreq_device->allowed_cpus; > - unsigned int cpu = cpumask_any(maskPtr); > + struct cpumask *mask = &cpufreq_device->allowed_cpus; > + unsigned int cpu = cpumask_any(mask); > > > /* Check if the old cooling action is same as new cooling action */ > @@ -184,7 +184,7 @@ static int cpufreq_apply_cooling(struct cpufreq_cooling_device *cpufreq_device, > cpufreq_device->cpufreq_val = clip_freq; > notify_device = cpufreq_device; > > - for_each_cpu(cpuid, maskPtr) { > + for_each_cpu(cpuid, mask) { > if (is_cpufreq_valid(cpuid)) > cpufreq_update_policy(cpuid); > } > @@ -235,13 +235,13 @@ static int cpufreq_get_max_state(struct thermal_cooling_device *cdev, > unsigned long *state) > { > struct cpufreq_cooling_device *cpufreq_device = cdev->devdata; > - struct cpumask *maskPtr = &cpufreq_device->allowed_cpus; > + struct cpumask *mask = &cpufreq_device->allowed_cpus; > unsigned int cpu; > struct cpufreq_frequency_table *table; > unsigned long count = 0; > int i = 0; > > - cpu = cpumask_any(maskPtr); > + cpu = cpumask_any(mask); > table = cpufreq_frequency_get_table(cpu); > if (!table) { > *state = 0; -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 8dc44cb..be2e6b0 100644 --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c @@ -168,8 +168,8 @@ static int cpufreq_apply_cooling(struct cpufreq_cooling_device *cpufreq_device, unsigned long cooling_state) { unsigned int cpuid, clip_freq; - struct cpumask *maskPtr = &cpufreq_device->allowed_cpus; - unsigned int cpu = cpumask_any(maskPtr); + struct cpumask *mask = &cpufreq_device->allowed_cpus; + unsigned int cpu = cpumask_any(mask); /* Check if the old cooling action is same as new cooling action */ @@ -184,7 +184,7 @@ static int cpufreq_apply_cooling(struct cpufreq_cooling_device *cpufreq_device, cpufreq_device->cpufreq_val = clip_freq; notify_device = cpufreq_device; - for_each_cpu(cpuid, maskPtr) { + for_each_cpu(cpuid, mask) { if (is_cpufreq_valid(cpuid)) cpufreq_update_policy(cpuid); } @@ -235,13 +235,13 @@ static int cpufreq_get_max_state(struct thermal_cooling_device *cdev, unsigned long *state) { struct cpufreq_cooling_device *cpufreq_device = cdev->devdata; - struct cpumask *maskPtr = &cpufreq_device->allowed_cpus; + struct cpumask *mask = &cpufreq_device->allowed_cpus; unsigned int cpu; struct cpufreq_frequency_table *table; unsigned long count = 0; int i = 0; - cpu = cpumask_any(maskPtr); + cpu = cpumask_any(mask); table = cpufreq_frequency_get_table(cpu); if (!table) { *state = 0;
- WARNING: Avoid CamelCase: <maskPtr> Signed-off-by: Laurent Navet <laurent.navet@gmail.com> --- drivers/thermal/cpu_cooling.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)