summaryrefslogtreecommitdiff
path: root/drivers/watchdog/da9062_wdt.c
diff options
context:
space:
mode:
authorMarco Felsch <m.felsch@pengutronix.de>2020-01-20 12:17:29 +0300
committerWim Van Sebroeck <wim@linux-watchdog.org>2020-02-17 15:19:08 +0300
commite9a0e65eda3f78d0b04ec6136c591c000cbc3b76 (patch)
tree586f820e308a810883c4d58b9a1c453bec0d50cc /drivers/watchdog/da9062_wdt.c
parent5eee7c625d414fb62985439ed58ab755d8988c76 (diff)
downloadlinux-e9a0e65eda3f78d0b04ec6136c591c000cbc3b76.tar.xz
watchdog: da9062: do not ping the hw during stop()
The da9062 hw has a minimum ping cool down phase of at least 200ms. The driver takes that into account by setting the min_hw_heartbeat_ms to 300ms and the core guarantees that the hw limit is observed for the ping() calls. But the core can't guarantee the required minimum ping cool down phase if a stop() command is send immediately after the ping() command. So it is not allowed to ping the watchdog within the stop() command as the driver does. Remove the ping can be done without doubts because the watchdog gets disabled anyway and a (re)start resets the watchdog counter too. Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> Reviewed-by: Guenter Roeck <linux@roeck-us.net> Link: https://lore.kernel.org/r/20200120091729.16256-1-m.felsch@pengutronix.de [groeck: Updated description] Signed-off-by: Guenter Roeck <linux@roeck-us.net> Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
Diffstat (limited to 'drivers/watchdog/da9062_wdt.c')
-rw-r--r--drivers/watchdog/da9062_wdt.c7
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/watchdog/da9062_wdt.c b/drivers/watchdog/da9062_wdt.c
index 47eefe072b40..777d7eec7f2e 100644
--- a/drivers/watchdog/da9062_wdt.c
+++ b/drivers/watchdog/da9062_wdt.c
@@ -95,13 +95,6 @@ static int da9062_wdt_stop(struct watchdog_device *wdd)
struct da9062_watchdog *wdt = watchdog_get_drvdata(wdd);
int ret;
- ret = da9062_reset_watchdog_timer(wdt);
- if (ret) {
- dev_err(wdt->hw->dev, "Failed to ping the watchdog (err = %d)\n",
- ret);
- return ret;
- }
-
ret = regmap_update_bits(wdt->hw->regmap,
DA9062AA_CONTROL_D,
DA9062AA_TWDSCALE_MASK,