cpufreq: Don't pass CPU to cpufreq_add_dev_{symlink|interface}()
Pointer to struct cpufreq_policy is already passed to these routines and we don't need to send policy->cpu to them as well. So, get rid of this extra argument and use policy->cpu everywhere. Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
parent
e8fdde1011
commit
308b60e715
|
@ -806,8 +806,7 @@ void cpufreq_sysfs_remove_file(const struct attribute *attr)
|
|||
EXPORT_SYMBOL(cpufreq_sysfs_remove_file);
|
||||
|
||||
/* symlink affected CPUs */
|
||||
static int cpufreq_add_dev_symlink(unsigned int cpu,
|
||||
struct cpufreq_policy *policy)
|
||||
static int cpufreq_add_dev_symlink(struct cpufreq_policy *policy)
|
||||
{
|
||||
unsigned int j;
|
||||
int ret = 0;
|
||||
|
@ -815,11 +814,11 @@ static int cpufreq_add_dev_symlink(unsigned int cpu,
|
|||
for_each_cpu(j, policy->cpus) {
|
||||
struct device *cpu_dev;
|
||||
|
||||
if (j == cpu)
|
||||
if (j == policy->cpu)
|
||||
continue;
|
||||
|
||||
pr_debug("Adding link for CPU: %u\n", j);
|
||||
cpufreq_cpu_get(cpu);
|
||||
cpufreq_cpu_get(policy->cpu);
|
||||
cpu_dev = get_cpu_device(j);
|
||||
ret = sysfs_create_link(&cpu_dev->kobj, &policy->kobj,
|
||||
"cpufreq");
|
||||
|
@ -831,8 +830,7 @@ static int cpufreq_add_dev_symlink(unsigned int cpu,
|
|||
return ret;
|
||||
}
|
||||
|
||||
static int cpufreq_add_dev_interface(unsigned int cpu,
|
||||
struct cpufreq_policy *policy,
|
||||
static int cpufreq_add_dev_interface(struct cpufreq_policy *policy,
|
||||
struct device *dev)
|
||||
{
|
||||
struct freq_attr **drv_attr;
|
||||
|
@ -868,7 +866,7 @@ static int cpufreq_add_dev_interface(unsigned int cpu,
|
|||
goto err_out_kobj_put;
|
||||
}
|
||||
|
||||
ret = cpufreq_add_dev_symlink(cpu, policy);
|
||||
ret = cpufreq_add_dev_symlink(policy);
|
||||
if (ret)
|
||||
goto err_out_kobj_put;
|
||||
|
||||
|
@ -1097,7 +1095,7 @@ static int __cpufreq_add_dev(struct device *dev, struct subsys_interface *sif,
|
|||
write_unlock_irqrestore(&cpufreq_driver_lock, flags);
|
||||
|
||||
if (!frozen) {
|
||||
ret = cpufreq_add_dev_interface(cpu, policy, dev);
|
||||
ret = cpufreq_add_dev_interface(policy, dev);
|
||||
if (ret)
|
||||
goto err_out_unregister;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue