commit: bea1906620ce72b63f83735c4cc2642b25ec54ae From: David Engraf <david.eng...@sysgo.com> Date: Wed, 20 Jul 2011 15:03:39 +0200 Subject: [PATCH] watchdog: shwdt: fix usage of mod_timer
Fix the usage of mod_timer() and make the driver usable. mod_timer() must be called with an absolute timeout in jiffies. The old implementation used a relative timeout thus the hardware watchdog was never triggered. Signed-off-by: David Engraf <david.eng...@sysgo.com> Signed-off-by: Paul Mundt <let...@linux-sh.org> Signed-off-by: Wim Van sebroeck <w...@iguana.be> Signed-off-by: Andrew Morton <a...@linux-foundation.org> Cc: stable <sta...@kernel.org> --- drivers/watchdog/shwdt.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/watchdog/shwdt.c b/drivers/watchdog/shwdt.c index db84f23..a267dc0 100644 --- a/drivers/watchdog/shwdt.c +++ b/drivers/watchdog/shwdt.c @@ -64,7 +64,7 @@ * misses its deadline, the kernel timer will allow the WDT to overflow. */ static int clock_division_ratio = WTCSR_CKS_4096; -#define next_ping_period(cks) msecs_to_jiffies(cks - 4) +#define next_ping_period(cks) (jiffies + msecs_to_jiffies(cks - 4)) static const struct watchdog_info sh_wdt_info; static struct platform_device *sh_wdt_dev; _______________________________________________ stable mailing list stable@linux.kernel.org http://linux.kernel.org/mailman/listinfo/stable