diff options
author | Jia-Ju Bai <baijiaju1990@gmail.com> | 2018-07-30 21:59:03 +0800 |
---|---|---|
committer | Alexandre Belloni <alexandre.belloni@bootlin.com> | 2018-08-27 22:51:54 +0200 |
commit | c6b5eb8dd33480104683d838da10c7ec77d32e8a (patch) | |
tree | 6de9fdf1150320727f9e6b0125f2b19fd0b6c26b /drivers/rtc/rtc-mrst.c | |
parent | 5b394b2ddf0347bef56e50c69a58773c94343ff3 (diff) | |
download | linux-c6b5eb8dd33480104683d838da10c7ec77d32e8a.tar.gz linux-c6b5eb8dd33480104683d838da10c7ec77d32e8a.tar.bz2 linux-c6b5eb8dd33480104683d838da10c7ec77d32e8a.zip |
rtc: rtc-mrst: Replace mdelay() with msleep() in mrst_read_time()
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>
Diffstat (limited to 'drivers/rtc/rtc-mrst.c')
-rw-r--r-- | drivers/rtc/rtc-mrst.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/rtc/rtc-mrst.c b/drivers/rtc/rtc-mrst.c index 1925aaf09093..bf2d7af03d07 100644 --- a/drivers/rtc/rtc-mrst.c +++ b/drivers/rtc/rtc-mrst.c @@ -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); |