Re: [PATCH] rtc-m48t59: fix setting of a year on m48t08 and m48t02

[Robert Reif <reif@earthlink.net>]

[-- Attachment #1: Type: text/plain, Size: 305 bytes --]

Krzysztof Helt wrote:
> From: Krzysztof Helt <krzysztof.h1@wp.pl>
>
> Fix setting of a year in m48t08 and m48t02 clocks.
> They do not have century bits and setting them
> causes the year to overflow (bit 0x80 set).
>   
I think this fix is also needed.

Signed-off-by: Robert Reif <reif@earthlink.net>



[-- Attachment #2: rtc.diff.txt --]
[-- Type: text/plain, Size: 591 bytes --]

diff --git a/drivers/rtc/rtc-m48t59.c b/drivers/rtc/rtc-m48t59.c
index 761e9d7..ce4eff6 100644
--- a/drivers/rtc/rtc-m48t59.c
+++ b/drivers/rtc/rtc-m48t59.c
@@ -82,7 +82,8 @@ static int m48t59_rtc_read_time(struct device *dev, struct rtc_time *tm)
 	tm->tm_mday	= BCD2BIN(M48T59_READ(M48T59_MDAY));
 
 	val = M48T59_READ(M48T59_WDAY);
-	if ((val & M48T59_WDAY_CEB) && (val & M48T59_WDAY_CB)) {
+	if ((pdata->type == M48T59RTC_TYPE_M48T59) &&
+	    (val & M48T59_WDAY_CEB) && (val & M48T59_WDAY_CB)) {
 		dev_dbg(dev, "Century bit is enabled\n");
 		tm->tm_year += 100;	/* one century */
 	}