On 23/07/2019 16:13:12+0200, Bruno Thomsen wrote:
> Hi all
>
> > Patches has been tested on a pcf2127 chip using 2MHz SPI
> > interface both as built-in and module including with and
> > without watchdog feature.
>
> I did some more testing as I could not understand this bugfix:
>
> 3769a375ab83 rtc: pcf2127: bulk read only date and time registers.
>
> This fix seems to be incomplete as root cause is not auto-increment
> read aka bulk read, but reading control register 2 triggers zero value
> in WD_VAL resulting in stopped watchdog until systemd kick the dog
> again :)
>
> As the watchdog has 2 control registers I will do some more testing
> to see if this also apply to the other register. But more importantly
> this issue also affect timestamp0_{store,show} functions in tamper
> detection support, and I will therefor send a v2 of the patch series.
>
Please Cc the watchdog maintainer on v2.
> /Bruno
--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
