diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2011-05-04 08:18:34 +0200 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2011-05-04 08:18:34 +0200 |
| commit | 179eb03268aa1da03d90f1566ea85dc1478d3ae3 (patch) | |
| tree | fda7d48258075bf84f12ce143ab7cd260bbc92e8 /kernel/time | |
| parent | ce788f930b0cdf821de7ee8f84cfe8cf7fcb6311 (diff) | |
| download | op-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/time')
| -rw-r--r-- | kernel/time/alarmtimer.c | 8 |
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"); |
