rtc: rtc-mrst: Replace mdelay() with msleep() in mrst_read_time()
authorJia-Ju Bai <baijiaju1990@gmail.com>
Mon, 30 Jul 2018 13:59:03 +0000 (21:59 +0800)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Mon, 27 Aug 2018 20:51:54 +0000 (22:51 +0200)
mrst_read_time() is never called in atomic context.
It calls mdelay() to busily wait, which is not necessary.
mdelay() can be replaced with msleep().

This is found by a static analysis tool named DCNS written by myself.

Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
drivers/rtc/rtc-mrst.c

index 1925aaf..bf2d7af 100644 (file)
@@ -90,7 +90,7 @@ static int mrst_read_time(struct device *dev, struct rtc_time *time)
        unsigned long flags;
 
        if (vrtc_is_updating())
-               mdelay(20);
+               msleep(20);
 
        spin_lock_irqsave(&rtc_lock, flags);
        time->tm_sec = vrtc_cmos_read(RTC_SECONDS);