diff options
author | Daniel Lezcano <daniel.lezcano@linaro.org> | 2013-06-12 09:08:55 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-07-14 20:09:48 -0400 |
commit | c878a52d3c7cabab5b24460825c24eafd8be7058 (patch) | |
tree | 98c5e7c8599b62deac5559dc08fef9174173a39e | |
parent | 5df0aa7341bd94ca2023a60c64c63faeb6ec209d (diff) |
cpuidle: Check if device is already registered
Make __cpuidle_register_device() check whether or not the device has
been registered already and return -EBUSY immediately if that's the
case.
[rjw: Changelog]
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r-- | drivers/cpuidle/cpuidle.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index 5b63185da59b..d75040ddd2b3 100644 --- a/drivers/cpuidle/cpuidle.c +++ b/drivers/cpuidle/cpuidle.c | |||
@@ -400,13 +400,16 @@ static int __cpuidle_register_device(struct cpuidle_device *dev) | |||
400 | */ | 400 | */ |
401 | int cpuidle_register_device(struct cpuidle_device *dev) | 401 | int cpuidle_register_device(struct cpuidle_device *dev) |
402 | { | 402 | { |
403 | int ret; | 403 | int ret = -EBUSY; |
404 | 404 | ||
405 | if (!dev) | 405 | if (!dev) |
406 | return -EINVAL; | 406 | return -EINVAL; |
407 | 407 | ||
408 | mutex_lock(&cpuidle_lock); | 408 | mutex_lock(&cpuidle_lock); |
409 | 409 | ||
410 | if (dev->registered) | ||
411 | goto out_unlock; | ||
412 | |||
410 | ret = __cpuidle_device_init(dev); | 413 | ret = __cpuidle_device_init(dev); |
411 | if (ret) | 414 | if (ret) |
412 | goto out_unlock; | 415 | goto out_unlock; |