On 11/18/2015 01:02 PM, Damien Riegel wrote:
gpio_wdt is one the very few drivers that do not stop the device on SYS_HALT and SYS_DOWN, but on SYS_HALT and SYS_POWER_OFF. As most of the drivers, it makes more sense to stop the watchdog on SYS_DOWN instead of SYS_POWER_OFF. Signed-off-by: Damien Riegel <damien.riegel@xxxxxxxxxxxxxxxxxxxx> Reviewed-by: Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx>
Makes sense. Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>
--- drivers/watchdog/gpio_wdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/watchdog/gpio_wdt.c b/drivers/watchdog/gpio_wdt.c index 90d59d3..1a3c6e8 100644 --- a/drivers/watchdog/gpio_wdt.c +++ b/drivers/watchdog/gpio_wdt.c @@ -136,7 +136,7 @@ static int gpio_wdt_notify_sys(struct notifier_block *nb, unsigned long code, switch (code) { case SYS_HALT: - case SYS_POWER_OFF: + case SYS_DOWN: gpio_wdt_disable(priv); break; default:
-- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html