summaryrefslogtreecommitdiffstats
path: root/kernel
diff options
context:
space:
mode:
authorThomas Gleixner <tglx@linutronix.de>2011-05-04 08:18:34 +0200
committerThomas Gleixner <tglx@linutronix.de>2011-05-04 08:18:34 +0200
commit179eb03268aa1da03d90f1566ea85dc1478d3ae3 (patch)
treefda7d48258075bf84f12ce143ab7cd260bbc92e8 /kernel
parentce788f930b0cdf821de7ee8f84cfe8cf7fcb6311 (diff)
downloadop-kernel-dev-179eb03268aa1da03d90f1566ea85dc1478d3ae3.zip
op-kernel-dev-179eb03268aa1da03d90f1566ea85dc1478d3ae3.tar.gz
alarmtimer: Drop device refcount after rtc_open()
class_find_device() takes a refcount on the rtc device. rtc_open() takes another one, so we can drop it after the rtc_open() call. Signed-off-by: Thomas Gleixner <tglx@linutronix.de> Cc: John Stultz <john.stultz@linaro.org>
Diffstat (limited to 'kernel')
-rw-r--r--kernel/time/alarmtimer.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/kernel/time/alarmtimer.c b/kernel/time/alarmtimer.c
index e5db9b0..c6027fe 100644
--- a/kernel/time/alarmtimer.c
+++ b/kernel/time/alarmtimer.c
@@ -675,8 +675,14 @@ static int __init alarmtimer_init_late(void)
/* Find an rtc device and init the rtc_timer */
dev = class_find_device(rtc_class, NULL, &str, has_wakealarm);
/* If we have a device then str is valid. See has_wakealarm() */
- if (dev)
+ if (dev) {
rtcdev = rtc_class_open(str);
+ /*
+ * Drop the reference we got in class_find_device,
+ * rtc_open takes its own.
+ */
+ put_device(dev);
+ }
if (!rtcdev) {
printk(KERN_WARNING "No RTC device found, ALARM timers will"
" not wake from suspend");
OpenPOWER on IntegriCloud