svn commit: r276286 - stable/10/sys/arm/broadcom/bcm2835

[Ian Lepore]

Author: ian
Date: Sat Dec 27 04:51:21 2014
New Revision: 276286
URL: https://svnweb.freebsd.org/changeset/base/276286

Log:
  MFC r275660:  Fix the watchdog timeout calculation to prevent wrap.

Modified:
  stable/10/sys/arm/broadcom/bcm2835/bcm2835_wdog.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/arm/broadcom/bcm2835/bcm2835_wdog.c
==============================================================================
--- stable/10/sys/arm/broadcom/bcm2835/bcm2835_wdog.c	Sat Dec 27 04:35:18 2014	(r276285)
+++ stable/10/sys/arm/broadcom/bcm2835/bcm2835_wdog.c	Sat Dec 27 04:51:21 2014	(r276286)
@@ -142,14 +142,13 @@ bcmwd_watchdog_fn(void *private, u_int c
 
 	if (cmd > 0) {
 		sec = ((uint64_t)1 << (cmd & WD_INTERVAL)) / 1000000000;
-		ticks = (sec << 16) & BCM2835_WDOG_TIME_MASK;
-		if (ticks == 0) {
+		if (sec == 0 || sec > 15) {
 			/* 
 			 * Can't arm
 			 * disable watchdog as watchdog(9) requires
 			 */
 			device_printf(sc->dev,
-			    "Can't arm, timeout is less than 1 second\n");
+			    "Can't arm, timeout must be between 1-15 seconds\n");
 			WRITE(sc, BCM2835_RSTC_REG, 
 			    (BCM2835_PASWORD << BCM2835_PASSWORD_SHIFT) |
 			    BCM2835_RSTC_RESET);
@@ -157,6 +156,7 @@ bcmwd_watchdog_fn(void *private, u_int c
 			return;
 		}
 
+		ticks = (sec << 16) & BCM2835_WDOG_TIME_MASK;
 		reg = (BCM2835_PASWORD << BCM2835_PASSWORD_SHIFT) | ticks;
 		WRITE(sc, BCM2835_WDOG_REG, reg);