Re: [PATCH 2/3] watchdog: Add Mstar MSC313e WDT driver
From: Randy Dunlap <hidden>
Date: 2021-05-25 18:50:02
Also in:
linux-arm-kernel, linux-watchdog, lkml
Hi, On 5/25/21 11:44 AM, Romain Perier wrote:
quoted hunk ↗ jump to hunk
diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index 355100dad60a..f53634ea0de6 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig@@ -980,6 +980,19 @@ config VISCONTI_WATCHDOG Say Y here to include support for the watchdog timer in Toshiba Visconti SoCs. +config MSC313E_WATCHDOG + tristate "MStar MSC313e watchdog" + depends on ARCH_MSTARV7 || COMPILE_TEST + depends on OF + select WATCHDOG_CORE + help + Say Y here to include support for the Watchdog timer embedded + into MStar MSC313e chips. This will reboot your system when the + timeout is reached. + + To compile this driver as a module, choose M here: the + module will be called msc313e_wdt.
AFAIK, you don't need the "depends on OF" line since the of*.h headers provide stubs for the cases of CONFIG_OF and/or CONFIG_OF_ADDRESS not set/enabled. Not having that line would also make COMPILE_TEST more effective. Can Rob or anyone else comment on this? thanks. -- ~Randy