| Message ID | 20210708051304.341278-1-nobuhiro1.iwamatsu@toshiba.co.jp |
|---|---|
| State | New |
| Headers | show |
| Series | rtc: s5m: Check return value of s5m_check_peding_alarm_interrupt() | expand |
On Thu, 8 Jul 2021 14:13:04 +0900, Nobuhiro Iwamatsu wrote: > 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. Applied, thanks! [1/1] rtc: s5m: Check return value of s5m_check_peding_alarm_interrupt() commit: f2581b1dfa9858e342afa8034b0f64f923bbf233 Best regards, -- Alexandre Belloni <alexandre.belloni@bootlin.com>
diff --git a/drivers/rtc/rtc-s5m.c b/drivers/rtc/rtc-s5m.c index 038269a6b08c55..6b56f8eacba6fa 100644 --- a/drivers/rtc/rtc-s5m.c +++ b/drivers/rtc/rtc-s5m.c @@ -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)
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> --- drivers/rtc/rtc-s5m.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)