rtc-parisc: remove redundant locking
authordann frazier <dannf@hp.com>
Tue, 31 Mar 2009 22:24:50 +0000 (15:24 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 1 Apr 2009 15:59:24 +0000 (08:59 -0700)
The RTC subsystem proides ops locking, no need to implement our own

Signed-off-by: dann frazier <dannf@hp.com>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: Kyle McMartin <kyle@mcmartin.ca>
Cc: Grant Grundler <grundler@parisc-linux.org>
Cc: Matthew Wilcox <matthew@wil.cx>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/rtc/rtc-parisc.c

index 319bb5d445ea8177d39e4ee48c69c4e037ef4d33..cb087ad407f60af50b0be1e6b2ec44e12d6fa851 100644 (file)
 /* as simple as can be, and no simpler. */
 struct parisc_rtc {
        struct rtc_device *rtc;
-       spinlock_t lock;
 };
 
 static int parisc_get_time(struct device *dev, struct rtc_time *tm)
 {
-       struct parisc_rtc *p = dev_get_drvdata(dev);
-       unsigned long flags, ret;
+       unsigned long ret;
 
-       spin_lock_irqsave(&p->lock, flags);
        ret = get_rtc_time(tm);
-       spin_unlock_irqrestore(&p->lock, flags);
 
        if (ret & RTC_BATT_BAD)
                return -EOPNOTSUPP;
@@ -34,13 +30,9 @@ static int parisc_get_time(struct device *dev, struct rtc_time *tm)
 
 static int parisc_set_time(struct device *dev, struct rtc_time *tm)
 {
-       struct parisc_rtc *p = dev_get_drvdata(dev);
-       unsigned long flags;
        int ret;
 
-       spin_lock_irqsave(&p->lock, flags);
        ret = set_rtc_time(tm);
-       spin_unlock_irqrestore(&p->lock, flags);
 
        if (ret < 0)
                return -EOPNOTSUPP;
@@ -61,8 +53,6 @@ static int __devinit parisc_rtc_probe(struct platform_device *dev)
        if (!p)
                return -ENOMEM;
 
-       spin_lock_init(&p->lock);
-
        p->rtc = rtc_device_register("rtc-parisc", &dev->dev, &parisc_rtc_ops,
                                        THIS_MODULE);
        if (IS_ERR(p->rtc)) {