[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a diff in pull request #6320: use rmutex inside of all repeated implementation

[GitBox <gi...@apache.org>]

xiaoxiang781216 commented on code in PR #6320:
URL: https://github.com/apache/incubator-nuttx/pull/6320#discussion_r882628608


##########
include/nuttx/mutex.h:
##########
@@ -214,7 +224,7 @@ static inline bool nxmutex_is_locked(FAR mutex_t *mutex)
 
 static inline int nxmutex_unlock(FAR mutex_t *mutex)
 {
-  return nxsem_post(mutex);
+  return _SEM_POST(mutex);

Review Comment:
   @pkarashchenko nxmutex is designed for kernel only, the userspace should use pthread_mutex_t instead, so we don't need call _SEM_POST/_SEM_WAIT in this file, I think.



-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org