On 4/12/20 4:01 PM, Fabio Estevam wrote:
Currently when running the samples/watchdog/watchdog-simple.c application and forcing a kernel crash by doing:
# ./watchdog-simple & # echo c > /proc/sysrq-trigger
The system does not reboot as expected.
Fix it by calling imx_sc_wdt_set_timeout() to configure the i.MX8QXP watchdog with a proper timeout.
Cc: stable@vger.kernel.org Fixes: 986857acbc9a ("watchdog: imx_sc: Add i.MX system controller watchdog support") Reported-by: Breno Lima breno.lima@nxp.com Signed-off-by: Fabio Estevam festevam@gmail.com
Reviewed-by: Guenter Roeck linux@roeck-us.net
drivers/watchdog/imx_sc_wdt.c | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/drivers/watchdog/imx_sc_wdt.c b/drivers/watchdog/imx_sc_wdt.c index 60a32469f7de..e9ee22a7cb45 100644 --- a/drivers/watchdog/imx_sc_wdt.c +++ b/drivers/watchdog/imx_sc_wdt.c @@ -175,6 +175,11 @@ static int imx_sc_wdt_probe(struct platform_device *pdev) wdog->timeout = DEFAULT_TIMEOUT; watchdog_init_timeout(wdog, 0, dev);
- ret = imx_sc_wdt_set_timeout(wdog, wdog->timeout);
- if (ret)
return ret;- watchdog_stop_on_reboot(wdog); watchdog_stop_on_unregister(wdog);