Re: [PATCH v5 5/7] watchdog: f71808e_wdt: migrate to new kernel watchdog API

From: Guenter Roeck
Date: Sat Aug 28 2021 - 12:12:53 EST


On Mon, Aug 09, 2021 at 06:20:35PM +0200, Ahmad Fatoum wrote:
> Migrating the driver lets us drop the watchdog misc device boilerplate
> and reduces size by 285 lines. It also brings us support for new
> functionality like CONFIG_WATCHDOG_HANDLE_BOOT_ENABLED.
>
> This incurs a slight backwards-compatibility break, because the new
> kernel watchdog API doesn't support unloading modules for drivers
> whose watchdog hardware is reported to be running.
>
> This means following scenario will be no longer supported:
> - BIOS has enabled watchdog
> - Module is loaded and unloaded without opening watchdog
> - module_exit is expected to succeed and disable watchdog HW
>
> Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx>

Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>