From 7d5cb8385e49285bfd8359b83f07cdb5104aba1b Mon Sep 17 00:00:00 2001 From: Gabor Kiss-Vamosi Date: Wed, 23 Oct 2024 21:21:24 +0200 Subject: [PATCH] Revert "fix(freertos): use xSemaphoreTakeRecursive (#6803)" It causes performance issues in some cases See https://github.com/lvgl/lvgl/pull/6981#issuecomment-2417037919 This reverts commit 53c846f4acee1f5a2fc8532799c279238f2df230. --- src/osal/lv_freertos.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/osal/lv_freertos.c b/src/osal/lv_freertos.c index b5fd17d3d..f65a299aa 100644 --- a/src/osal/lv_freertos.c +++ b/src/osal/lv_freertos.c @@ -129,7 +129,7 @@ lv_result_t lv_mutex_lock(lv_mutex_t * pxMutex) /* If mutex in uninitialized, perform initialization. */ prvCheckMutexInit(pxMutex); - BaseType_t xMutexTakeStatus = xSemaphoreTakeRecursive(pxMutex->xMutex, portMAX_DELAY); + BaseType_t xMutexTakeStatus = xSemaphoreTake(pxMutex->xMutex, portMAX_DELAY); if(xMutexTakeStatus != pdTRUE) { LV_LOG_ERROR("xSemaphoreTake failed!"); return LV_RESULT_INVALID; @@ -165,7 +165,7 @@ lv_result_t lv_mutex_unlock(lv_mutex_t * pxMutex) /* If mutex in uninitialized, perform initialization. */ prvCheckMutexInit(pxMutex); - BaseType_t xMutexGiveStatus = xSemaphoreGiveRecursive(pxMutex->xMutex); + BaseType_t xMutexGiveStatus = xSemaphoreGive(pxMutex->xMutex); if(xMutexGiveStatus != pdTRUE) { LV_LOG_ERROR("xSemaphoreGive failed!"); return LV_RESULT_INVALID;