rtc: s5m: Check return value of s5m_check_peding_alarm_interrupt()
authorNobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>
Thu, 8 Jul 2021 05:13:04 +0000 (14:13 +0900)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Sat, 10 Jul 2021 00:41:52 +0000 (02:41 +0200)
s5m_check_peding_alarm_interrupt() in s5m_rtc_read_alarm() gets the return
value, but doesn't use it.
This modifies using the s5m_check_peding_alarm_interrupt()"s return value
as the s5m_rtc_read_alarm()'s return value.

Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: linux-samsung-soc@vger.kernel.org
Signed-off-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Link: https://lore.kernel.org/r/20210708051304.341278-1-nobuhiro1.iwamatsu@toshiba.co.jp
drivers/rtc/rtc-s5m.c

index 038269a..6b56f8e 100644 (file)
@@ -488,9 +488,7 @@ static int s5m_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm)
 
        dev_dbg(dev, "%s: %ptR(%d)\n", __func__, &alrm->time, alrm->time.tm_wday);
 
-       ret = s5m_check_peding_alarm_interrupt(info, alrm);
-
-       return 0;
+       return s5m_check_peding_alarm_interrupt(info, alrm);
 }
 
 static int s5m_rtc_stop_alarm(struct s5m_rtc_info *info)