diff options
| author |   Bumsik Kim <kbumsik@gmail.com> | 2020-04-03 12:15:07 +0900 |
|---|---|---|
| committer |   Wim Van Sebroeck <wim@linux-watchdog.org> | 2020-05-25 08:55:42 +0200 |
| commit | f249eef9e66e46065a42a0c164ecc3a40f52102a (patch) | |
| tree | 1d1ea1bcf8d37e0b28544fcc96f9b173fddae8ad /include | |
| parent | watchdog: imx_sc_wdt: Fix reboot on crash (diff) | |
| download | wireguard-linux-f249eef9e66e46065a42a0c164ecc3a40f52102a.tar.xz wireguard-linux-f249eef9e66e46065a42a0c164ecc3a40f52102a.zip | |
watchdog: clarify that stop() is optional
The commit d0684c8a9354 ("watchdog: Make stop function optional")
made stop function not mandatory, but the comments
and the doc weren't reflected. Fix it to clarify.
Signed-off-by: Bumsik Kim <k.bumsik@gmail.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Link: https://lore.kernel.org/r/20200403031507.63487-1-k.bumsik@gmail.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/linux/watchdog.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/watchdog.h b/include/linux/watchdog.h index 417d9f37077a..1464ce6ffa31 100644 --- a/include/linux/watchdog.h +++ b/include/linux/watchdog.h @@ -37,15 +37,15 @@ struct watchdog_governor; * * The watchdog_ops structure contains a list of low-level operations * that control a watchdog device. It also contains the module that owns - * these operations. The start and stop function are mandatory, all other + * these operations. The start function is mandatory, all other * functions are optional. */ struct watchdog_ops { struct module *owner; /* mandatory operations */ int (*start)(struct watchdog_device *); - int (*stop)(struct watchdog_device *); /* optional operations */ + int (*stop)(struct watchdog_device *); int (*ping)(struct watchdog_device *); unsigned int (*status)(struct watchdog_device *); int (*set_timeout)(struct watchdog_device *, unsigned int); |