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

Kaspar Schleiser notifications at github.com
Wed May 15 15:05:41 CEST 2019


kaspar030 requested changes on this pull request.



> + */
+void wdt_stop(void);
+
+/**
+ * @brief    Reset the watchdog timer counter, delay system reset
+ */
+void wdt_kick(void);
+
+/**
+ * @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:
https://github.com/RIOT-OS/RIOT/pull/11527#pullrequestreview-237806376
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190515/e581f970/attachment.html>


More information about the notifications mailing list