Skip to contentSkip to conversation

watchdog: ep93xx: Use the WatchDog Timer Core

PATCH New H Hartley Sweeten 1 participant 1 message Aug 8, 2011 Watchdog device drivers

lore
Share & tools
lore
#0 lore raw

PATCH v3 1/1 watchdog: ep93xx: Use the WatchDog Timer Core

Convert the ep93xx watchdog driver to using the WatchDog Timer Driver Core.

---

v3: add select WATCHDOG_CORE to Kconfig
reorder the code so that the #defines follow the #includes
    fix the namespace for some of the #defines
    remove WDIOF_CARDRESET from the watchdog_info options (this is a status flag not an options flag) hookup the nowayout module parameter to the status bit WDOG_NO_WAY_OUT
    reorder the code so that the relationship between the internal timer ping and the watchdog ping is more obvious

v2: use the pr_* functions instead of printk(KERN_* PFX
    document the register magic numbers and bits
    fold the register access functions into the callers
    add WDIOF_KEEPALIVEPING to the options
    fix the bootstatus use, any detected reset is reported as WDIOF_CARDRESET add a boot message so the user can see what actually caused the reset validate that the user's requested timeout is actually valid fix a typo in the v1 patch (extra void in ep93xx_wdt_start) fix ep93xx_wdt_exit, ep93xx_wdt_stop should be called not wdt_shutdown

drivers/watchdog/Kconfig +1 -0
index 86b0735..55fa024 100644
@@ -215,6 +215,7 @@ config MPCORE_WATCHDOG
 config EP93XX_WATCHDOG
 	tristate "EP93xx Watchdog"
 	depends on ARCH_EP93XX
+	select WATCHDOG_CORE
 	help
 	  Say Y here if to include support for the watchdog timer
 	  embedded in the Cirrus Logic EP93xx family of devices.
drivers/watchdog/ep93xx_wdt.c +96 -171

329 lines not on this page · raw message

--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html