RE: [PATCH v4 1/3] watchdog: aspeed: Update bootstatus handling

Hi Andrew,

-----Original Message-----
From: Andrew Jeffery <andrew@codeconstruct.com.au>
Sent: Wednesday, November 20, 2024 12:54 PM
Subject: Re: [PATCH v4 1/3] watchdog: aspeed: Update bootstatus handling

On Mon, 2024-11-18 at 17:27 -0800, Guenter Roeck wrote:

quoted

So, again, what exactly would userspace do with the information that
this was a watchdog triggered warm reboot ? Why would it need that
information ?

I'll defer to the others on To/Cc to answer that.

My only position is I don't think changing behaviour of existing drivers to
exploit WDIOF_EXTERN1 as a graceful-reboot indicator is a good idea either.
Obviously I don't have much skin in the game with watchdog maintenance, so
my thoughts shouldn't have much influence beyond the Aspeed-specifics, but I
just didn't want to see some fun new confusion or incompatibility arise as a
result.

Agree to your misgiving, in the next patches, only two categories, "Power on reset"
and "WDT reset" (Card reset), will be taken into consideration. The graceful-reboot
scenario will be postponed to the patches in the future.


Thanks,
Chin-Ting