From 2ca1606359695cc7ce371c7829c4082d9095fc06 Mon Sep 17 00:00:00 2001 From: Wim Van Sebroeck Date: Thu, 22 Mar 2012 09:37:10 +0100 Subject: [PATCH] watchdog: ep93xx_wdt: timeout is an unsigned int value. the timeout is a positive thus unsigned int value. Also re-add the comment about the actual heartbeat. Signed-off-by: Wim Van Sebroeck --- drivers/watchdog/ep93xx_wdt.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/watchdog/ep93xx_wdt.c b/drivers/watchdog/ep93xx_wdt.c index 29575f8228b..77050037597 100644 --- a/drivers/watchdog/ep93xx_wdt.c +++ b/drivers/watchdog/ep93xx_wdt.c @@ -42,8 +42,8 @@ static bool nowayout = WATCHDOG_NOWAYOUT; module_param(nowayout, bool, 0); MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started"); -static int timeout = WDT_TIMEOUT; -module_param(timeout, int, 0); +static unsigned int timeout = WDT_TIMEOUT; +module_param(timeout, uint, 0); MODULE_PARM_DESC(timeout, "Watchdog timeout in seconds. (1<=timeout<=3600, default=" __MODULE_STRING(WDT_TIMEOUT) ")"); @@ -55,7 +55,7 @@ static unsigned long next_heartbeat; #define EP93XX_WATCHDOG 0x00 #define EP93XX_WDSTATUS 0x04 -/* reset the wdt every ~200ms */ +/* reset the wdt every ~200ms - the heartbeat of the device is 0.250 seconds*/ #define WDT_INTERVAL (HZ/5) static void ep93xx_wdt_timer_ping(unsigned long data) -- 2.39.5