[riot-notifications] [RIOT-OS/RIOT] drivers/include: add header definition for wdt (#11527)

benpicco notifications at github.com
Wed May 15 15:13:41 CEST 2019


benpicco commented on this pull request.



> +typedef void(*wdt_cb_t)(void *arg);
+
+/**
+ * @brief    Start watchdog timer
+ */
+void wdt_start(void);
+
+/**
+ * @brief    Stop watchdog timer
+ */
+void wdt_stop(void);
+
+/**
+ * @brief    Reset the watchdog timer counter, delay system reset
+ */
+void wdt_kick(void);

I think that analogy is needlessly cruel. Instead of perpetually beating the watchdog unconscious I think we should rather `wdt_feed` it and have it reset the system when it starves.

We could also `wdt_pet` it, but I think the behavior when stopped being petted is not universal among all watchdogs.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11527#pullrequestreview-237811042
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190515/ecd93a5e/attachment.html>


More information about the notifications mailing list