pkarashchenko commented on PR #6376: URL: https://github.com/apache/incubator-nuttx/pull/6376#issuecomment-1148403153
> > One of the possible ways may be to get back original code for the files that I've listed in this PR and switching `nxmutex_` APIs to use `nxsem_` APIs only. > > How about we follow sem_t approach: > > * include/mutex.h which call sem_xxx and set errno > > * include/nuttx/mutex.h which call nxsem_xxx but not set errno > > * add MUTEX_XXXX macro in include/nuttx/mutex.h which call first or second version by check `__KERNEL__` Done, but IMO we should merge `mutex_t` and `rmutex_t` (as well as APIs) and just add `bool recursive;` flag to `mutex_t` and just keep one set of APIs -- 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
