Merge pull request #524 from espressif/fix/lvgl_draw_buf_alignment

fix(esp_lvgl_port): Fix draw buffer alignment

Author: tore-espressif

bsp/m5dial/idf_component.yml

@@ -16,6 +16,7 @@ dependencies:
   espressif/esp_lvgl_port:
     version: "^2"
     public: true
+    override_path: "../../components/esp_lvgl_port"
 
   button:
     public: true

bsp/m5stack_core_s3/idf_component.yml

@@ -16,6 +16,7 @@ dependencies:
   espressif/esp_lvgl_port:
     version: "^2"
     public: true
+    override_path: "../../components/esp_lvgl_port"
 
   esp_codec_dev:
     version: "~1.3.1"

components/esp_lvgl_port/CHANGELOG.md

@@ -8,6 +8,7 @@
 ### Fixes
 - Fixed buffer size by selected color format
 - Fixed memory leak in LVGL8 in display removing https://github.com/espressif/esp-bsp/issues/462
+- Fixed draw buffer alignment
 
 ## 2.4.4
 

components/esp_lvgl_port/include/esp_lvgl_port.h

@@ -62,7 +62,7 @@ typedef struct {
 #define ESP_LVGL_PORT_INIT_CONFIG() \
     {                               \
         .task_priority = 4,       \
-        .task_stack = 6144,       \
+        .task_stack = 7168,       \
         .task_affinity = -1,      \
         .task_max_sleep_ms = 500, \
         .timer_period_ms = 5,     \

components/esp_lvgl_port/src/lvgl9/esp_lvgl_port_disp.c

@@ -1,10 +1,11 @@
 /*
- * SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2024-2025 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
 
 #include <string.h>
+#include "sdkconfig.h"
 #include "esp_log.h"
 #include "esp_err.h"
 #include "esp_check.h"
@@ -32,6 +33,10 @@
 #define LVGL_PORT_HANDLE_FLUSH_READY 1
 #endif
 
+#ifndef CONFIG_LV_DRAW_BUF_ALIGN
+#define CONFIG_LV_DRAW_BUF_ALIGN 1
+#endif
+
 static const char *TAG = "LVGL";
 
 /*******************************************************************************
@@ -308,10 +313,10 @@ static lv_display_t *lvgl_port_add_disp_priv(const lvgl_port_display_cfg_t *disp
     } else {
         /* alloc draw buffers used by LVGL */
         /* it's recommended to choose the size of the draw buffer(s) to be at least 1/10 screen sized */
-        buf1 = heap_caps_malloc(buffer_size * color_bytes, buff_caps);
+        buf1 = heap_caps_aligned_alloc(CONFIG_LV_DRAW_BUF_ALIGN, buffer_size * color_bytes, buff_caps);
         ESP_GOTO_ON_FALSE(buf1, ESP_ERR_NO_MEM, err, TAG, "Not enough memory for LVGL buffer (buf1) allocation!");
         if (disp_cfg->double_buffer) {
-            buf2 = heap_caps_malloc(buffer_size * color_bytes, buff_caps);
+            buf2 = heap_caps_aligned_alloc(CONFIG_LV_DRAW_BUF_ALIGN, buffer_size * color_bytes, buff_caps);
             ESP_GOTO_ON_FALSE(buf2, ESP_ERR_NO_MEM, err, TAG, "Not enough memory for LVGL buffer (buf2) allocation!");
         }
 

examples/display_lvgl_benchmark/sdkconfig.bsp.esp-box

@@ -31,6 +31,7 @@ CONFIG_LV_MEMCPY_MEMSET_STD=y
 
 ## LVGL9 ##
 CONFIG_LV_CONF_SKIP=y
+CONFIG_LV_DEF_REFR_PERIOD=10
 
 #CLIB default
 CONFIG_LV_USE_CLIB_MALLOC=y

examples/display_lvgl_benchmark/sdkconfig.bsp.esp-box-3

@@ -32,6 +32,7 @@ CONFIG_LV_MEMCPY_MEMSET_STD=y
 
 ## LVGL9 ##
 CONFIG_LV_CONF_SKIP=y
+CONFIG_LV_DEF_REFR_PERIOD=10
 
 #CLIB default
 CONFIG_LV_USE_CLIB_MALLOC=y

examples/display_lvgl_benchmark/sdkconfig.bsp.esp-box-lite

@@ -31,6 +31,7 @@ CONFIG_LV_MEMCPY_MEMSET_STD=y
 
 ## LVGL9 ##
 CONFIG_LV_CONF_SKIP=y
+CONFIG_LV_DEF_REFR_PERIOD=10
 
 #CLIB default
 CONFIG_LV_USE_CLIB_MALLOC=y

examples/display_lvgl_benchmark/sdkconfig.bsp.esp32_p4_function_ev_board

@@ -39,6 +39,7 @@ CONFIG_LV_USE_PERF_MONITOR=y
 
 ## LVGL9 ##
 CONFIG_LV_CONF_SKIP=y
+CONFIG_LV_DEF_REFR_PERIOD=10
 
 #CLIB default
 CONFIG_LV_USE_CLIB_MALLOC=y

examples/display_lvgl_benchmark/sdkconfig.bsp.esp32_s2_kaluga_kit

@@ -31,6 +31,7 @@ CONFIG_LV_MEMCPY_MEMSET_STD=y
 
 ## LVGL9 ##
 CONFIG_LV_CONF_SKIP=y
+CONFIG_LV_DEF_REFR_PERIOD=10
 
 #CLIB default
 CONFIG_LV_USE_CLIB_MALLOC=y