Re: [PATCH v3 2/3] clocksource/drivers/timer-tegra186: fix watchdog self-pinging

From: Thierry Reding
Date: Thu Apr 10 2025 - 06:16:02 EST


On Mon, Apr 07, 2025 at 06:23:22PM +0800, Robert Lin wrote:
> From: Pohsun Su <pohsuns@xxxxxxxxxx>
>
> This change removes watchdog self-pinging behavior.
>
> The timer irq handler is triggered due to the 1st expiration,
> the handler disables and enables watchdog but also implicitly
> clears the expiration count so the count can only be 0 or 1.
>
> Since this watchdog supports opened, configured, or pinged by
> systemd, We remove this behavior or the watchdog may not bark
> when systemd crashes since the 5th expiration never comes.
>
> Signed-off-by: Pohsun Su <pohsuns@xxxxxxxxxx>
> Signed-off-by: Robert Lin <robelin@xxxxxxxxxx>
> ---
> drivers/clocksource/timer-tegra186.c | 24 ------------------------
> 1 file changed, 24 deletions(-)

Yeah, in retrospect this doesn't make sense:

Acked-by: Thierry Reding <treding@xxxxxxxxxx>

Attachment: signature.asc
Description: PGP signature