[riot-notifications] [RIOT-OS/RIOT] drivers/include: add header definition for wdt (#11527)
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);
+ * @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:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications