watchdog: s3c2410_wdt: Set timeout to actually achieved timeout

While rebasing my "watchdog_dev: Let the driver update the timeout field on set_timeout success" patch (before I noticed it was already picked up by Wim), I noticed that the s3c2410_wdt driver may not always have a 1 second resolution, this patch changes s3c2410wdt_set_heartbeat to update the timeout to the actually achieved timeout. Signed-off-by: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Wim Van Sebroeck <wim@iguana.be>

watchdog: s3c2410_wdt: Set timeout to actually achieved timeout
While rebasing my "watchdog_dev: Let the driver update the timeout field on set_timeout success" patch (before I noticed it was already picked up by Wim), I noticed that the s3c2410_wdt driver may not always have a 1 second resolution, this patch changes s3c2410wdt_set_heartbeat to update the timeout to the actually achieved timeout. Signed-off-by: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Hans de Goede · Wim Van Sebroeck
1 parent abcf834b80
Showing 1 changed file with 1 additions and 1 deletions Side-by-side Diff
drivers/watchdog/s3c2410_wdt.c
@@ -202,7 +202,7 @@
 	writel(count, wdt_base + S3C2410_WTDAT);
 	writel(wtcon, wdt_base + S3C2410_WTCON);
  
-	wdd->timeout = timeout;
+	wdd->timeout = (count * divisor) / freq;
  
 	return 0;
 }
...	...	@@ -202,7 +202,7 @@
202	202	writel(count, wdt_base + S3C2410_WTDAT);
203	203	writel(wtcon, wdt_base + S3C2410_WTCON);
204	204
205		- wdd->timeout = timeout;
	205	+ wdd->timeout = (count * divisor) / freq;
206	206
207	207	return 0;
208	208	}