fix(lcd): fix the infinite wait when lvgl timer is no ready

Merges https://github.com/espressif/esp-idf/pull/15853
This commit is contained in:
Chen Jichang
2025-04-25 18:32:58 +08:00
parent 7cef584834
commit 0daccd7f30
4 changed files with 29 additions and 31 deletions

View File

@ -1,11 +1,12 @@
/* /*
* SPDX-FileCopyrightText: 2021-2024 Espressif Systems (Shanghai) CO LTD * SPDX-FileCopyrightText: 2021-2025 Espressif Systems (Shanghai) CO LTD
* *
* SPDX-License-Identifier: CC0-1.0 * SPDX-License-Identifier: CC0-1.0
*/ */
#include <stdio.h> #include <stdio.h>
#include <sys/lock.h> #include <sys/lock.h>
#include <sys/param.h>
#include "freertos/FreeRTOS.h" #include "freertos/FreeRTOS.h"
#include "freertos/task.h" #include "freertos/task.h"
#include "freertos/semphr.h" #include "freertos/semphr.h"
@ -69,7 +70,7 @@ static const char *TAG = "example";
#define EXAMPLE_LVGL_TICK_PERIOD_MS 2 #define EXAMPLE_LVGL_TICK_PERIOD_MS 2
#define EXAMPLE_LVGL_TASK_MAX_DELAY_MS 500 #define EXAMPLE_LVGL_TASK_MAX_DELAY_MS 500
#define EXAMPLE_LVGL_TASK_MIN_DELAY_MS 1 #define EXAMPLE_LVGL_TASK_MIN_DELAY_MS 1000 / CONFIG_FREERTOS_HZ
#define EXAMPLE_LVGL_TASK_STACK_SIZE (4 * 1024) #define EXAMPLE_LVGL_TASK_STACK_SIZE (4 * 1024)
#define EXAMPLE_LVGL_TASK_PRIORITY 2 #define EXAMPLE_LVGL_TASK_PRIORITY 2
#define EXAMPLE_LVGL_DRAW_BUF_LINES 100 #define EXAMPLE_LVGL_DRAW_BUF_LINES 100
@ -116,11 +117,10 @@ static void example_lvgl_port_task(void *arg)
_lock_acquire(&lvgl_api_lock); _lock_acquire(&lvgl_api_lock);
time_till_next_ms = lv_timer_handler(); time_till_next_ms = lv_timer_handler();
_lock_release(&lvgl_api_lock); _lock_release(&lvgl_api_lock);
if (time_till_next_ms > EXAMPLE_LVGL_TASK_MAX_DELAY_MS) { // in case of triggering a task watch dog time out
time_till_next_ms = EXAMPLE_LVGL_TASK_MAX_DELAY_MS; time_till_next_ms = MAX(time_till_next_ms, EXAMPLE_LVGL_TASK_MIN_DELAY_MS);
} else if (time_till_next_ms < EXAMPLE_LVGL_TASK_MIN_DELAY_MS) { // in case of lvgl display not ready yet
time_till_next_ms = EXAMPLE_LVGL_TASK_MIN_DELAY_MS; time_till_next_ms = MIN(time_till_next_ms, EXAMPLE_LVGL_TASK_MAX_DELAY_MS);
}
vTaskDelay(pdMS_TO_TICKS(time_till_next_ms)); vTaskDelay(pdMS_TO_TICKS(time_till_next_ms));
} }
} }

View File

@ -1,5 +1,5 @@
/* /*
* SPDX-FileCopyrightText: 2023-2024 Espressif Systems (Shanghai) CO LTD * SPDX-FileCopyrightText: 2023-2025 Espressif Systems (Shanghai) CO LTD
* *
* SPDX-License-Identifier: CC0-1.0 * SPDX-License-Identifier: CC0-1.0
*/ */
@ -7,6 +7,7 @@
#include <stdio.h> #include <stdio.h>
#include <unistd.h> #include <unistd.h>
#include <sys/lock.h> #include <sys/lock.h>
#include <sys/param.h>
#include "sdkconfig.h" #include "sdkconfig.h"
#include "freertos/FreeRTOS.h" #include "freertos/FreeRTOS.h"
#include "freertos/task.h" #include "freertos/task.h"
@ -79,6 +80,8 @@ static const char *TAG = "example";
#define EXAMPLE_LVGL_TICK_PERIOD_MS 2 #define EXAMPLE_LVGL_TICK_PERIOD_MS 2
#define EXAMPLE_LVGL_TASK_STACK_SIZE (4 * 1024) #define EXAMPLE_LVGL_TASK_STACK_SIZE (4 * 1024)
#define EXAMPLE_LVGL_TASK_PRIORITY 2 #define EXAMPLE_LVGL_TASK_PRIORITY 2
#define EXAMPLE_LVGL_TASK_MAX_DELAY_MS 500
#define EXAMPLE_LVGL_TASK_MIN_DELAY_MS 1000 / CONFIG_FREERTOS_HZ
// LVGL library is not thread-safe, this example will call LVGL APIs from different tasks, so use a mutex to protect it // LVGL library is not thread-safe, this example will call LVGL APIs from different tasks, so use a mutex to protect it
static _lock_t lvgl_api_lock; static _lock_t lvgl_api_lock;
@ -110,11 +113,10 @@ static void example_lvgl_port_task(void *arg)
_lock_acquire(&lvgl_api_lock); _lock_acquire(&lvgl_api_lock);
time_till_next_ms = lv_timer_handler(); time_till_next_ms = lv_timer_handler();
_lock_release(&lvgl_api_lock); _lock_release(&lvgl_api_lock);
// in case of triggering a task watch dog time out
// in case of task watch dog timeout, set the minimal delay to 10ms time_till_next_ms = MAX(time_till_next_ms, EXAMPLE_LVGL_TASK_MIN_DELAY_MS);
if (time_till_next_ms < 10) { // in case of lvgl display not ready yet
time_till_next_ms = 10; time_till_next_ms = MIN(time_till_next_ms, EXAMPLE_LVGL_TASK_MAX_DELAY_MS);
}
usleep(1000 * time_till_next_ms); usleep(1000 * time_till_next_ms);
} }
} }

View File

@ -1,5 +1,5 @@
/* /*
* SPDX-FileCopyrightText: 2022-2024 Espressif Systems (Shanghai) CO LTD * SPDX-FileCopyrightText: 2022-2025 Espressif Systems (Shanghai) CO LTD
* *
* SPDX-License-Identifier: CC0-1.0 * SPDX-License-Identifier: CC0-1.0
*/ */
@ -7,6 +7,7 @@
#include <stdio.h> #include <stdio.h>
#include <unistd.h> #include <unistd.h>
#include <sys/lock.h> #include <sys/lock.h>
#include <sys/param.h>
#include "sdkconfig.h" #include "sdkconfig.h"
#include "freertos/FreeRTOS.h" #include "freertos/FreeRTOS.h"
#include "freertos/task.h" #include "freertos/task.h"
@ -95,6 +96,8 @@ static const char *TAG = "example";
#define EXAMPLE_LVGL_TICK_PERIOD_MS 2 #define EXAMPLE_LVGL_TICK_PERIOD_MS 2
#define EXAMPLE_LVGL_TASK_STACK_SIZE (5 * 1024) #define EXAMPLE_LVGL_TASK_STACK_SIZE (5 * 1024)
#define EXAMPLE_LVGL_TASK_PRIORITY 2 #define EXAMPLE_LVGL_TASK_PRIORITY 2
#define EXAMPLE_LVGL_TASK_MAX_DELAY_MS 500
#define EXAMPLE_LVGL_TASK_MIN_DELAY_MS 1000 / CONFIG_FREERTOS_HZ
// LVGL library is not thread-safe, this example will call LVGL APIs from different tasks, so use a mutex to protect it // LVGL library is not thread-safe, this example will call LVGL APIs from different tasks, so use a mutex to protect it
static _lock_t lvgl_api_lock; static _lock_t lvgl_api_lock;
@ -133,12 +136,10 @@ static void example_lvgl_port_task(void *arg)
_lock_acquire(&lvgl_api_lock); _lock_acquire(&lvgl_api_lock);
time_till_next_ms = lv_timer_handler(); time_till_next_ms = lv_timer_handler();
_lock_release(&lvgl_api_lock); _lock_release(&lvgl_api_lock);
// in case of task watch dog timeout
// in case of task watch dog timeout, set the minimal delay to 10ms time_till_next_ms = MAX(time_till_next_ms, EXAMPLE_LVGL_TASK_MIN_DELAY_MS);
if (time_till_next_ms < 10) { // in case of lvgl display not ready yet
time_till_next_ms = 10; time_till_next_ms = MIN(time_till_next_ms, EXAMPLE_LVGL_TASK_MAX_DELAY_MS);
}
usleep(1000 * time_till_next_ms); usleep(1000 * time_till_next_ms);
} }
} }

View File

@ -65,7 +65,7 @@ static const char *TAG = "example";
#define EXAMPLE_LVGL_DRAW_BUF_LINES 20 // number of display lines in each draw buffer #define EXAMPLE_LVGL_DRAW_BUF_LINES 20 // number of display lines in each draw buffer
#define EXAMPLE_LVGL_TICK_PERIOD_MS 2 #define EXAMPLE_LVGL_TICK_PERIOD_MS 2
#define EXAMPLE_LVGL_TASK_MAX_DELAY_MS 500 #define EXAMPLE_LVGL_TASK_MAX_DELAY_MS 500
#define EXAMPLE_LVGL_TASK_MIN_DELAY_MS 1 #define EXAMPLE_LVGL_TASK_MIN_DELAY_MS 1000 / CONFIG_FREERTOS_HZ
#define EXAMPLE_LVGL_TASK_STACK_SIZE (4 * 1024) #define EXAMPLE_LVGL_TASK_STACK_SIZE (4 * 1024)
#define EXAMPLE_LVGL_TASK_PRIORITY 2 #define EXAMPLE_LVGL_TASK_PRIORITY 2
@ -157,20 +157,15 @@ static void example_lvgl_port_task(void *arg)
{ {
ESP_LOGI(TAG, "Starting LVGL task"); ESP_LOGI(TAG, "Starting LVGL task");
uint32_t time_till_next_ms = 0; uint32_t time_till_next_ms = 0;
uint32_t time_threshold_ms = 1000 / CONFIG_FREERTOS_HZ;
while (1) { while (1) {
_lock_acquire(&lvgl_api_lock); _lock_acquire(&lvgl_api_lock);
time_till_next_ms = lv_timer_handler(); time_till_next_ms = lv_timer_handler();
_lock_release(&lvgl_api_lock); _lock_release(&lvgl_api_lock);
if ( time_till_next_ms == LV_NO_TIMER_READY ) { // in case of triggering a task watch dog time out
//most probably lvgl display not ready yet time_till_next_ms = MAX(time_till_next_ms, EXAMPLE_LVGL_TASK_MIN_DELAY_MS);
usleep( 1000 * 1000 ); // in case of lvgl display not ready yet
} time_till_next_ms = MIN(time_till_next_ms, EXAMPLE_LVGL_TASK_MAX_DELAY_MS);
else { usleep(1000 * time_till_next_ms);
// in case of triggering a task watch dog time out
time_till_next_ms = MAX(time_till_next_ms, time_threshold_ms);
usleep(1000 * time_till_next_ms);
}
} }
} }