feat(P4): Brew-Control zweispaltig auf dem 7-Zoll-Panel (1.7.0)

Die Seiten sind als einspaltige Stapel für 800x480 entstanden. Auf 1280x720
bleibt daneben rund 40 % der Fläche leer, während unten Karten aus dem Bild
fallen - und Scrollen ist auf diesem Panel teuer, weil ohne Doppelpufferung
jedes Bild komplett neu entsteht.

- Neue Helfer page_grid() und grid_at() legen eine Seite als LVGL-Raster an.
- build_brew() ordnet die vier Karten in zwei Spalten: links was den Bezug
  startet (Pre-Infusion, Brew-by-Time), rechts was ihn beendet
  (Brew-by-Weight, Dampf-Timer).
- Der Speichern-Knopf steht frei unter allen Karten statt in einer davon -
  er speichert die ganze Seite.
- Nur für WS_PANEL_7H; die JC-Panels bleiben einspaltig.

Beide Panel-Varianten mit arduino-cli gegengebaut.

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

P4_Display_Firmware/Changelog.txt

@@ -1,3 +1,136 @@
+Version 1.7.0:
+- Brew-Control ist auf dem 7-Zoll-Panel zweispaltig: Pre-Infusion und Brew-by-Weight oben,
+  Brew-by-Time und Dampf-Timer darunter, der Speichern-Knopf frei unter allen Karten.
+  Damit passt die Seite vollstaendig auf den Bildschirm und muss nicht mehr gescrollt
+  werden - der teuerste Vorgang der Oberflaeche entfaellt dort ersatzlos.
+- Die Anordnung folgt der Bedeutung: links steht, was den Bezug startet, rechts, was ihn
+  beendet.
+- Der Speichern-Knopf sitzt bewusst nicht in einer der Karten. Er speichert die ganze Seite
+  und wuerde in einer einzelnen Karte so wirken, als gaelte er nur fuer diese.
+- Die JC-Panels bleiben einspaltig - dort ist der Platz knapp, nicht ueppig.
+- Cold Extraction und Reinigung/Wartung folgen als naechstes.
+
+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
+  ersten Reset nur die Beleuchtung, erst nach dem zweiten das Bild. Ein Ab- und Wiederaufbau
+  im laufenden Betrieb ist einem echten Chip-Reset also nicht gleichwertig. WS7_DSI_RESTART
+  steht deshalb wieder auf 0; der Schalter bleibt samt Begruendung erhalten.
+- Stattdessen der Weg, den die Messungen eindeutig stuetzen: Nach einem Kaltstart startet
+  der P4 sich genau einmal selbst neu, sobald der erste Hochlauf durch ist. Damit passiert
+  genau das, was bei jedem Versuch zuverlaessig geholfen hat - ohne dass jemand die
+  Reset-Taste druecken muss.
+- Eine Schleife ist ausgeschlossen: Beim zweiten Durchlauf ist der Startgrund kein Kaltstart
+  mehr, die Bedingung greift nicht erneut. Der Fruehstart-Zaehler der Absturzsicherung wird
+  vor dem gewollten Neustart zurueckgesetzt, damit dieser nicht als Absturz zaehlt.
+- Kostet rund drei Sekunden, und nur beim Einschalten. Schalter WS7_COLD_BOOT_RESTART in
+  pins_config.h.
+- Das ist eine Umgehung, keine Erklaerung: Warum das Panel den ersten Anlauf nach dem
+  Einschalten nicht annimmt, ist weiterhin offen. Der Hochlauf meldet durchgehend Erfolg.
+
+Version 1.6.21:
+- Der Hochlauf lief zuletzt vollstaendig fehlerfrei durch - Bridge zweimal eingestellt,
+  Panel zweimal geweckt, alle I2C-Bausteine erreichbar - und trotzdem kam beim ersten
+  Einschalten kein Bild. Erst ein Reset half, und der zuverlaessig. Es fehlte also kein
+  Befehl mehr; es fehlte der Vorgang selbst.
+- Ein Reset unterscheidet sich an dieser Stelle in genau einem Punkt: Die DSI-Verbindung
+  geht einmal weg und neu auf, waehrend Bridge und Panel bereits versorgt und eingestellt
+  sind. Weder die wiederholte Grundeinstellung noch die wiederholten Weckbefehle ersetzen
+  das - beide laufen ueber eine Verbindung, die durchgehend bestand.
+- Der Hochlauf baut die DSI-Strecke jetzt einmal komplett ab und wieder auf: Bildausgabe,
+  Kommandokanal und Bus werden freigegeben und neu angelegt. Damit passiert ohne Reset,
+  was vorher erst der zweite Start brachte. Schalter WS7_DSI_RESTART in pins_config.h,
+  Pause dazwischen WS7_DSI_RESTART_MS (Vorgabe 200 ms).
+- Der DSI-Aufbau steckt dafuer in einer eigenen Funktion, damit der Vorgang wiederholbar
+  ist statt zweimal im Quelltext zu stehen.
+
+Version 1.6.20:
+- Nach 1.6.19 lief es reproduzierbar so ab: Beim Einschalten ging nach zwei bis drei
+  Sekunden die Hintergrundbeleuchtung an, dann kam nichts mehr; nach einem Reset ging die
+  Beleuchtung kurz aus und danach erschien Beleuchtung samt Bild. Daraus folgt der letzte
+  fehlende Schritt: Auch die Grundeinstellung der Bridge lief bisher nur einmal, und zwar
+  bevor die DSI-Strecke ueberhaupt existierte. Beim Reset half es nur deshalb, weil die
+  Bridge ihre Einstellung aus dem vorigen Durchlauf noch hatte und diesmal ein DSI-Signal
+  vorfand.
+- Die Grundeinstellung der Bridge wird jetzt ein zweites Mal geschrieben, sobald die
+  Videoausgabe laeuft - dasselbe Vorgehen wie bei den Weckbefehlen des Panels in 1.6.19.
+  Damit wird ohne Reset erreicht, was vorher erst der zweite Durchlauf brachte.
+- Im Start-Log sind beide Durchgaenge benannt ("Bridge-Grundeinstellung vor dem DSI-Start"
+  und "... nach dem Start der Videoausgabe").
+
+Version 1.6.19:
+- Die Wartezeit aus 1.6.18 hat den Start deutlich verbessert, aber noch nicht zuverlässig
+  gemacht. Aus dem verbliebenen Fehlerbild folgt der nächste Schritt: Die Beleuchtung ging
+  an, das Bild fehlte. Die Beleuchtung hängt an der Bridge und wird über I2C geschaltet -
+  sie funktioniert also. Verloren gehen die Weckbefehle an das Panel selbst.
+- Diese Befehle wurden bisher genau einmal geschickt, bevor die Videoausgabe lief. War der
+  Panel-Controller in dem Moment noch nicht aufnahmebereit, waren sie weg, und danach hat
+  nie wieder jemand nachgefragt. Sie werden jetzt ein zweites Mal gesendet, sobald die
+  Videoausgabe steht. Die Befehle sind wiederholbar, der zweite Anlauf kostet nichts.
+- Die Pause dazwischen steht als WS7_PANEL_SETTLE_MS in pins_config.h (Vorgabe 200 ms).
+- Im Start-Log ist jeder Anlauf einzeln zu sehen ("Weckbefehle vor/nach dem Start der
+  Videoausgabe").
+
 Version 1.6.18:
 - Ursache des unzuverlässigen Starts gefunden: Der P4 sprach Bridge und Panel sofort nach
   dem Einschalten an. Das Displaymodul braucht danach aber einen Moment, bis es bereit ist.

P4_Display_Firmware/P4_Display_Firmware.ino

@@ -24,6 +24,7 @@
 #include "protocol_client.h"
 #include "ui.h"
 #include "display_hal.h"
+#include "pins_config.h"
 #include <ArduinoJson.h>
 #include <Update.h>
 #include <Preferences.h>
@@ -296,6 +297,31 @@ void setup() {
     g_client.onOta(onOta);
 
     hal_init(&g_client);   // LVGL + Panel/Touch + Dashboard (ruft ui_init intern auf)
+
+    // --- Waveshare-Panel: einmaliger Neustart nach dem Einschalten ---
+    //
+    // Das Panel kommt nach dem Anlegen der Versorgung im ersten Anlauf nicht zuverlaessig
+    // hoch: Die Beleuchtung geht an, ein Bild bleibt aus - obwohl jeder Schritt des
+    // Hochlaufs Erfolg meldet. Ein Reset behebt es zuverlaessig, und zwar nur ein echter
+    // Chip-Reset; die DSI-Strecke im laufenden Betrieb ab- und wieder aufzubauen genuegt
+    // nicht (siehe WS7_DSI_RESTART).
+    //
+    // Deshalb startet der P4 sich nach einem Kaltstart genau einmal selbst neu, nachdem
+    // der erste Hochlauf durch ist. Beim zweiten Durchlauf ist der Startgrund kein
+    // Kaltstart mehr, die Bedingung greift also nicht erneut - eine Schleife ist
+    // ausgeschlossen. Kostet rund drei Sekunden, aber nur beim Einschalten.
+#if (JC_PANEL_TYPE == WS_PANEL_7H) && WS7_COLD_BOOT_RESTART
+    if (esp_reset_reason() == ESP_RST_POWERON) {
+        // Der Fruehstart-Zaehler wurde oben bereits erhoeht. Diesen gewollten Neustart
+        // darf er nicht mitzaehlen, sonst laeuft die Absturzsicherung nach ein paar
+        // Einschaltvorgaengen in den Update-Modus.
+        { Preferences p; p.begin("bootguard", false); p.putUShort("early", 0); p.end(); }
+        DBG_SERIAL.println(F("[Panel] Kaltstart - einmaliger Neustart fuer das Displaypanel"));
+        DBG_SERIAL.flush();
+        delay(50);
+        esp_restart();
+    }
+#endif
     g_client.begin();      // UART starten, hello folgt automatisch in loop()
 
     DBG_SERIAL.println(F("Bereit. Warte auf Verbindung zur Hauptplatine..."));

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

@@ -193,8 +193,24 @@ static void ws7_bridge_write(uint8_t reg, uint8_t value)
     }
 }
 
-// Bridge am I2C-Bus anmelden und Grundinitialisierung schreiben. Muss VOR dem
-// DSI-Bringup laufen; die Hintergrundbeleuchtung bleibt dabei noch aus.
+// Grundeinstellung der Bridge. Bewusst mehrfach aufrufbar, siehe unten.
+//
+// Warum zweimal: Beim ersten Aufruf existiert die DSI-Strecke noch gar nicht - die Bridge
+// bekommt ihre Einstellung also, bevor ein Signal anliegt. Beim ersten Einschalten reichte
+// das nicht: Die Beleuchtung ging an (die haengt an der Bridge), ein Bild kam aber nicht.
+// Erst ein Reset half - und dann nur deshalb, weil die Bridge ihre Einstellung aus dem
+// vorigen Durchlauf noch hatte und diesmal ein DSI-Signal vorfand. Genau das holt der
+// zweite Aufruf nach, ohne dass ein Reset noetig ist. Die Register sind wiederholbar.
+static void ws7_bridge_core_init(const char *wann)
+{
+    esp_rom_printf("[Panel] Bridge-Grundeinstellung %s\n", wann);
+    ws7_bridge_write(0xC0, 0x01);
+    ws7_bridge_write(0xC2, 0x01);
+    ws7_bridge_write(0xAC, 0x01);
+}
+
+// Bridge am I2C-Bus anmelden und erstmalig einstellen. Laeuft VOR dem DSI-Bringup;
+// die Hintergrundbeleuchtung bleibt dabei noch aus.
 static void ws7_bridge_pre_init(void)
 {
     i2c_device_config_t dev_cfg = {
@@ -206,9 +222,7 @@ static void ws7_bridge_pre_init(void)
         s_bridge_handle = NULL;
         return;
     }
-    ws7_bridge_write(0xC0, 0x01);
-    ws7_bridge_write(0xC2, 0x01);
-    ws7_bridge_write(0xAC, 0x01);
+    ws7_bridge_core_init("vor dem DSI-Start");
 }
 
 void jc_backlight_init(void)
@@ -383,6 +397,111 @@ static void jc_touch_scale(esp_lcd_touch_handle_t tp, uint16_t *x, uint16_t *y,
 }
 #endif
 
+#if JC_PANEL_TYPE == WS_PANEL_7H
+// Panel wecken. Bewusst mehrfach, siehe unten.
+//
+// Ein reines DPI-Panel kennt keine reset-Funktion, esp_lcd_panel_reset() wuerde ins Leere
+// greifen - deshalb nur die DCS-Befehle.
+//
+// Warum mehrfach: Verpasst der Panel-Controller diese Befehle, weil er nach dem
+// Einschalten noch nicht bereit war, bleibt der Bildschirm dunkel, obwohl der P4 alles
+// korrekt gesendet hat und jeder Schritt Erfolg meldet. Die Hintergrundbeleuchtung geht
+// trotzdem an, weil sie an der Bridge haengt und nicht am Panel - genau dieses Bild.
+// Die Befehle sind wiederholbar, ein zweiter Anlauf nach dem Start der Videoausgabe
+// kostet nichts und faengt den Fall ab.
+static void ws7_panel_wake(esp_lcd_panel_io_handle_t io, const char *wann)
+{
+    uint8_t zero = 0x00;
+    uint8_t madctl = WS7_MADCTL;
+    esp_rom_printf("[Panel] Weckbefehle %s\n", wann);
+    BSP_STEP(esp_lcd_panel_io_tx_param(io, 0x36, &madctl, 1), "Panel-Befehl MADCTL");
+    BSP_STEP(esp_lcd_panel_io_tx_param(io, 0x11, &zero, 1), "Panel-Befehl Sleep Out");
+    vTaskDelay(pdMS_TO_TICKS(120));
+    BSP_STEP(esp_lcd_panel_io_tx_param(io, 0x29, &zero, 1), "Panel-Befehl Display On");
+    vTaskDelay(pdMS_TO_TICKS(20));
+}
+#endif
+
+#if JC_PANEL_TYPE == WS_PANEL_7H
+// Baut die komplette DSI-Strecke auf: Bus, Kommandokanal, Bildausgabe, und weckt das
+// Panel. Bewusst als eigene Funktion, damit der ganze Vorgang wiederholbar ist.
+static bool ws7_dsi_bringup(esp_lcd_dsi_bus_handle_t *out_bus,
+                            esp_lcd_panel_io_handle_t *out_io,
+                            esp_lcd_panel_handle_t *out_panel,
+                            esp_lcd_dpi_panel_config_t *out_dpi_config)
+{
+    esp_lcd_dsi_bus_handle_t mipi_dsi_bus = NULL;
+    esp_lcd_panel_io_handle_t io = NULL;
+    esp_lcd_panel_handle_t disp_panel = NULL;
+
+    esp_lcd_dsi_bus_config_t bus_config = {
+        .bus_id = 0,
+        .num_data_lanes = WS7_DSI_LANES,
+        .phy_clk_src = MIPI_DSI_PHY_PLLREF_CLK_SRC_PLL_F20M,
+        .lane_bit_rate_mbps = WS7_DSI_LANE_MBPS,
+    };
+    BSP_STEP(esp_lcd_new_dsi_bus(&bus_config, &mipi_dsi_bus), "MIPI-DSI-Bus");
+
+    esp_lcd_dbi_io_config_t dbi_config = {
+        .virtual_channel = 0,
+        .lcd_cmd_bits = 8,
+        .lcd_param_bits = 8,
+    };
+    BSP_STEP(esp_lcd_new_panel_io_dbi(mipi_dsi_bus, &dbi_config, &io), "DBI-Kommandokanal");
+
+    esp_lcd_dpi_panel_config_t dpi_config = {
+        .virtual_channel = 0,
+        .dpi_clk_src = MIPI_DSI_DPI_CLK_SRC_DEFAULT,
+        .dpi_clock_freq_mhz = WS7_DPI_CLOCK_MHZ,
+#if WS7_COLOR_BITS == 24
+        .pixel_format = LCD_COLOR_PIXEL_FORMAT_RGB888,
+#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,
+            .v_size = BSP_LCD_V_RES,
+            .hsync_back_porch = WS7_HSYNC_BACK,
+            .hsync_pulse_width = WS7_HSYNC_PULSE,
+            .hsync_front_porch = WS7_HSYNC_FRONT,
+            .vsync_back_porch = WS7_VSYNC_BACK,
+            .vsync_pulse_width = WS7_VSYNC_PULSE,
+            .vsync_front_porch = WS7_VSYNC_FRONT,
+        },
+        .flags = {
+            .use_dma2d = true,
+        },
+    };
+    BSP_STEP(esp_lcd_new_panel_dpi(mipi_dsi_bus, &dpi_config, &disp_panel),
+             "DPI-Panel (Framebuffer im PSRAM)");
+    if (disp_panel == NULL) {
+        esp_rom_printf("[Panel] Abbruch: Panel wurde nicht angelegt. Haeufigste Ursache ist zu "
+                       "wenig PSRAM - PSRAM in den Board-Einstellungen aktivieren oder in "
+                       "pins_config.h weniger Framebuffer waehlen.\n");
+        return false;
+    }
+
+    ws7_panel_wake(io, "vor dem Start der Videoausgabe");
+
+    BSP_STEP(esp_lcd_panel_init(disp_panel), "DPI-Videoausgabe starten");
+
+    // Zweiter Anlauf, jetzt bei laufender Videoausgabe: faengt den Fall ab, dass das
+    // Panel beim ersten Mal noch nicht aufnahmebereit war.
+    vTaskDelay(pdMS_TO_TICKS(WS7_PANEL_SETTLE_MS));
+    ws7_bridge_core_init("nach dem Start der Videoausgabe");
+    ws7_panel_wake(io, "nach dem Start der Videoausgabe");
+
+    *out_bus = mipi_dsi_bus;
+    *out_io = io;
+    *out_panel = disp_panel;
+    *out_dpi_config = dpi_config;
+    return true;
+}
+#endif
+
 // Bringt Panel + Touch + LVGL-Port hoch (LVGL laeuft danach in eigenem Task).
 // Rueckgabe false: Panel kam nicht hoch - der Aufrufer darf dann KEINE LVGL-Funktion
 // benutzen, sonst folgt ein zweiter Absturz, der die eigentliche Ursache ueberdeckt.
@@ -444,9 +563,12 @@ bool jc_board_bringup(void)
     };
     BSP_STEP(esp_ldo_acquire_channel(&ldo_cfg, &phy_pwr_chan), "Stromversorgung MIPI-D-PHY");
 
-    esp_lcd_dsi_bus_handle_t mipi_dsi_bus;
+    esp_lcd_dsi_bus_handle_t mipi_dsi_bus = NULL;
     esp_lcd_panel_io_handle_t io = NULL;
     esp_lcd_panel_handle_t disp_panel = NULL;
+#if JC_PANEL_TYPE == WS_PANEL_7H
+    esp_lcd_dpi_panel_config_t dpi_config;
+#endif
 
 #if JC_PANEL_TYPE == WS_PANEL_7H
     // ---------------- Waveshare 7inch DSI LCD (H) (1280x720) ----------------
@@ -456,67 +578,29 @@ bool jc_board_bringup(void)
     bsp_report_psram((size_t)BSP_LCD_H_RES * BSP_LCD_V_RES * BSP_LCD_FB_BYTES_PER_PX
                      * LVGL_PORT_LCD_BUFFER_NUMS);
 
-    esp_lcd_dsi_bus_config_t bus_config = {
-        .bus_id = 0,
-        .num_data_lanes = WS7_DSI_LANES,
-        .phy_clk_src = MIPI_DSI_PHY_PLLREF_CLK_SRC_PLL_F20M,
-        .lane_bit_rate_mbps = WS7_DSI_LANE_MBPS,
-    };
-    BSP_STEP(esp_lcd_new_dsi_bus(&bus_config, &mipi_dsi_bus), "MIPI-DSI-Bus");
-
-    esp_lcd_dbi_io_config_t dbi_config = {
-        .virtual_channel = 0,
-        .lcd_cmd_bits = 8,
-        .lcd_param_bits = 8,
-    };
-    BSP_STEP(esp_lcd_new_panel_io_dbi(mipi_dsi_bus, &dbi_config, &io), "DBI-Kommandokanal");
-
-    esp_lcd_dpi_panel_config_t dpi_config = {
-        .virtual_channel = 0,
-        .dpi_clk_src = MIPI_DSI_DPI_CLK_SRC_DEFAULT,
-        .dpi_clock_freq_mhz = WS7_DPI_CLOCK_MHZ,
-#if WS7_COLOR_BITS == 24
-        .pixel_format = LCD_COLOR_PIXEL_FORMAT_RGB888,
-#else
-        .pixel_format = LCD_COLOR_PIXEL_FORMAT_RGB565,
-#endif
-        .num_fbs = LVGL_PORT_LCD_BUFFER_NUMS,
-        .video_timing = {
-            .h_size = BSP_LCD_H_RES,
-            .v_size = BSP_LCD_V_RES,
-            .hsync_back_porch = WS7_HSYNC_BACK,
-            .hsync_pulse_width = WS7_HSYNC_PULSE,
-            .hsync_front_porch = WS7_HSYNC_FRONT,
-            .vsync_back_porch = WS7_VSYNC_BACK,
-            .vsync_pulse_width = WS7_VSYNC_PULSE,
-            .vsync_front_porch = WS7_VSYNC_FRONT,
-        },
-        .flags = {
-            .use_dma2d = true,
-        },
-    };
-    BSP_STEP(esp_lcd_new_panel_dpi(mipi_dsi_bus, &dpi_config, &disp_panel),
-             "DPI-Panel (Framebuffer im PSRAM)");
-    if (disp_panel == NULL) {
-        esp_rom_printf("[Panel] Abbruch: Panel wurde nicht angelegt. Haeufigste Ursache ist zu "
-                       "wenig PSRAM - PSRAM in den Board-Einstellungen aktivieren oder in "
-                       "pins_config.h weniger Framebuffer waehlen.\n");
+    if (!ws7_dsi_bringup(&mipi_dsi_bus, &io, &disp_panel, &dpi_config)) {
         return false;
     }
 
-    // Panel wecken (MADCTL / Sleep Out / Display On). Ein Reset gibt es hier nicht:
-    // ein reines DPI-Panel kennt keine reset-Funktion, esp_lcd_panel_reset() wuerde
-    // ins Leere greifen.
-    {
-        uint8_t zero = 0x00;
-        uint8_t madctl = WS7_MADCTL;
-        BSP_STEP(esp_lcd_panel_io_tx_param(io, 0x36, &madctl, 1), "Panel-Befehl MADCTL");
-        BSP_STEP(esp_lcd_panel_io_tx_param(io, 0x11, &zero, 1), "Panel-Befehl Sleep Out");
-        vTaskDelay(pdMS_TO_TICKS(120));
-        BSP_STEP(esp_lcd_panel_io_tx_param(io, 0x29, &zero, 1), "Panel-Befehl Display On");
-        vTaskDelay(pdMS_TO_TICKS(20));
+#if WS7_DSI_RESTART
+    // Die Strecke einmal komplett ab- und wieder aufbauen.
+    //
+    // Beim ersten Einschalten kam trotz fehlerfreiem Hochlauf kein Bild; erst ein Reset
+    // half, und zwar zuverlaessig. Ein Reset unterscheidet sich an dieser Stelle nur in
+    // einem Punkt von unserem Ablauf: Die DSI-Verbindung geht einmal weg und neu auf,
+    // waehrend Bridge und Panel bereits versorgt und eingestellt sind. Weder die
+    // wiederholte Grundeinstellung der Bridge noch die wiederholten Weckbefehle ersetzen
+    // das. Genau dieser Ab- und Wiederaufbau wird hier nachgeholt - ohne Reset.
+    esp_rom_printf("[Panel] DSI-Strecke wird neu aufgebaut\n");
+    esp_lcd_panel_del(disp_panel);
+    esp_lcd_panel_io_del(io);
+    esp_lcd_del_dsi_bus(mipi_dsi_bus);
+    vTaskDelay(pdMS_TO_TICKS(WS7_DSI_RESTART_MS));
+
+    if (!ws7_dsi_bringup(&mipi_dsi_bus, &io, &disp_panel, &dpi_config)) {
+        return false;
     }
-    BSP_STEP(esp_lcd_panel_init(disp_panel), "DPI-Videoausgabe starten");
+#endif
 #if WS7_TEST_PATTERN
     // Prueffarbbalken: Sie entstehen im DSI-Baustein selbst und benutzen weder den
     // Bildspeicher noch LVGL. Erscheinen sie, arbeitet die Anzeigekette bis zum Panel und

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
@@ -154,6 +185,27 @@
 // der erste Wert zum Erhoehen; laeuft es sicher, kann er vorsichtig gesenkt werden.
 #define WS7_PANEL_WARMUP_MS 2000
 
+// Pause zwischen dem Start der Videoausgabe und dem zweiten Anlauf der Weckbefehle.
+// Verpasst das Panel den ersten Anlauf, bleibt der Bildschirm dunkel, waehrend die
+// Beleuchtung brennt - sie haengt an der Bridge, nicht am Panel.
+#define WS7_PANEL_SETTLE_MS 200
+
+// Die DSI-Strecke beim Hochlauf einmal ab- und wieder aufbauen. Beim ersten Einschalten
+// kam sonst trotz fehlerfreiem Ablauf kein Bild; erst ein Reset half. Der einzige
+// Unterschied dabei: Die DSI-Verbindung geht einmal weg und neu auf, waehrend Bridge und
+// Panel schon versorgt und eingestellt sind. Genau das holt der zweite Aufbau nach.
+// Kostet WS7_DSI_RESTART_MS zusaetzliche Startzeit.
+#define WS7_DSI_RESTART    0
+#define WS7_DSI_RESTART_MS 200
+
+// Nach einem Kaltstart einmal selbst neu starten, sobald der erste Hochlauf durch ist.
+// Das Panel kommt beim ersten Anlauf nach dem Einschalten nicht zuverlaessig hoch;
+// ein echter Chip-Reset behebt es zuverlaessig, der Ab- und Wiederaufbau der
+// DSI-Strecke im Betrieb dagegen nicht. Kostet rund drei Sekunden beim Einschalten.
+// Eine Schleife ist ausgeschlossen: Beim zweiten Durchlauf ist der Startgrund kein
+// Kaltstart mehr.
+#define WS7_COLD_BOOT_RESTART 1
+
 #define WS7_TEST_PATTERN 0
 
 #elif JC_PANEL_TYPE == JC_PANEL_70

P4_Display_Firmware/ui.cpp

@@ -449,6 +449,61 @@ 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.
+
+// ------------------------------------------------------------------------------------
+//  Mehrspaltige Seiten fuer das grosse Panel
+//
+//  Die Seiten sind als einspaltige Stapel entstanden - entworfen fuer 800x480, wo eine
+//  Spalte alles ist, was hineinpasst. Auf 1280x720 bleibt daneben rund 40 % der Flaeche
+//  leer, waehrend unten Karten aus dem Bild fallen und gescrollt werden muss. Und Scrollen
+//  ist auf diesem Panel teuer: Ohne Doppelpufferung entsteht dabei jedes Bild komplett neu.
+//
+//  Deshalb hier ein Raster statt eines Stapels. Passt eine Seite vollstaendig auf den
+//  Bildschirm, entfaellt der teuerste Vorgang der Oberflaeche ersatzlos.
+//
+//  Die JC-Panels bleiben einspaltig: Dort ist der Platz knapp, nicht ueppig.
+// ------------------------------------------------------------------------------------
+#if JC_PANEL_TYPE == WS_PANEL_7H
+static const int32_t PAGE_COLS_2[]  = { LV_GRID_FR(1), LV_GRID_FR(1), LV_GRID_TEMPLATE_LAST };
+// Zwei Kartenreihen, darunter eine Reihe fuer den Speichern-Knopf.
+static const int32_t PAGE_ROWS_2C[] = { LV_GRID_CONTENT, LV_GRID_CONTENT, LV_GRID_CONTENT,
+                                        LV_GRID_TEMPLATE_LAST };
+
+static void page_grid(lv_obj_t* p, const int32_t* cols, const int32_t* rows) {
+    lv_obj_set_layout(p, LV_LAYOUT_GRID);
+    lv_obj_set_grid_dsc_array(p, cols, rows);
+    lv_obj_set_style_pad_column(p, 12, 0);
+    lv_obj_set_style_pad_row(p, 12, 0);
+}
+
+// Karte in eine Rasterzelle setzen. Die Breite kommt ab jetzt aus der Spalte, deshalb muss
+// die Prozentbreite aus group_card() zurueckgenommen werden - sie bezoege sich sonst auf
+// die ganze Seite statt auf die Zelle.
+static void grid_at(lv_obj_t* c, uint8_t col, uint8_t row, uint8_t col_span = 1) {
+    lv_obj_set_width(c, LV_SIZE_CONTENT);
+    lv_obj_set_grid_cell(c, LV_GRID_ALIGN_STRETCH, col, col_span,
+                            LV_GRID_ALIGN_START, row, 1);
+}
+#endif
+
+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 +517,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;
 }
 
@@ -2083,7 +2139,23 @@ static void build_brew(lv_obj_t* p) {
     swSBT      = add_switch(cSBT, "Aktiv");
     taSbtSecs  = add_num(cSBT, "Zielzeit (s)");
 
+#if JC_PANEL_TYPE == WS_PANEL_7H
+    // Zwei Spalten: links, was den Bezug startet, rechts, was ihn beendet.
+    page_grid(p, PAGE_COLS_2, PAGE_ROWS_2C);
+    grid_at(cPI,  0, 0);
+    grid_at(cBBW, 1, 0);
+    grid_at(cBBT, 0, 1);
+    grid_at(cSBT, 1, 1);
+
+    // Speichern frei unter den Karten, nicht in einer davon: Der Knopf speichert die ganze
+    // Seite und wuerde in einer einzelnen Karte so wirken, als gaelte er nur fuer diese.
+    lv_obj_t* sb = th_accent_button(p, "Brühen speichern");
+    lv_obj_add_event_cb(sb, send_save_brew, LV_EVENT_CLICKED, nullptr);
+    lv_obj_set_width(sb, LV_SIZE_CONTENT);
+    lv_obj_set_grid_cell(sb, LV_GRID_ALIGN_END, 0, 2, LV_GRID_ALIGN_START, 2, 1);
+#else
     add_save_button(p, "Brühen speichern", send_save_brew);
+#endif
 }
 
 // =====================================================================================
@@ -2304,6 +2376,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 +3362,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);