| From 7372971c1be5b7d4fdd8ad237798bdc1d1d54162 Mon Sep 17 00:00:00 2001 |
| From: Dan Carpenter <dan.carpenter@oracle.com> |
| Date: Tue, 11 Jan 2022 10:19:22 +0300 |
| Subject: rtc: mc146818-lib: fix signedness bug in mc146818_get_time() |
| MIME-Version: 1.0 |
| Content-Type: text/plain; charset=UTF-8 |
| Content-Transfer-Encoding: 8bit |
| |
| From: Dan Carpenter <dan.carpenter@oracle.com> |
| |
| commit 7372971c1be5b7d4fdd8ad237798bdc1d1d54162 upstream. |
| |
| The mc146818_get_time() function returns zero on success or negative |
| a error code on failure. It needs to be type int. |
| |
| Fixes: d35786b3a28d ("rtc: mc146818-lib: change return values of mc146818_get_time()") |
| Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> |
| Reviewed-by: Mateusz Jończyk <mat.jonczyk@o2.pl> |
| Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com> |
| Link: https://lore.kernel.org/r/20220111071922.GE11243@kili |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/rtc/rtc-mc146818-lib.c | 2 +- |
| include/linux/mc146818rtc.h | 2 +- |
| 2 files changed, 2 insertions(+), 2 deletions(-) |
| |
| --- a/drivers/rtc/rtc-mc146818-lib.c |
| +++ b/drivers/rtc/rtc-mc146818-lib.c |
| @@ -33,7 +33,7 @@ bool mc146818_does_rtc_work(void) |
| } |
| EXPORT_SYMBOL_GPL(mc146818_does_rtc_work); |
| |
| -unsigned int mc146818_get_time(struct rtc_time *time) |
| +int mc146818_get_time(struct rtc_time *time) |
| { |
| unsigned char ctrl; |
| unsigned long flags; |
| --- a/include/linux/mc146818rtc.h |
| +++ b/include/linux/mc146818rtc.h |
| @@ -124,7 +124,7 @@ struct cmos_rtc_board_info { |
| #endif /* ARCH_RTC_LOCATION */ |
| |
| bool mc146818_does_rtc_work(void); |
| -unsigned int mc146818_get_time(struct rtc_time *time); |
| +int mc146818_get_time(struct rtc_time *time); |
| int mc146818_set_time(struct rtc_time *time); |
| |
| #endif /* _MC146818RTC_H */ |