drivers: rtc: correct week day for mc146818
authorHeinrich Schuchardt <xypron.glpk@gmx.de>
Sat, 7 Jul 2018 22:07:26 +0000 (00:07 +0200)
committerTom Rini <trini@konsulko.com>
Fri, 20 Jul 2018 19:55:04 +0000 (15:55 -0400)
For qemu-x86 the date command produces wrong days of the week:
Date: 2018-07-06 (Saturday)    Time: 18:02:03
Date: 2018-07-07 (unknown day)    Time: 21:02:06

According to a comment in the Linux driver the mc146818 only updates the
day of the week if the register value is non-zero.

Sunday is 1, saturday is 7 unlike in U-Boot (see data sheet
https://www.nxp.com/docs/en/data-sheet/MC146818.pdf).

So let's use our library function to determine the day of the week.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
drivers/rtc/mc146818.c

index 500eef2ee0c520d3ab145ec65a6aa9bce5751304..ee29c711aaacb15fe4295c2ed3d16bd38d1feddb 100644 (file)
@@ -83,7 +83,7 @@ static void mc146818_write8(int reg, uchar val)
 
 static int mc146818_get(struct rtc_time *tmp)
 {
-       uchar sec, min, hour, mday, wdaymon, year;
+       uchar sec, min, hour, mday, wday __attribute__((unused)),mon, year;
 
        /* here check if rtc can be accessed */
        while ((mc146818_read8(RTC_CONFIG_A) & 0x80) == 0x80)
@@ -111,7 +111,6 @@ static int mc146818_get(struct rtc_time *tmp)
        tmp->tm_mday = bcd2bin(mday & 0x3f);
        tmp->tm_mon  = bcd2bin(mon & 0x1f);
        tmp->tm_year = bcd2bin(year);
-       tmp->tm_wday = bcd2bin(wday & 0x07);
 
        if (tmp->tm_year < 70)
                tmp->tm_year += 2000;
@@ -120,6 +119,11 @@ static int mc146818_get(struct rtc_time *tmp)
 
        tmp->tm_yday = 0;
        tmp->tm_isdst = 0;
+       /*
+        * The mc146818 only updates wday if it is non-zero, sunday is 1
+        * saturday is 7. So let's use our library routine.
+        */
+       rtc_calc_weekday(tmp);
 #ifdef RTC_DEBUG
        printf("Get DATE: %4d-%02d-%02d (wday=%d)  TIME: %2d:%02d:%02d\n",
               tmp->tm_year, tmp->tm_mon, tmp->tm_mday, tmp->tm_wday,