Update osal_freertos.h

jbruneaux31 · web-flow · commit a46ad8fcdfef · 2022-12-19T13:58:54.000+01:00
Fix FreeRTOS task switch even if not required (unitialized variable usage)
diff --git a/src/osal/osal_freertos.h b/src/osal/osal_freertos.h
@@ -115,7 +115,7 @@ TU_ATTR_ALWAYS_INLINE static inline bool osal_semaphore_post(osal_semaphore_t se
   }
   else
   {
-    BaseType_t xHigherPriorityTaskWoken;
+    BaseType_t xHigherPriorityTaskWoken = pdFALSE;
     BaseType_t res = xSemaphoreGiveFromISR(sem_hdl, &xHigherPriorityTaskWoken);
 
 #if CFG_TUSB_MCU == OPT_MCU_ESP32S2 || CFG_TUSB_MCU == OPT_MCU_ESP32S3
@@ -189,7 +189,7 @@ TU_ATTR_ALWAYS_INLINE static inline bool osal_queue_send(osal_queue_t qhdl, void
   }
   else
   {
-    BaseType_t xHigherPriorityTaskWoken;
+    BaseType_t xHigherPriorityTaskWoken = pdFALSE;
     BaseType_t res = xQueueSendToBackFromISR(qhdl, data, &xHigherPriorityTaskWoken);
 
 #if CFG_TUSB_MCU == OPT_MCU_ESP32S2 || CFG_TUSB_MCU == OPT_MCU_ESP32S3

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ TU_ATTR_ALWAYS_INLINE static inline bool osal_semaphore_post(osal_semaphore_t se`
`115`	`115`	`}`
`116`	`116`	`else`
`117`	`117`	`{`
`118`		`- BaseType_t xHigherPriorityTaskWoken;`
	`118`	`+ BaseType_t xHigherPriorityTaskWoken = pdFALSE;`
`119`	`119`	`BaseType_t res = xSemaphoreGiveFromISR(sem_hdl, &xHigherPriorityTaskWoken);`
`120`	`120`
`121`	`121`	`#if CFG_TUSB_MCU == OPT_MCU_ESP32S2 \|\| CFG_TUSB_MCU == OPT_MCU_ESP32S3`
`@@ -189,7 +189,7 @@ TU_ATTR_ALWAYS_INLINE static inline bool osal_queue_send(osal_queue_t qhdl, void`
`189`	`189`	`}`
`190`	`190`	`else`
`191`	`191`	`{`
`192`		`- BaseType_t xHigherPriorityTaskWoken;`
	`192`	`+ BaseType_t xHigherPriorityTaskWoken = pdFALSE;`
`193`	`193`	`BaseType_t res = xQueueSendToBackFromISR(qhdl, data, &xHigherPriorityTaskWoken);`
`194`	`194`
`195`	`195`	`#if CFG_TUSB_MCU == OPT_MCU_ESP32S2 \|\| CFG_TUSB_MCU == OPT_MCU_ESP32S3`