Commit a4b6fec9 authored by Jonghwa Lee's avatar Jonghwa Lee Committed by Zhang Rui

Thermal: Fix bug on cpu_cooling, cooling device's id conflict problem.

This patch fixes small bug on cpu_cooling. CPU cooling device has own
id generated with idr mathod. However in the previous version, it swapped
to all same id at last stage of probing as 0. This makes id's collision and
also occures error when it releases that id.
Signed-off-by: default avatarJonghwa Lee <jonghwa3.lee@samsung.com>
parent 79e093c3
...@@ -351,7 +351,7 @@ struct thermal_cooling_device *cpufreq_cooling_register( ...@@ -351,7 +351,7 @@ struct thermal_cooling_device *cpufreq_cooling_register(
struct cpufreq_cooling_device *cpufreq_dev = NULL; struct cpufreq_cooling_device *cpufreq_dev = NULL;
unsigned int cpufreq_dev_count = 0, min = 0, max = 0; unsigned int cpufreq_dev_count = 0, min = 0, max = 0;
char dev_name[THERMAL_NAME_LENGTH]; char dev_name[THERMAL_NAME_LENGTH];
int ret = 0, id = 0, i; int ret = 0, i;
struct cpufreq_policy policy; struct cpufreq_policy policy;
list_for_each_entry(cpufreq_dev, &cooling_cpufreq_list, node) list_for_each_entry(cpufreq_dev, &cooling_cpufreq_list, node)
...@@ -396,7 +396,6 @@ struct thermal_cooling_device *cpufreq_cooling_register( ...@@ -396,7 +396,6 @@ struct thermal_cooling_device *cpufreq_cooling_register(
kfree(cpufreq_dev); kfree(cpufreq_dev);
return ERR_PTR(-EINVAL); return ERR_PTR(-EINVAL);
} }
cpufreq_dev->id = id;
cpufreq_dev->cool_dev = cool_dev; cpufreq_dev->cool_dev = cool_dev;
cpufreq_dev->cpufreq_state = 0; cpufreq_dev->cpufreq_state = 0;
mutex_lock(&cooling_cpufreq_lock); mutex_lock(&cooling_cpufreq_lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment