xiaoxiang781216 commented on code in PR #18025: URL: https://github.com/apache/nuttx/pull/18025#discussion_r2706543256
########## include/nuttx/hrtimer.h: ########## @@ -61,7 +61,7 @@ struct hrtimer_s; * timer context and must not block. */ -typedef CODE uint64_t (*hrtimer_entry_t)(FAR const struct hrtimer_s *hrtimer, +typedef CODE uint64_t (*hrtimer_entry_t)(FAR struct hrtimer_s *hrtimer, Review Comment: but the deisgn forbid invoking hrtimer_cancel and hrtimer_start in the callback to avoid the race. You can achieve the same effect by: 1. return zero equals hrtimer_cancel 2. return non zero equal hrtimer_start -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
