All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] at91sam9_wdt: Allow watchdog to reset device at early boot
@ 2014-10-23 10:40 ` Timo Kokkonen
  0 siblings, 0 replies; 110+ messages in thread
From: Timo Kokkonen @ 2014-10-23 10:40 UTC (permalink / raw)
  To: linux-arm-kernel, linux-watchdog; +Cc: Timo Kokkonen

By default the driver will start a kernel timer which keeps on kicking
the watchdog HW until user space has opened the watchdog
device. Usually this is desirable as the watchdog HW is running by
default and the user space may not have any watchdog daemon running at
all.

However, on production systems it may be mandatory that also early
crashes and lockups will lead to a watchdog reset, even if they happen
before the user space has opened the watchdog device.

To resolve the issue, add a new device tree property
"atmel,no-early-timer" which will prevent the kernel timer from
pinging the watchdog HW on behalf of user space. The default is still
to use kernel timer, but more strict behavior can be enabled via the
device tree property.

Signed-off-by: Timo Kokkonen <timo.kokkonen@offcode.fi>
---
 Documentation/devicetree/bindings/watchdog/atmel-wdt.txt | 4 ++++
 drivers/watchdog/at91sam9_wdt.c                          | 6 +++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt b/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt
index f90e294..1b9289e 100644
--- a/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt
+++ b/Documentation/devicetree/bindings/watchdog/atmel-wdt.txt
@@ -28,6 +28,10 @@ Optional properties:
 	entering idle state.
 - atmel,dbg-halt : Should be present if you want to stop the watchdog when
 	entering debug state.
+- atmel,no-early-timer : Should be present if you want to let the
+	watchdog timer to expire even before user space has opened the
+	device. If not set, a kernel timer will keep on pinging the
+	watchdog until it is opened.
 
 Example:
 	watchdog@fffffd40 {
diff --git a/drivers/watchdog/at91sam9_wdt.c b/drivers/watchdog/at91sam9_wdt.c
index 489729b..8cac712 100644
--- a/drivers/watchdog/at91sam9_wdt.c
+++ b/drivers/watchdog/at91sam9_wdt.c
@@ -89,6 +89,7 @@ struct at91wdt {
 	u32 mr_mask;
 	unsigned long heartbeat;	/* WDT heartbeat in jiffies */
 	bool nowayout;
+	bool no_early_timer;
 	unsigned int irq;
 };
 
@@ -122,7 +123,7 @@ static void at91_ping(unsigned long data)
 {
 	struct at91wdt *wdt = (struct at91wdt *)data;
 	if (time_before(jiffies, wdt->next_heartbeat) ||
-	    !watchdog_active(&wdt->wdd)) {
+		(!watchdog_active(&wdt->wdd) && !wdt->no_early_timer)) {
 		at91_wdt_reset(wdt);
 		mod_timer(&wdt->timer, jiffies + wdt->heartbeat);
 	} else {
@@ -316,6 +317,9 @@ static int of_at91wdt_init(struct device_node *np, struct at91wdt *wdt)
 
 	wdt->mr |= max | ((max - min) << 16);
 
+	if (of_property_read_bool(np, "atmel,no-early-timer"))
+		wdt->no_early_timer = 1;
+
 	return 0;
 }
 #else
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 110+ messages in thread

end of thread, other threads:[~2015-02-23 17:43 UTC | newest]

Thread overview: 110+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-10-23 10:40 [PATCH] at91sam9_wdt: Allow watchdog to reset device at early boot Timo Kokkonen
2014-10-23 10:40 ` Timo Kokkonen
2014-11-12  8:20 ` Timo Kokkonen
2014-11-12  8:20   ` Timo Kokkonen
2014-11-13  9:12   ` Nicolas Ferre
2014-11-13  9:12     ` Nicolas Ferre
2014-11-14  8:40     ` Timo Kokkonen
2014-11-14  8:40       ` Timo Kokkonen
2014-11-21 12:23       ` Timo Kokkonen
2014-11-21 12:23         ` Timo Kokkonen
2014-11-27  6:53         ` Timo Kokkonen
2014-11-27  6:53           ` Timo Kokkonen
2014-11-27  9:22           ` Nicolas Ferre
2014-11-27  9:22             ` Nicolas Ferre
2014-11-27 17:23             ` Guenter Roeck
2014-11-27 17:23               ` Guenter Roeck
2014-11-27 19:06               ` Boris Brezillon
2014-11-27 19:06                 ` Boris Brezillon
2014-11-27 19:31                 ` Guenter Roeck
2014-11-27 19:31                   ` Guenter Roeck
2014-11-28  0:30                   ` Alexandre Belloni
2014-11-28  0:30                     ` Alexandre Belloni
2014-11-28  6:40                   ` Timo Kokkonen
2014-11-28  6:40                     ` Timo Kokkonen
2014-11-27 19:00         ` Boris Brezillon
2014-11-27 19:00           ` Boris Brezillon
2014-11-28  6:42           ` Timo Kokkonen
2014-11-28  6:42             ` Timo Kokkonen
2014-12-05 12:57           ` Timo Kokkonen
2014-12-05 12:57             ` Timo Kokkonen
2014-12-05 14:12             ` Boris Brezillon
2014-12-05 14:12               ` Boris Brezillon
2014-12-05 18:42               ` Timo Kokkonen
2014-12-05 18:42                 ` Timo Kokkonen
2014-12-05 19:02                 ` Guenter Roeck
2014-12-05 19:02                   ` Guenter Roeck
2014-12-05 20:32                   ` Timo Kokkonen
2014-12-05 20:32                     ` Timo Kokkonen
2014-12-05 21:39                     ` Guenter Roeck
2014-12-05 21:39                       ` Guenter Roeck
2014-12-06 10:11                       ` Timo Kokkonen
2014-12-06 10:11                         ` Timo Kokkonen
2015-01-13 14:53                         ` Guenter Roeck
2015-01-13 14:53                           ` Guenter Roeck
2015-01-14  6:09                           ` Timo Kokkonen
2015-01-14  6:09                             ` Timo Kokkonen
2015-02-18 12:57                           ` [PATCHv3 0/2] watchdog: Introduce "early-timeout-sec" property Timo Kokkonen
2015-02-18 12:57                             ` Timo Kokkonen
2015-02-18 12:57                             ` [PATCH 1/2] devicetree: Document generic watchdog properties Timo Kokkonen
2015-02-18 12:57                               ` Timo Kokkonen
2015-02-18 12:57                             ` [PATCH 2/2] at91sam9_wdt: Allow watchdog to reset device at early boot Timo Kokkonen
2015-02-18 12:57                               ` Timo Kokkonen
2015-02-18 13:21                               ` Boris Brezillon
2015-02-18 13:21                                 ` Boris Brezillon
2015-02-18 13:59                               ` Guenter Roeck
2015-02-18 13:59                                 ` Guenter Roeck
2015-02-18 14:17                                 ` Boris Brezillon
2015-02-18 14:17                                   ` Boris Brezillon
2015-02-18 14:50                                   ` Guenter Roeck
2015-02-18 14:50                                     ` Guenter Roeck
2015-02-18 16:00                                     ` Alexandre Belloni
2015-02-18 16:00                                       ` Alexandre Belloni
2015-02-18 17:50                                       ` Guenter Roeck
2015-02-18 17:50                                         ` Guenter Roeck
2015-02-18 20:21                                         ` Boris Brezillon
2015-02-18 20:21                                           ` Boris Brezillon
2015-02-19  6:02                                           ` Timo Kokkonen
2015-02-19  6:02                                             ` Timo Kokkonen
2015-02-18 21:11                                       ` Rob Herring
2015-02-18 21:11                                         ` Rob Herring
2015-02-19  6:14                                         ` Timo Kokkonen
2015-02-19  6:14                                           ` Timo Kokkonen
2015-02-20 14:06                                           ` Rob Herring
2015-02-20 14:06                                             ` Rob Herring
2015-02-20 16:28                                             ` Guenter Roeck
2015-02-20 16:28                                               ` Guenter Roeck
2015-02-20 19:43                                               ` Boris Brezillon
2015-02-20 19:43                                                 ` Boris Brezillon
2015-02-20 20:04                                                 ` Guenter Roeck
2015-02-20 20:04                                                   ` Guenter Roeck
2015-02-20  7:48                               ` Jean-Christophe PLAGNIOL-VILLARD
2015-02-20  7:48                                 ` Jean-Christophe PLAGNIOL-VILLARD
2015-02-20  7:51                                 ` Boris Brezillon
2015-02-20  7:51                                   ` Boris Brezillon
2015-02-20 16:33                                   ` Jean-Christophe PLAGNIOL-VILLARD
2015-02-20 16:33                                     ` Jean-Christophe PLAGNIOL-VILLARD
2015-02-20 17:16                                     ` Boris Brezillon
2015-02-20 17:16                                       ` Boris Brezillon
2015-02-20 18:06                                       ` Guenter Roeck
2015-02-20 18:06                                         ` Guenter Roeck
2015-02-23  7:29                                         ` Timo Kokkonen
2015-02-23  7:29                                           ` Timo Kokkonen
2015-02-23  8:51                                           ` Boris Brezillon
2015-02-23  8:51                                             ` Boris Brezillon
2015-02-23  9:11                                             ` Timo Kokkonen
2015-02-23  9:11                                               ` Timo Kokkonen
2015-02-23 16:19                                               ` Guenter Roeck
2015-02-23 16:19                                                 ` Guenter Roeck
2015-02-23 17:10                                                 ` Rob Herring
2015-02-23 17:10                                                   ` Rob Herring
2015-02-23 17:43                                                   ` Guenter Roeck
2015-02-23 17:43                                                     ` Guenter Roeck
2015-02-20  8:00                                 ` Timo Kokkonen
2015-02-20  8:00                                   ` Timo Kokkonen
2015-02-20 16:09                                 ` Guenter Roeck
2015-02-20 16:09                                   ` Guenter Roeck
2015-02-18 13:16                             ` [PATCHv3 0/2] watchdog: Introduce "early-timeout-sec" property Boris Brezillon
2015-02-18 13:16                               ` Boris Brezillon
2015-02-18 13:51                               ` Timo Kokkonen
2015-02-18 13:51                                 ` Timo Kokkonen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.