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

benpicco notifications at github.com
Thu May 16 18:07:14 CEST 2019


benpicco commented on this pull request.



> +void wdt_setup_reboot(uint32_t min_time, uint32_t max_time);
+
+#ifdef MODULE_PERIPH_WDT_CB
+/**
+ * @brief    Set up the wdt timer, only use max_time if normal operation
+ *           set min_time and max_time for windowed timer.
+ *
+ * @param[in] min_time       lower bound for windowed watchdog in ms, has to be 0
+ *                           for normal mode operation
+ * @param[in] max_time       upper bound for windowed watchdog in ms, time before
+ *                           reset for normal watchdog
+ * @param[in] wdt_cb         wdt callback, can be NULL
+ * @param[in] arg            optional argument which is passed to the
+ *                           callback, can be NULL
+ */
+void wdt_setup_callback(uint32_t min_time, uint32_t max_time, wdt_cb_t wdt_cb, void* arg);

I'm fine with `wdt_setup_reset_with_warning()` if that means that the compiler can LTO away those pesky 8 bytes if not used. 

Log messages on the other hand tend to be larger than 8 byte (yea, ROM hurts a little less than RAM, but still), but I guess we are good when we enable them when `DEVELHELP` is set.

-- 
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#discussion_r284785247
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190516/9ebc1c0a/attachment.html>


More information about the notifications mailing list