rtc: rx8581: error out when time invalid
authorAlexandre Belloni <alexandre.belloni@bootlin.com>
Thu, 17 May 2018 20:33:30 +0000 (22:33 +0200)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Thu, 17 May 2018 20:43:22 +0000 (22:43 +0200)
Return an error when the date is unreliable because the battery is low.

Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
drivers/rtc/rtc-rx8581.c

index 0a70f1a..54631a3 100644 (file)
@@ -109,6 +109,12 @@ static int rx8581_rtc_read_time(struct device *dev, struct rtc_time *tm)
                return -EIO;
        }
 
+       if (data & RX8581_FLAG_VLF) {
+               dev_warn(dev,
+                        "low voltage detected, date/time is not reliable.\n");
+               return -EINVAL;
+       }
+
        do {
                /* If update flag set, clear it */
                if (data & RX8581_FLAG_UF) {
@@ -136,10 +142,6 @@ static int rx8581_rtc_read_time(struct device *dev, struct rtc_time *tm)
                }
        } while (data & RX8581_FLAG_UF);
 
-       if (data & RX8581_FLAG_VLF)
-               dev_info(dev,
-                       "low voltage detected, date/time is not reliable.\n");
-
        dev_dbg(dev, "%s: raw data is sec=%02x, min=%02x, hr=%02x, "
                "wday=%02x, mday=%02x, mon=%02x, year=%02x\n",
                __func__,