moonjuice/kernel_arpi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert "rtc: pcf8523: properly handle oscillator stop bit"

Browse Source 
This reverts commit ede44c908d.

See: https://github.com/raspberrypi/firmware/issues/1065

Signed-off-by: Phil Elwell <phil@raspberrypi.org>
This commit is contained in:
Phil Elwell
2018-10-29 14:45:45 +00:00
committed by Dom Cobley
parent 52f1453513
commit 8119688311
1 changed files with 22 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
drivers/rtc/rtc-pcf8523.c
View File

@@ -242,8 +242,28 @@ static int pcf8523_rtc_read_time(struct device *dev, struct rtc_time *tm)
if (err < 0) if (err < 0)
return err; return err;
if (regs[0] & PCF8523_SECONDS_OS) if (regs[0] & PCF8523_SECONDS_OS) {
return -EINVAL; /*
* If the oscillator was stopped, try to clear the flag. Upon
* power-up the flag is always set, but if we cannot clear it
* the oscillator isn't running properly for some reason. The
* sensible thing therefore is to return an error, signalling
* that the clock cannot be assumed to be correct.
*/
regs[0] &= ~PCF8523_SECONDS_OS;
err = pcf8523_write(client, PCF8523_REG_SECONDS, regs[0]);
if (err < 0)
return err;
err = pcf8523_read(client, PCF8523_REG_SECONDS, &regs[0]);
if (err < 0)
return err;
if (regs[0] & PCF8523_SECONDS_OS)
return -EAGAIN;
}
tm->tm_sec = bcd2bin(regs[0] & 0x7f); tm->tm_sec = bcd2bin(regs[0] & 0x7f);
tm->tm_min = bcd2bin(regs[1] & 0x7f); tm->tm_min = bcd2bin(regs[1] & 0x7f);