Thread (15 messages) 15 messages, 3 authors, 2021-11-10

Re: [PATCH v4 3/3] watchdog: meson_gxbb_wdt: remove stop_on_reboot

From: Art Nikpal <hidden>
Date: 2021-11-09 08:00:00
Also in: linux-amlogic, linux-watchdog, lkml

hi Guenter Roeck
why still not merged to upstream ?

On Fri, Jul 30, 2021 at 12:14 PM Artem Lapkin [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Remove watchdog_stop_on_reboot()

Meson platform still have some hardware drivers problems for some
configurations which can freeze device on shutdown/reboot stage and i
think better to have reboot warranty by default.

I feel that it is important to keep the watchdog running during the
reboot sequence, in the event that an abnormal driver freezes the reboot
process.

This is my personal opinion and I hope the driver authors will agree
with my proposal, or just ignore this commit if not.

https://lore.kernel.org/linux-watchdog/20210729072308.1908904-1-art@khadas.com/T/#t (local)

Signed-off-by: Artem Lapkin <redacted>
---
 drivers/watchdog/meson_gxbb_wdt.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/watchdog/meson_gxbb_wdt.c b/drivers/watchdog/meson_gxbb_wdt.c
index 945f5e65db57..d3c9e2f6e63b 100644
--- a/drivers/watchdog/meson_gxbb_wdt.c
+++ b/drivers/watchdog/meson_gxbb_wdt.c
@@ -198,7 +198,6 @@ static int meson_gxbb_wdt_probe(struct platform_device *pdev)

        meson_gxbb_wdt_set_timeout(&data->wdt_dev, data->wdt_dev.timeout);

-       watchdog_stop_on_reboot(&data->wdt_dev);
        return devm_watchdog_register_device(dev, &data->wdt_dev);
 }

--
2.25.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help