Re: [PATCH] watchdog: imx2_wdt: Drop .remove callback

From: Guenter Roeck
Date: Mon Feb 24 2020 - 13:30:37 EST


On Mon, Feb 24, 2020 at 07:25:22PM +0100, Uwe Kleine-König wrote:
> On Mon, Feb 24, 2020 at 06:15:17AM -0800, Guenter Roeck wrote:
> > How would you expect the watchdog core to stop the watchdog
> > with no stop function in the driver ?
>
> I'm not 100% sure, but I think the situation is that you cannot stop the
> watchdog in the watchdog register range, but if you stop the clock it
> will never expire.
>
> That's why I asked if the devm action callback stops the watchdog.
>

Again, it should not be possible to unload the driver if the watchdog
is running.

Guenter