perf(P4): Scrollen ohne Nachlauf am Waveshare-Panel (1.6.27)

Ohne Doppelpufferung entsteht beim Scrollen jedes Bild komplett neu
(1280x676 in 24 Bit, per Software). Am stärksten fällt das beim Ausrollen
nach dem Loslassen auf: Dort bewegt sich das Bild weiter, ohne dass ein
Finger es führt, und jedes Stocken ist unmittelbar zu sehen.

LV_OBJ_FLAG_SCROLL_MOMENTUM ist für die scrollbaren Container deshalb
abgeschaltet - die Liste folgt nur noch dem Finger. Nur für WS_PANEL_7H;
die JC-Panels haben das Tempoproblem nicht und bleiben unverändert.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

P4_Display_Firmware/Changelog.txt

@@ -1,3 +1,63 @@
+Version 1.6.27:
+- Beim Waveshare-Panel rollen scrollbare Listen nach dem Loslassen nicht mehr aus, sondern
+  folgen nur dem Finger. Ohne Doppelpufferung entsteht beim Scrollen jedes Bild komplett
+  neu; am staerksten faellt das beim Ausrollen auf, wo sich das Bild ohne fuehrenden Finger
+  weiterbewegt und jedes Stocken unmittelbar sichtbar ist.
+- Die JC-Panels sind ausgenommen, ihr Bediengefuehl bleibt unveraendert.
+
+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
+  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
+  durchgehend Erfolg, das Panel blieb dunkel, und erst ein Reset half. Mit genau einem
+  startet es zuverlaessig. Belegt wurde das mit den Prueffarbbalken: Sie erscheinen mit
+  einem Speicher sofort, mit zweien nicht. Der Werks-Testsketch benutzt ebenfalls einen -
+  das war der letzte verbliebene Unterschied zu ihm.
+- Die Oberflaeche laeuft deshalb ohne Doppelpufferung: LVGL zeichnet in einen
+  Zwischenpuffer von 120 Zeilen im PSRAM, der dann in den einen Bildspeicher uebertragen
+  wird. Bei schnellen Bildwechseln kann kurz eine Kante sichtbar werden; ein zuverlaessig
+  startendes Display wiegt das auf.
+- Schalter WS7_SINGLE_FB in pins_config.h. WS7_PARTIAL_REFRESH ist damit wirkungslos, weil
+  es ohne Doppelpufferung nichts umzuschalten gibt.
+- Der PSRAM-Bedarf sinkt von 5,4 MB auf 2,8 MB.
+- Die Umgehungen aus den Vorversionen bleiben erhalten: Anlaufzeit, wiederholte
+  Bridge-Grundeinstellung, wiederholte Weckbefehle und der Neustart nach Kaltstart. Ob
+  jede davon noch noetig ist, laesst sich jetzt einzeln pruefen - der Startfehler selbst
+  ist behoben.
+
+Version 1.6.23:
+- Der Selbst-Neustart aus 1.6.22 reicht nicht: Ein Software-Neustart setzt offenbar nicht
+  dasselbe zurueck wie die Reset-Taste. Der Schalter bleibt eingeschaltet, er schadet nicht,
+  loest das Problem aber nicht allein.
+- Zurueck zu dem einen Unterschied zum funktionierenden Werks-Testsketch, der bisher nie
+  ueberprueft wurde: Dieser legt EINEN Bildspeicher an, die Firmware zwei. Der Pruefbild-Test
+  lief bislang ebenfalls mit zwei und war damit nie ein echter Nachbau des Testsketches - er
+  wich an genau der Stelle ab, die noch offen ist.
+- Mit WS7_TEST_PATTERN = 1 wird jetzt genau ein Bildspeicher angelegt, wie im Testsketch.
+  Erscheinen die Farbbalken damit zuverlaessig, liegt es an der Anzahl der Bildspeicher, und
+  die Oberflaeche laesst sich darauf umstellen.
+
 Version 1.6.22:
 - Der Ab- und Wiederaufbau der DSI-Strecke aus 1.6.21 hat nicht geholfen, sondern das Bild
   um einen Schritt verschlechtert: Nach dem Einschalten kam gar nichts mehr, nach dem

P4_Display_Firmware/README.md

@@ -50,7 +50,7 @@ Vendor-Pakete: 4,3" im Ordner `JC4880P443C_I_W/`, 7,0" im Repo-Ordner
 | Touch-Rohkoordinaten | 480×800 (= Panelraster) | **800×480** → wird auf 1024×600 skaliert | **720×1280** (Hochformat) → X/Y werden getauscht |
 | LCD-Reset | GPIO5 | **GPIO27** | kein Reset-Pin (macht die Bridge) |
 | Backlight | GPIO23 (LEDC-PWM) | GPIO23 (LEDC-PWM) | **Bridge-Register** auf I²C **0x45** (`AD`/`AB`/`AA`) |
-| Framebuffer | 3 × 750 KB ≈ 2,3 MB PSRAM | 3 × 1,2 MB ≈ 3,6 MB PSRAM | 3 × 2,8 MB ≈ 8,3 MB PSRAM |
+| Framebuffer | 3 × 750 KB ≈ 2,3 MB PSRAM | 3 × 1,2 MB ≈ 3,6 MB PSRAM | **1 × 2,8 MB** — mehr verträgt das Panel beim Start nicht |
 | Freie Expansion-IO | Stiftleiste **JP1**: GPIO 52/51/50/49/35/34/33/32/31/30/29/28 | FPC 1,0 mm 16-pol. **FPC4**: GPIO 34/33/32/31/30/29/28 · **FPC3**: 48/47/46/45/5/4/3/2 | 40-poliger Header im Raspberry-Pi-Pico-Format; belegt sind I²C (7/8), TF-Karte (9/10/11/12/13/53) und Debug-UART0 (37/38) |
 | UART zur Hauptplatine | TX GPIO33 / RX GPIO31 | TX GPIO33 / RX GPIO31 | **TX GPIO20 / RX GPIO21** |
 

P4_Display_Firmware/board_bringup.c

@@ -458,6 +458,8 @@ static bool ws7_dsi_bringup(esp_lcd_dsi_bus_handle_t *out_bus,
 #else
         .pixel_format = LCD_COLOR_PIXEL_FORMAT_RGB565,
 #endif
+        // Anzahl der Bildspeicher. Beim Waveshare-Panel genau einer: mit zweien kam nach
+        // dem Einschalten kein Bild (siehe WS7_SINGLE_FB in pins_config.h).
         .num_fbs = LVGL_PORT_LCD_BUFFER_NUMS,
         .video_timing = {
             .h_size = BSP_LCD_H_RES,

P4_Display_Firmware/pins_config.h

@@ -32,7 +32,13 @@
 //                           startet der P4 nicht mehr, hier zuerst auf 0 stellen.
 // -------------------------------------------------------------------------------------
 #define WS7_COLOR_BITS        24
+// Ohne Wirkung, solange WS7_SINGLE_FB = 1 ist: Mit nur einem Bildspeicher gibt es keine
+// Doppelpufferung, zwischen deren Betriebsarten man waehlen koennte.
 #define WS7_PARTIAL_REFRESH    1
+// Nur EIN Bildspeicher. Zwei vertraegt die DSI-Ausgabe dieses Panels beim Einschalten
+// nicht - siehe die ausfuehrliche Begruendung weiter unten beim Avoid-Tear-Schalter.
+// Auf 0 nur, wenn das Startverhalten erneut untersucht wird.
+#define WS7_SINGLE_FB          1
 #define WS7_PARALLEL_RENDER    1
 
 #define EXAMPLE_LVGL_PORT_TASK_MAX_DELAY_MS 500  //range 2 to 2000
@@ -42,7 +48,32 @@
 #define EXAMPLE_LVGL_PORT_TASK_CORE         -1  //range -1 to 1
 #define EXAMPLE_LVGL_PORT_TICK              2   //ragne 1 to 100
 
+// Ruckelfreie Ausgabe braucht mindestens zwei Bildspeicher, aus denen abwechselnd
+// angezeigt und in die gezeichnet wird.
+//
+// Das Waveshare-Panel vertraegt das nicht: Mit zwei Bildspeichern kam nach dem Einschalten
+// kein Bild - der Hochlauf meldete durchgehend Erfolg, das Panel blieb dunkel, und erst ein
+// Reset half. Mit genau einem Bildspeicher startet es zuverlaessig; belegt wurde das mit den
+// Prueffarbbalken, die der Werks-Testsketch ebenfalls mit einem Speicher anzeigt.
+// Deshalb hier ohne Doppelpufferung: LVGL zeichnet in einen kleinen Zwischenpuffer, der
+// dann in den einen Bildspeicher uebertragen wird. Bei schnellen Bildwechseln kann dabei
+// 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. 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. 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
+#else
 #define EXAMPLE_LVGL_PORT_AVOID_TEAR_ENABLE   1
+#endif
 
 #ifdef  EXAMPLE_LVGL_PORT_AVOID_TEAR_ENABLE
 // 2 = Triple-Buffer + LVGL Full-Refresh: zeichnet bei jeder Aenderung das komplette

P4_Display_Firmware/ui.cpp

@@ -449,6 +449,24 @@ static void add_save_button(lv_obj_t* parent, const char* txt, lv_event_cb_t cb)
     lv_obj_add_event_cb(b, cb, LV_EVENT_CLICKED, nullptr);
 }
 // Eine Seite (Vollflaeche im Content-Bereich, scrollbar, Spalten-Layout)
+// Nachlauf beim Scrollen abschalten.
+//
+// Ohne Doppelpufferung entsteht beim Scrollen jedes Bild komplett neu - 1280x676 Pixel in
+// 24 Bit, per Software. Am staerksten faellt das beim Ausrollen nach dem Loslassen auf:
+// Dort bewegt sich das Bild weiter, ohne dass ein Finger es fuehrt, und jedes Stocken ist
+// unmittelbar zu sehen. Folgt die Liste dagegen nur dem Finger, entstehen weniger Bilder,
+// und die verbleibenden fallen kaum auf.
+//
+// Nur fuer das Waveshare-Panel: Die JC-Panels haben das Tempoproblem nicht, ihr
+// Bediengefuehl soll unveraendert bleiben.
+static void scroll_ohne_nachlauf(lv_obj_t* o) {
+#if JC_PANEL_TYPE == WS_PANEL_7H
+    lv_obj_clear_flag(o, LV_OBJ_FLAG_SCROLL_MOMENTUM);
+#else
+    (void)o;
+#endif
+}
+
 static lv_obj_t* make_page(lv_obj_t* parent) {
     lv_obj_t* p = lv_obj_create(parent);
     lv_obj_set_size(p, LV_PCT(100), LV_PCT(100));
@@ -462,6 +480,7 @@ static lv_obj_t* make_page(lv_obj_t* parent) {
     lv_obj_set_style_pad_all(p, 8, 0);
     lv_obj_set_style_pad_row(p, 6, 0);
     lv_obj_set_flex_flow(p, LV_FLEX_FLOW_COLUMN);
+    scroll_ohne_nachlauf(p);
     return p;
 }
 
@@ -2304,6 +2323,7 @@ static void build_cleaning_screen() {
     lv_obj_set_style_pad_all(g_clnCfg, 0, 0); lv_obj_set_style_pad_row(g_clnCfg, 8, 0);
     lv_obj_set_flex_flow(g_clnCfg, LV_FLEX_FLOW_COLUMN);
     lv_obj_set_scroll_dir(g_clnCfg, LV_DIR_VER);
+    scroll_ohne_nachlauf(g_clnCfg);
     lv_obj_set_scrollbar_mode(g_clnCfg, LV_SCROLLBAR_MODE_AUTO);
 
     lv_obj_t* hint = lv_label_create(g_clnCfg);
@@ -3289,6 +3309,7 @@ void ui_init(ProtocolClient* client) {
     lv_obj_set_style_pad_row(menu, 6, 0);
     lv_obj_set_flex_flow(menu, LV_FLEX_FLOW_COLUMN);
     lv_obj_set_scroll_dir(menu, LV_DIR_VER);                       // bei vielen Eintraegen vertikal scrollbar
+    scroll_ohne_nachlauf(menu);
     lv_obj_set_scrollbar_mode(menu, LV_SCROLLBAR_MODE_AUTO);
     for (int i = 0; i < PG_COUNT; i++) {
         lv_obj_t* mb = th_button(menu, PAGE_NAMES[i], COL_CARD, COL_TEXT);