On Fri, 2017-03-10 at 18:51 +0200, Andy Shevchenko wrote: > Firmware followed by bootloader leaves watchdog running. > > Keep it running in the driver. > > User will not need any additional options to reboot in case of panic > during boot. > - /* Make sure the watchdog is not running */ > - wdt_stop(wdt_dev); > + /* > + * Make sure the watchdog is serviced. > + * > + * The firmware followed by U-Boot leaves the watchdog > running > + * with the default threshold 60 seconds. Our default timeout > is > + * 90 seconds, but internal worker divides it by two, which > is > + * 45 seconds and should be enough (less by 15 seconds than > + * threshold). > + */ > + set_bit(WDOG_HW_RUNNING, &wdt_dev->status); Giving one more thought here perhaps more robust will be to start watchdog unconditionally here? In that case it will be no matter how it's configured in firmware and/or boot loader. > > ret = devm_watchdog_register_device(&pdev->dev, wdt_dev); > if (ret) { -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html