Re: [PATCH] watchdog: da9052_wdt: respect TWDMIN

From: Marcus Folkesson
Date: Wed Jan 29 2025 - 10:09:51 EST


Hello,

On Wed, Jan 29, 2025 at 06:48:04AM -0800, Guenter Roeck wrote:
> On 1/29/25 01:24, Marcus Folkesson wrote:
> > From: Marcus Folkesson <marcus.folkesson@xxxxxxxxxxxxx>
> >
> > We have to wait at least the minimium time for the watchdog window
> > (TWDMIN) before writings to the wdt register after the
> > watchdog is activated.
> > Otherwise the chip will assert TWD_ERROR and power down to reset mode.
> >
>
> This is missing a key explanation:
>
> Why does setting @min_hw_heartbeat_ms in struct watchdog_device
> not work for this driver ?

Thank you Guenter, I wasn't aware of min_hw_heartbeat_ms. I will see
if it helps.

I've also noticed that the same problem occur when, for example, I set
the timeout with wdctl and .stop is called too soon as the application
terminates.

>
> Guenter
>

Best regards,
Marcus Folkesson

Attachment: signature.asc
Description: PGP signature