fix(P4): Glyphenpuffer ins PSRAM - behebt Stillstand bei großer Standby-Uhr (1.9.8)

Ursache gefunden. lv_draw_label.c legt pro Glyphe einen A8-Draw-Buf an:
box_w * LV_ROUND_UP(box_h, 32) Byte. Größte Ziffer des 360er-Fonts ist
190x247 -> 190*256 = 48 KB. Mit LV_DRAW_SW_DRAW_UNIT_CNT 2 sind zwei davon
gleichzeitig offen, also ~97 KB. Der LVGL-Pool (LV_MEM_SIZE 256 KB) hatte
laut Herzschlag nur 98068 Byte am Stück frei - es reichte knapp nicht.

LV_USE_ASSERT_MALLOC ist 1 und LV_ASSERT_HANDLER war "while(1);". Bei
fehlgeschlagener Allokation hält der LVGL-Task also stumm an: kein Panic,
kein Backtrace, Log bricht einfach ab. Genau das beobachtete Bild.

- font_buf_malloc/-free über lv_draw_buf_get_font_handlers(): Puffer ab
  16 KB kommen per heap_caps_malloc aus dem PSRAM, kleinere weiter aus dem
  LVGL-Pool (schneller; sonst würde jeder Text langsamer). Freigabe
  unterscheidet über esp_ptr_external_ram().
- LV_ASSERT_HANDLER gibt jetzt eine Meldung aus, bevor er anhält.

Der interne RAM bleibt unangetastet - LV_MEM_SIZE zu erhöhen war keine
Option, der statische Speicher liegt bereits bei ~98 %.

Beide Panel-Varianten mit arduino-cli gegengebaut.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NeFMr439WcLMdebE3kU7fS

P4_Display_Firmware/Changelog.txt

@@ -1,3 +1,36 @@
+Version 1.9.8:
+- Der Stillstand beim Wechsel in den Standby ist gefunden und behoben. LVGL legt für JEDE
+  Textglyphe einen eigenen Zwischenpuffer an - bei der großen Uhr rund 48 KB pro Ziffer, mit
+  zwei parallel arbeitenden Zeicheneinheiten fast 100 KB gleichzeitig. Der interne
+  LVGL-Speicher hatte nur rund 98 KB am Stück frei; es reichte knapp nicht. Schlägt die
+  Anforderung fehl, hält LVGL per Zusicherung in einer Endlosschleife an - kein Absturz,
+  keine Meldung, das Display steht einfach. Deshalb brach der Mitschnitt ohne Fehler ab.
+- Große Glyphenpuffer liegen jetzt im externen Speicher (PSRAM, 32 MB frei). Kleine bleiben
+  im internen, der ist schneller - sonst würde jeder Text langsamer.
+- Hält LVGL künftig wegen einer verletzten Zusicherung an, sagt es das jetzt, statt stumm
+  stehen zu bleiben.
+
+Version 1.9.7:
+- Messpunkte für den Stillstand beim Wechsel in den Standby: Beide Abläufe - der Haupt- und
+  der Zeichenablauf - melden jetzt alle zwei Sekunden einen Herzschlag mit Speicherstand.
+  Bleibt nur einer von beiden aus, ist klar, welcher stehen bleibt.
+- Das Einblenden der Standby-Uhr meldet Anfang und Ende mit Dauer. Bleibt die zweite Meldung
+  aus, hängt es genau dort.
+- Jeder Helligkeitswechsel, der länger als 50 ms braucht, wird mit seiner Dauer gemeldet. Der
+  Schreibzugriff geht über denselben Bus wie der Touch - ein Verdacht für die bekannte
+  Verzögerung von ein bis zwei Sekunden.
+- Die Rückmeldung aus 1.9.5, die einen komplett ungültigen Inhaltsbereich melden sollte, ist
+  wieder entfernt. Sie konnte nie auslösen: LVGL schickt diese Meldung an das Display, nicht
+  an einzelne Elemente. Ihr Ausbleiben war also kein Beweis.
+
+Version 1.9.6:
+- Die Darstellungseinstellungen haben einen eigenen Menüpunkt „Darstellung" bekommen. Sie
+  standen bisher oben auf der Info-Seite, zwischen Sachen, mit denen sie nichts zu tun
+  haben. Die Info-Seite zeigt jetzt nur noch Firmware-Stände und Verbindungsdaten.
+- Die Uhr im Standby ist auf dem 7-Zoll-Panel deutlich größer geworden: Die drei Stufen
+  liegen jetzt bei 180, 270 und 360 Pixeln statt bei 120, 180 und 240. Auf dem 4,3-Zoll-
+  Panel bleiben die bisherigen Stufen, dort passen größere schlicht nicht auf den Schirm.
+
 Version 1.9.5:
 - Die festen Breiten aus 1.9.4 haben nicht gereicht: Auch die HÖHE muss festliegen. LVGL
   prüft beim Setzen eines Textes, ob sich die Eigengröße der Beschriftung geändert haben

P4_Display_Firmware/P4_Display_Firmware.ino

@@ -67,11 +67,25 @@ static void onState(const MachineState& st) {
     }
     if (bl < 0) bl = 0; if (bl > 100) bl = 100;
     if (bl != s_lastBacklight) {
+        // Vermessen: Rund um jeden Helligkeitswechsel hing die Bedienung bisher ein bis
+        // zwei Sekunden. Der Schreibzugriff geht ueber denselben I2C-Bus wie der Touch.
+        uint32_t t0 = millis();
         hal_backlight(bl);
+        uint32_t dt = millis() - t0;
+        if (dt > 50) Serial.printf("[HAL] Helligkeit %d %% gesetzt, %u ms\n", bl, (unsigned)dt);
         s_lastBacklight = bl;
     }
 }
 
+// Herzschlag aus dem Haupt-Ablauf. Zusammen mit dem Herzschlag aus dem Zeichen-Ablauf
+// laesst sich unterscheiden, welcher der beiden stehen bleibt, wenn nichts mehr geht.
+static void main_heartbeat() {
+    static uint32_t s_last = 0;
+    if (millis() - s_last < 2000) return;
+    s_last = millis();
+    Serial.printf("[Haupt] Herzschlag, freier Speicher %u Byte\n", (unsigned)ESP.getFreeHeap());
+}
+
 static void onAck(uint32_t id, bool ok, const String& message) {
     // Automatische/Heartbeat-Acks nicht als Toast zeigen (pong, Handshake, Listen-Refresh).
     bool routine = ok && (message == "pong" || message == "hello" ||
@@ -332,6 +346,7 @@ static unsigned long s_lastDbgMs = 0;
 #endif
 
 void loop() {
+    main_heartbeat();
     g_client.loop();   // UART lesen, hello/heartbeat, state -> UI
     hal_loop();        // realer Modus: leer; Stub-Modus: lv_timer_handler()
 

P4_Display_Firmware/lv_conf.h

@@ -374,8 +374,10 @@
 #define LV_USE_ASSERT_OBJ           0   /*Check the object's type and existence (e.g. not deleted). (Slow)*/
 
 /*Add a custom handler when assert happens e.g. to restart the MCU*/
-#define LV_ASSERT_HANDLER_INCLUDE <stdint.h>
-#define LV_ASSERT_HANDLER while(1);   /*Halt by default*/
+#define LV_ASSERT_HANDLER_INCLUDE <esp_rom_sys.h>
+// Frueher hielt LVGL hier stumm in einer Endlosschleife an - das Display stand, ohne dass
+// irgendwo etwas davon zu sehen war. Jetzt sagt es wenigstens, dass es angehalten hat.
+#define LV_ASSERT_HANDLER   esp_rom_printf("[LVGL] Zusicherung verletzt - Zeichenablauf haelt an\n"); while(1);
 
 /*-------------
  * Debug
@@ -571,7 +573,9 @@
                                 LV_FONT_DECLARE(lv_font_maven_pro_bold_140) \
                                 LV_FONT_DECLARE(lv_font_clock_120) \
                                 LV_FONT_DECLARE(lv_font_clock_180) \
-                                LV_FONT_DECLARE(lv_font_clock_240)
+                                LV_FONT_DECLARE(lv_font_clock_240) \
+                                LV_FONT_DECLARE(lv_font_clock_270) \
+                                LV_FONT_DECLARE(lv_font_clock_360)
 
 /*Always set a default font*/
 #define LV_FONT_DEFAULT &lv_font_maven_pro_14

P4_Display_Firmware/ui.cpp

@@ -18,6 +18,7 @@
 #include <Arduino.h>
 #include <ArduinoJson.h>
 #include <Preferences.h>   // Dashboard-Design P4-lokal merken (NVS)
+#include <esp_memory_utils.h>  // esp_ptr_external_ram(): welcher Speicher einen Puffer traegt
 #include <esp_heap_caps.h> // Verlaufs-Historie im PSRAM allozieren (interner RAM ist knapp)
 #include "ui.h"
 #include <Preferences.h>
@@ -127,8 +128,8 @@ static lv_obj_t* g_otaStatus = nullptr;
 static lv_obj_t* g_otaBytes  = nullptr;
 
 // Navigation
-enum { PG_DASH = 0, PG_CHART, PG_STATS, PG_TEMP, PG_BREW, PG_COLDEX, PG_SERVICE, PG_PROFILE, PG_LIGHT, PG_WIFI, PG_INFO, PG_COUNT };
-static const char* PAGE_NAMES[PG_COUNT] = { "Dashboard", "Verlauf", "Statistik", "Temperatur", "Brew-Control", "Cold Extraction", "Reinigung/Wartung", "Profile", "Licht", "WiFi", "Info" };
+enum { PG_DASH = 0, PG_CHART, PG_STATS, PG_TEMP, PG_BREW, PG_COLDEX, PG_SERVICE, PG_PROFILE, PG_LIGHT, PG_WIFI, PG_DISPLAY, PG_INFO, PG_COUNT };
+static const char* PAGE_NAMES[PG_COUNT] = { "Dashboard", "Verlauf", "Statistik", "Temperatur", "Brew-Control", "Cold Extraction", "Reinigung/Wartung", "Profile", "Licht", "WiFi", "Darstellung", "Info" };
 static lv_obj_t* g_content = nullptr;
 static lv_obj_t* g_pages[PG_COUNT] = { nullptr };
 static lv_obj_t* g_drawer = nullptr;
@@ -700,9 +701,17 @@ static lv_obj_t* g_clockSizeBtn[3] = {};
 
 static const lv_font_t* clock_font() {
     switch (g_clockSize) {
+#if JC_PANEL_TYPE == WS_PANEL_7H
+        // 7 Zoll, 1280x720: Hier war selbst die groesste Stufe noch zu klein.
+        case UHR_KLEIN:  return &lv_font_clock_180;
+        case UHR_MITTEL: return &lv_font_clock_270;
+        default:         return &lv_font_clock_360;
+#else
+        // 4,3 Zoll, 480x480: Groessere Stufen passen schlicht nicht auf das Panel.
         case UHR_KLEIN:  return &lv_font_clock_120;
         case UHR_MITTEL: return &lv_font_clock_180;
         default:         return &lv_font_clock_240;
+#endif
     }
 }
 
@@ -3100,8 +3109,10 @@ static void brew_live_sw_cb(lv_event_t* e) {
     { Preferences prefs; prefs.begin("ui", false); prefs.putBool("brewlive", g_brewLiveEnabled); prefs.end(); }
     if (!g_brewLiveEnabled) brew_live_hide();   // laeuft gerade ein Bezug: sofort ausblenden
 }
-static void build_info(lv_obj_t* p) {
-    section_title(p, "Darstellung");
+// Eigener Menuepunkt "Darstellung". Die Einstellungen sassen frueher oben auf der
+// Info-Seite; dort standen sie zwischen Firmware-Staenden und Verbindungsdaten,
+// mit denen sie nichts zu tun haben.
+static void build_display(lv_obj_t* p) {
     lv_obj_t* dcard = th_card(p);
     lv_obj_set_width(dcard, LV_PCT(100));
     lv_obj_set_height(dcard, LV_SIZE_CONTENT);
@@ -3154,6 +3165,9 @@ static void build_info(lv_obj_t* p) {
     sw_set(swBrewLive, g_brewLiveEnabled);
     lv_obj_add_event_cb(swBrewLive, brew_live_sw_cb, LV_EVENT_VALUE_CHANGED, nullptr);
 
+}
+
+static void build_info(lv_obj_t* p) {
     section_title(p, "Firmware-Stände & Verbindung");
 
     lv_obj_t* card = th_card(p);
@@ -3552,12 +3566,40 @@ static void refr_ready_cb(lv_event_t*) {
                        (int)g_refrArea[i].y1, (int)g_refrArea[i].y2);
 }
 
-// Meldet, wenn ein ganzer Container ungueltig gemacht wird. Damit laesst sich unterscheiden,
-// ob eine einzelne Beschriftung den Bildaufbau ausloest oder ob gleich der komplette
-// Inhaltsbereich neu gezeichnet werden soll - zwei verschiedene Ursachen.
-static void inval_watch_cb(lv_event_t* e) {
-    const char* name = (const char*)lv_event_get_user_data(e);
-    esp_rom_printf("[UI] %s wird komplett ungueltig\n", name);
+static void ui_heartbeat_cb(lv_timer_t*) {
+    lv_mem_monitor_t m; lv_mem_monitor(&m);
+    esp_rom_printf("[UI] Herzschlag, LVGL-Speicher frei %u von %u Byte, groesster Block %u\n",
+                   (unsigned)m.free_size, (unsigned)m.total_size, (unsigned)m.free_biggest_size);
+}
+
+// LVGL legt fuer JEDE Textglyphe einen eigenen Zwischenpuffer an - Breite mal Hoehe, ein
+// Byte je Bildpunkt. Bei der grossen Standby-Uhr sind das 190 x 256 = rund 48 KB pro Ziffer,
+// und mit zwei parallel arbeitenden Zeicheneinheiten fast 100 KB gleichzeitig. Der interne
+// LVGL-Speicher fasst nur 245 KB, wovon rund 98 KB am Stueck frei sind - es reichte knapp
+// nicht. Schlaegt die Anforderung fehl, haelt LVGL per Zusicherung in einer Endlosschleife
+// an: kein Absturz, keine Meldung, das Display steht einfach. Genau das war der Stillstand
+// beim Wechsel in den Standby.
+//
+// Diese Puffer wandern daher in den externen Speicher (PSRAM, 32 MB frei). Kleine Glyphen
+// bleiben im internen Speicher, weil der schneller ist und Text sonst ueberall langsamer
+// wuerde; nur die grossen gehen nach aussen.
+#define FONT_BUF_EXTERN_AB 16384   // ab dieser Groesse in den externen Speicher
+
+static void* font_buf_malloc(size_t size, lv_color_format_t cf) {
+    LV_UNUSED(cf);
+    size += LV_DRAW_BUF_ALIGN - 1;
+    if (size >= FONT_BUF_EXTERN_AB) {
+        void* p = heap_caps_malloc(size, MALLOC_CAP_SPIRAM | MALLOC_CAP_8BIT);
+        if (p) return p;
+    }
+    return lv_malloc(size);
+}
+
+static void font_buf_free(void* buf) {
+    // Der externe Speicher liegt ausserhalb des LVGL-Bereichs; daran ist zu erkennen,
+    // welcher der beiden Wege den Puffer vergeben hat.
+    if (esp_ptr_external_ram(buf)) heap_caps_free(buf);
+    else                           lv_free(buf);
 }
 
 static void refr_monitor_install() {
@@ -3709,6 +3751,7 @@ void ui_init(ProtocolClient* client) {
     g_pages[PG_SERVICE] = make_page(g_content); build_service(g_pages[PG_SERVICE]);
     g_pages[PG_WIFI]    = make_page(g_content); build_wifi(g_pages[PG_WIFI]);
     g_pages[PG_PROFILE] = make_page(g_content); build_profiles(g_pages[PG_PROFILE]);
+    g_pages[PG_DISPLAY] = make_page(g_content); build_display(g_pages[PG_DISPLAY]);
     g_pages[PG_INFO]    = make_page(g_content); build_info(g_pages[PG_INFO]);
 
     // --- Schublade (Scrim + Menue-Panel) ---
@@ -3969,9 +4012,12 @@ void ui_init(ProtocolClient* client) {
     show_page(PG_DASH);
     if (g_client) g_client->sendListProfiles();
 
+    // Glyphenpuffer umlenken, bevor der erste Text gezeichnet wird.
+    lv_draw_buf_handlers_t* fh = lv_draw_buf_get_font_handlers();
+    fh->buf_malloc_cb = font_buf_malloc;
+    fh->buf_free_cb   = font_buf_free;
     refr_monitor_install();
-    if (g_content) lv_obj_add_event_cb(g_content, inval_watch_cb, LV_EVENT_INVALIDATE_AREA, (void*)"Inhaltsbereich");
-    if (g_pages[PG_DASH]) lv_obj_add_event_cb(g_pages[PG_DASH], inval_watch_cb, LV_EVENT_INVALIDATE_AREA, (void*)"Dashboard-Seite");
+    lv_timer_create(ui_heartbeat_cb, 2000, nullptr);
     g_uiReady = true;
 }
 
@@ -4161,8 +4207,14 @@ void ui_update(const MachineState& st) {
         if (shouldShow) {
             label_set_if_changed(g_standbyClock, zeit.length() ? zeit.c_str() : "--:--");
             if (!g_standbyShown) {
+                // Vermessen: Das Einblenden der Uhr ist der Moment, in dem das Display
+                // zuletzt stehen geblieben ist. Bleibt die zweite Zeile aus, haengt es hier.
+                esp_rom_printf("[UI] Standby-Uhr wird eingeblendet\n");
+                uint32_t t0 = lv_tick_get();
                 ui_fade_in(g_standbyScreen, UI_FADE_MS);   // Einschlaf-Animation
                 g_standbyShown = true;
+                esp_rom_printf("[UI] Standby-Uhr eingeblendet, %u ms\n",
+                               (unsigned)(lv_tick_get() - t0));
             }
             // Frueher wurde die Uhr hier bei JEDER Zustandsmeldung nach vorn geholt. Das
             // ordnet die Ebenen neu und laesst LVGL den gesamten Bildschirm neu zeichnen -