pussuw commented on code in PR #16194: URL: https://github.com/apache/nuttx/pull/16194#discussion_r2058936809
########## sched/semaphore/sem_reset.c: ########## @@ -76,38 +81,63 @@ int nxsem_reset(FAR sem_t *sem, int16_t count) flags = enter_critical_section(); - /* A negative count indicates that the negated number of threads are - * waiting to take a count from the semaphore. Loop here, handing - * out counts to any waiting threads. - */ - - while (atomic_read(NXSEM_COUNT(sem)) < 0 && count > 0) + if (NXSEM_IS_MUTEX(sem)) Review Comment: A mutex does not have a count, what is the use case for nxmutex_reset ? -- 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
