[riot-notifications] [RIOT-OS/RIOT] drivers/include: add header definition for wdt (#11527)
notifications at github.com
Wed May 15 15:05:41 CEST 2019
kaspar030 requested changes on this pull request.
> + */
+ * @brief Reset the watchdog timer counter, delay system reset
+ * @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 us
+ * @param[in] max_time upper bound for windowed watchdog in us, time before
+ * reset for normal watchdog
+ * @param[in] wdt_cb wdt callback
We could default to reboot, of no callback is given.
It might make sense to split this into to:
void wdt_setup_reboot(uint32_t min_time, uint32_t max_time);
void wdt_setup_callback(uint32_t min_time, uint32_t max_time, wdt_cb_t wdt_cb, void* arg);
1. the latter can be guarded by a feature (e.g., periph_wdt_callback), should it not be available or desired
2. even if available but not used, the actual code will always have to reserve the 8 bytes of memory.
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