diff options
author | Tejun Heo <tj@kernel.org> | 2013-02-27 20:04:46 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-02-27 22:10:19 -0500 |
commit | 6deb69facebb2f9a2b15a8e5e33ab00ebc7c44cb (patch) | |
tree | 4f23dc9cea95c526ef23c28e895379a19197e3b8 /drivers/thermal/cpu_cooling.c | |
parent | ab51603672a335d325963ca410d9c527d9f834f7 (diff) |
thermal: convert to idr_alloc()
Convert to the much saner new idr interface.
Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Zhang Rui <rui.zhang@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/thermal/cpu_cooling.c')
-rw-r--r-- | drivers/thermal/cpu_cooling.c | 17 |
1 files changed, 5 insertions, 12 deletions
diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 836828e29a87..c33fa5315d6b 100644 --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c | |||
@@ -73,21 +73,14 @@ static struct cpufreq_cooling_device *notify_device; | |||
73 | */ | 73 | */ |
74 | static int get_idr(struct idr *idr, int *id) | 74 | static int get_idr(struct idr *idr, int *id) |
75 | { | 75 | { |
76 | int err; | 76 | int ret; |
77 | again: | ||
78 | if (unlikely(idr_pre_get(idr, GFP_KERNEL) == 0)) | ||
79 | return -ENOMEM; | ||
80 | 77 | ||
81 | mutex_lock(&cooling_cpufreq_lock); | 78 | mutex_lock(&cooling_cpufreq_lock); |
82 | err = idr_get_new(idr, NULL, id); | 79 | ret = idr_alloc(idr, NULL, 0, 0, GFP_KERNEL); |
83 | mutex_unlock(&cooling_cpufreq_lock); | 80 | mutex_unlock(&cooling_cpufreq_lock); |
84 | 81 | if (unlikely(ret < 0)) | |
85 | if (unlikely(err == -EAGAIN)) | 82 | return ret; |
86 | goto again; | 83 | *id = ret; |
87 | else if (unlikely(err)) | ||
88 | return err; | ||
89 | |||
90 | *id = *id & MAX_IDR_MASK; | ||
91 | return 0; | 84 | return 0; |
92 | } | 85 | } |
93 | 86 | ||