Messages in this thread | ![/](/images/icornerl.gif) | | Date | Mon, 29 Jan 2024 22:07:16 -0800 | Subject | Re: [PATCH] watchdog: starfive: check watchdog status before enabling in system resume | From | Guenter Roeck <> |
| |
On 1/29/24 21:51, Ji Sheng Teoh wrote: > System resume will start and enable watchdog regardless of whether the > watchdog is enabled/disabled during a system suspend. > Add a check to the watchdog status and only start and enable the > watchdog if the watchdog status is running/active. > > Signed-off-by: Sia Jee Heng <jeeheng.sia@starfivetech.com> > Signed-off-by: Ji Sheng Teoh <jisheng.teoh@starfivetech.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> --- > drivers/watchdog/starfive-wdt.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/watchdog/starfive-wdt.c b/drivers/watchdog/starfive-wdt.c > index e28ead24c520..f72fc38a41f8 100644 > --- a/drivers/watchdog/starfive-wdt.c > +++ b/drivers/watchdog/starfive-wdt.c > @@ -554,7 +554,10 @@ static int starfive_wdt_resume(struct device *dev) > starfive_wdt_set_reload_count(wdt, wdt->reload); > starfive_wdt_lock(wdt); > > - return starfive_wdt_start(wdt); > + if (watchdog_active(&wdt->wdd)) > + return starfive_wdt_start(wdt); > + > + return 0; > } > > static int starfive_wdt_runtime_suspend(struct device *dev)
| ![\](/images/icornerr.gif) |