From a3461fe960dfcdfaec955ee518ccdb306baa9346 Mon Sep 17 00:00:00 2001 From: raw-designs Date: Wed, 2 Sep 2026 03:47:59 +0200 Subject: [PATCH] fix(P4): Blaue Zwischenbilder beim Bildlauf beseitigt (1.6.26) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Ursache war der zweite Zeichenpuffer aus 1.6.25. esp_lcd_dpi_panel_draw_bitmap lehnt eine Übertragung ab, solange die vorige noch läuft: ESP_RETURN_ON_FALSE(xSemaphoreTake(dpi_panel->draw_sem, 0) == pdTRUE, ESP_ERR_INVALID_STATE, ...) Mit zwei Puffern schickt LVGL die nächste Übertragung sofort los, statt auf lv_disp_flush_ready zu warten. Die abgelehnte Übertragung fällt ersatzlos aus, und der betroffene Bildbereich behält seinen alten oder noch nicht geschriebenen Inhalt - sichtbar als blaue Zwischenbilder. Der zweite Zeichenpuffer ist entfernt, lvgl_port_v9.c ist damit wieder unverändert gegenüber der Hersteller-Portierung. Der eigentliche Gewinn bleibt: Der Zwischenpuffer hat weiterhin volle Bildhöhe, ein Bildlauf braucht einen Durchgang statt sechs. Co-Authored-By: Claude Opus 5 --- P4_Display_Firmware/Changelog.txt | 11 +++++++++++ P4_Display_Firmware/lvgl_port_v9.c | 10 ++-------- P4_Display_Firmware/pins_config.h | 12 +++++++----- 3 files changed, 20 insertions(+), 13 deletions(-) diff --git a/P4_Display_Firmware/Changelog.txt b/P4_Display_Firmware/Changelog.txt index f153186..c18e574 100644 --- a/P4_Display_Firmware/Changelog.txt +++ b/P4_Display_Firmware/Changelog.txt @@ -1,3 +1,14 @@ +Version 1.6.26: +- Fehlerbehebung: Beim Bildlauf blitzten blaue Zwischenbilder auf. Ursache war der zweite + Zeichenpuffer aus 1.6.25. Der Panel-Treiber lehnt eine Uebertragung ab, solange die + vorige noch laeuft; mit zwei Puffern schickt LVGL die naechste aber sofort los. Die + abgelehnte Uebertragung faellt ersatzlos aus, und der betroffene Bildbereich behaelt + seinen alten Inhalt. +- Der zweite Zeichenpuffer ist deshalb wieder entfernt, die Herstellerdatei lvgl_port_v9.c + damit wieder unveraendert. Der eigentliche Gewinn beim Bildlauf bleibt erhalten: Der + Zwischenpuffer hat weiterhin volle Bildhoehe, ein Bildlauf braucht also einen Durchgang + statt sechs. + Version 1.6.25: - Der Bildlauf war nach der Umstellung auf einen Bildspeicher sehr zaeh. Grund war der Zwischenpuffer, in den LVGL zeichnet: Mit 120 Zeilen brauchte ein Bildlauf ueber die diff --git a/P4_Display_Firmware/lvgl_port_v9.c b/P4_Display_Firmware/lvgl_port_v9.c index d185141..962d373 100644 --- a/P4_Display_Firmware/lvgl_port_v9.c +++ b/P4_Display_Firmware/lvgl_port_v9.c @@ -492,15 +492,9 @@ static lv_display_t *display_init(esp_lcd_panel_handle_t panel_handle) // Normmaly, for RGB LCD, we just use one buffer for LVGL rendering buffer_size = LVGL_PORT_H_RES * LVGL_PORT_BUFFER_HEIGHT; buf1 = heap_caps_malloc(buffer_size * sizeof(lv_color_t), LVGL_PORT_BUFFER_MALLOC_CAPS); + // buffer_size = LVGL_PORT_H_RES * LVGL_PORT_BUFFER_HEIGHT; + // buf1 = heap_caps_malloc(buffer_size * sizeof(lv_color_t), MALLOC_CAP_DMA); assert(buf1); -#ifdef LVGL_PORT_SECOND_DRAW_BUFFER - // PROJEKTAENDERUNG gegenueber der Hersteller-Portierung: ein zweiter Zeichenpuffer. - // Mit nur einem Puffer steht das Zeichnen still, solange der fertige Inhalt in den - // Bildspeicher uebertragen wird. Mit zwei Puffern zeichnet LVGL bereits den naechsten - // Bereich, waehrend der vorige noch uebertragen wird - beim Bildlauf deutlich spuerbar. - buf2 = heap_caps_malloc(buffer_size * sizeof(lv_color_t), LVGL_PORT_BUFFER_MALLOC_CAPS); - assert(buf2); -#endif ESP_LOGI(TAG, "LVGL buffer size: %dKB", buffer_size * sizeof(lv_color_t) / 1024); #endif /* LVGL_PORT_AVOID_TEAR_ENABLE */ diff --git a/P4_Display_Firmware/pins_config.h b/P4_Display_Firmware/pins_config.h index 5d877c1..33cae88 100644 --- a/P4_Display_Firmware/pins_config.h +++ b/P4_Display_Firmware/pins_config.h @@ -62,13 +62,15 @@ #define EXAMPLE_LVGL_PORT_AVOID_TEAR_ENABLE 0 // Zwischenpuffer fuer das Zeichnen. Volle Bildhoehe, damit ein Bildlauf in EINEM Durchgang // gezeichnet und uebertragen wird. Mit 120 Zeilen waren es sechs Durchgaenge samt sechs -// Uebertragungen - das war der Grund fuer den zaehen Bildlauf. -// Dazu ein zweiter Puffer: LVGL zeichnet damit schon den naechsten Bereich, waehrend der -// vorige noch in den Bildspeicher uebertragen wird. -// Kosten: 2 x 2,8 MB PSRAM. Bei 32 MB unkritisch. Der interne Speicher waere zu knapp. +// Uebertragungen - das war der Grund fuer den zaehen Bildlauf. Kostet 2,8 MB PSRAM. +// +// Bewusst nur EIN Zeichenpuffer: Ein zweiter wuerde LVGL erlauben, schon den naechsten +// Bereich zu zeichnen, waehrend der vorige uebertragen wird. Der Panel-Treiber lehnt eine +// Uebertragung aber ab, solange die vorige laeuft (ESP_ERR_INVALID_STATE) - der betroffene +// Bildbereich wird dann gar nicht geschrieben und behaelt seinen alten Inhalt. Sichtbar +// war das als blaue Zwischenbilder beim Bildlauf. #define CONFIG_EXAMPLE_LVGL_PORT_BUF_PSRAM 1 #define CONFIG_EXAMPLE_LVGL_PORT_BUF_HEIGHT LCD_V_RES -#define LVGL_PORT_SECOND_DRAW_BUFFER 1 #else #define EXAMPLE_LVGL_PORT_AVOID_TEAR_ENABLE 1 #endif -- 2.54.0