acassis commented on code in PR #16481: URL: https://github.com/apache/nuttx/pull/16481#discussion_r2132255083
########## include/nuttx/wdog.h: ########## @@ -320,8 +314,19 @@ static inline int wd_start_realtime(FAR struct wdog_s *wdog, * ****************************************************************************/ -int wd_start_period(FAR struct wdog_period_s *wdog, clock_t delay, - clock_t period, wdentry_t wdentry, wdparm_t arg); +static inline_function +int wd_start_next(FAR struct wdog_s *wdog, clock_t delay, Review Comment: @Fix-Point I think the previous function wd_start_period() was very intuitive to use, wd_start_next() although using the arguments it not so intuitive to use, also you comment: "can be used to implement a periodic watchdog timer", but you don't explain how? Please include a documentation and/or an usage example. We need to improve NuttX in relation to documentation and usage examples. Please help us to achieve this goal. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
