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,63 @@
+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,

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,14 +458,9 @@ static bool ws7_dsi_bringup(esp_lcd_dsi_bus_handle_t *out_bus,
 #else
         .pixel_format = LCD_COLOR_PIXEL_FORMAT_RGB565,
 #endif
-#if WS7_TEST_PATTERN
+        // Anzahl der Bildspeicher. Beim Waveshare-Panel genau einer: mit zweien kam nach
-        // Im Pruefbild-Modus genau ein Bildspeicher - wie im funktionierenden
+        // dem Einschalten kein Bild (siehe WS7_SINGLE_FB in pins_config.h).
-        // Werks-Testsketch. Nur so ist der Test wirklich ein Nachbau davon: Bisher lief er
-        // mit zwei Speichern und wich damit an genau der Stelle ab, die noch offen ist.
-        .num_fbs = 1,
-#else
         .num_fbs = LVGL_PORT_LCD_BUFFER_NUMS,
-#endif
         .video_timing = {
             .h_size = BSP_LCD_H_RES,
             .v_size = BSP_LCD_V_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,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);