watchdog: Improve description of min_hw_heartbeat_ms
authorGuenter Roeck <linux@roeck-us.net>
Fri, 27 May 2016 21:19:06 +0000 (14:19 -0700)
committerWim Van Sebroeck <wim@iguana.be>
Sun, 17 Jul 2016 19:00:14 +0000 (21:00 +0200)
The description of min_hw_heartbeat_ms is misleading and needs some
improvements.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Documentation/watchdog/watchdog-kernel-api.txt
include/linux/watchdog.h

index 917eeeabfa5eaefd3396c66e4b1032dd49a6f2b4..92700d229bf975f4d3a5efd5e165ef1d25fd34a4 100644 (file)
@@ -82,8 +82,9 @@ It contains following fields:
 * max_timeout: the watchdog timer's maximum timeout value (in seconds),
   as seen from userspace. If set, the maximum configurable value for
   'timeout'. Not used if max_hw_heartbeat_ms is non-zero.
-* min_hw_heartbeat_ms: Minimum time between heartbeats sent to the chip,
-  in milli-seconds.
+* min_hw_heartbeat_ms: Hardware limit for minimum time between heartbeats,
+  in milli-seconds. This value is normally 0; it should only be provided
+  if the hardware can not tolerate lower intervals between heartbeats.
 * max_hw_heartbeat_ms: Maximum hardware heartbeat, in milli-seconds.
   If set, the infrastructure will send heartbeats to the watchdog driver
   if 'timeout' is larger than max_hw_heartbeat_ms, unless WDOG_ACTIVE
index 6b75e38b683fc3c7b04a7839c827fed18ab3ca9e..7047bc7f810688bff4511a2d087d5207676b62b3 100644 (file)
@@ -66,7 +66,8 @@ struct watchdog_ops {
  *             as configurable from user space. Only relevant if
  *             max_hw_heartbeat_ms is not provided.
  * @min_hw_heartbeat_ms:
- *             Minimum time between heartbeats, in milli-seconds.
+ *             Hardware limit for minimum time between heartbeats,
+ *             in milli-seconds.
  * @max_hw_heartbeat_ms:
  *             Hardware limit for maximum timeout, in milli-seconds.
  *             Replaces max_timeout if specified.