diff --git a/list.c b/list.c index 5e8649a16..e6dbaaa3a 100644 --- a/list.c +++ b/list.c @@ -184,16 +184,14 @@ void vListInsert( List_t * const pxList, * 4) Using a queue or semaphore before it has been initialised or * before the scheduler has been started (are interrupts firing * before vTaskStartScheduler() has been called?). - * - This includes initializing binary semaphores before taking them. If - * you create one with `xSemaphoreCreateBinary()` or - * `xSemaphoreCreateBinaryStatic()`, you must call `xSemaphoreGive()` - * before calling `xSemaphoreTake(). See: - * https://freertos.org/xSemaphoreCreateBinaryStatic.html: - * > The semaphore is created in the 'empty' state, meaning the - * > semaphore must first be given using the xSemaphoreGive() API - * > function before it can subsequently be taken (obtained) using the - * > xSemaphoreTake() function. - * 5) If the FreeRTOS port supports interrupt nesting then ensure that + * 5) Attempting to 'take' binary semaphores created using + * `xSemaphoreCreateBinary()` or `xSemaphoreCreateBinaryStatic()` + * APIs, before 'giving' them. Binary semaphores created using + * `xSemaphoreCreateBinary()` or `xSemaphoreCreateBinaryStatic()`, + * are created in a state such that the semaphore must first be + * 'given' using xSemaphoreGive() API before it can be 'taken' using + * xSemaphoreTake() API. + * 6) If the FreeRTOS port supports interrupt nesting then ensure that * the priority of the tick interrupt is at or below * configMAX_SYSCALL_INTERRUPT_PRIORITY. **********************************************************************/ @@ -201,7 +199,7 @@ void vListInsert( List_t * const pxList, for( pxIterator = ( ListItem_t * ) &( pxList->xListEnd ); pxIterator->pxNext->xItemValue <= xValueOfInsertion; pxIterator = pxIterator->pxNext ) { /* There is nothing to do here, just iterating to the wanted - * insertion position. + * insertion position. * IF YOU FIND YOUR CODE STUCK HERE, SEE THE NOTE JUST ABOVE. */ }