Merge pull request 'perf(P4): Zeichenpuffer für flüssigen Bildlauf (1.6.25)' (#53) from perf/zeichenpuffer-bildlauf into main

P4_Display_Firmware/Changelog.txt

@@ -1,3 +1,14 @@
+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
+  ganze Seite sechs Durchgaenge, jeder mit eigener Uebertragung in den Bildspeicher.
+- Der Zwischenpuffer hat jetzt volle Bildhoehe - ein Durchgang statt sechs. Dazu kommt ein
+  zweiter Puffer, damit LVGL bereits den naechsten Bereich zeichnen kann, waehrend der
+  vorige noch uebertragen wird.
+- Kosten: 2 x 2,8 MB PSRAM zusaetzlich, bei 32 MB unkritisch. Am Startverhalten aendert
+  sich nichts: Diese Puffer sind Zeichenpuffer, keine Bildspeicher der Anzeige - es bleibt
+  bei EINEM Bildspeicher, und nur der war fuer den Startfehler verantwortlich.
+
 Version 1.6.24:
 - Ursache des unzuverlaessigen Starts gefunden und behoben: die Anzahl der Bildspeicher.
   Mit zwei Bildspeichern kam nach dem Einschalten kein Bild - der Hochlauf meldete

P4_Display_Firmware/lvgl_port_v9.c

@@ -492,9 +492,15 @@ 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 */
 

P4_Display_Firmware/pins_config.h

@@ -60,10 +60,15 @@
 // kurz eine Kante sichtbar werden - ein zuverlaessig startendes Display wiegt das auf.
 #if (JC_PANEL_TYPE == WS_PANEL_7H) && WS7_SINGLE_FB
 #define EXAMPLE_LVGL_PORT_AVOID_TEAR_ENABLE   0
-// Zwischenpuffer fuer das Zeichnen: 120 Zeilen im PSRAM (rund 460 KB). Der interne
-// Speicher ist zu knapp, dort passt er nicht.
+// 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.
 #define CONFIG_EXAMPLE_LVGL_PORT_BUF_PSRAM    1
-#define CONFIG_EXAMPLE_LVGL_PORT_BUF_HEIGHT   120
+#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