Hello, On 13/10/2021 10:48:48+0100, Phil Elwell wrote: > Hi, > > We're using the PCF85063A on our Compute Module 4 I/O board, and a number of > users have experienced problems with invalid state after inserting a backup > battery. So far the problem has been confined to EXT_TEST (bit 7 of CTRL_1) > being set, but there are a number of registers for which the driver only > modifies a subset of the bits. > > I can think of a number of ways to clear this invalid state, none of which > are difficult - force a software reset whenever certain bits of certain > registers have certain values, always specifically force some bits to known > values, etc. - > but are there any preferences or pitfalls to be aware of? > This is a very difficult topic because contrary to most of the other IPs, the RTC is still running while Linux is not and it is expected that it will retain its configuration across reboots (well, this is exactly what the RTC is for). This means that you may have some code (bootloader, firmware) running before Linux configuring the RTC and so setting or clearing bits in registers that are never touched by Linux. So, it is not possible to go and change random bits because they don't have the default value or the value we expect. Instead it is usually necessary to provide a way to set those bits. However, for EXT_TEST, I would assume the RTC lost its time once it is set so I would treat it just like OS: if it is set, then leave it that way until .set_time is called and return -EINVAL in .read_time. I didn't try but if OS is always set by the RTC when EXT_TEST is set, then maybe we don't need to check in .read_time. -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
