fix(P4): Hängenden I2C-Bus freitakten - Ursache des schwarzen Bildes (1.6.13)

Der Log nach einem Reset zeigte alle Panel-Schritte fehlerfrei, aber
"i2c transaction failed" beim GT911. Bridge und Touch teilen sich den Bus:
Trifft ein Reset des P4 mitten in eine Übertragung, bleibt der Baustein in
seiner Bitausgabe stehen und hält SDA dauerhaft low. Beide hängen an
Dauerstrom, der Zustand überlebt also Reset und Flashen.

- ws7_i2c_bus_recover() taktet SCL vor dem Anlegen des Busses bis zu neunmal
  von Hand, bis SDA freigegeben wird, und erzeugt dann eine Stop-Bedingung.
  Bei freiem Bus läuft die Funktion wirkungslos durch.
- ws7_bridge_write() prüft den Rückgabewert. Bisher liefen fehlgeschlagene
  Schreibzugriffe stillschweigend ins Leere - der eigentliche Grund, warum
  das schwarze Bild keinerlei Spur hinterließ.
- pins_config.h: "#elsealle" statt "#else" seit 1.6.10 korrigiert. Der
  Tippfehler traf nur den JC_PANEL_43-Zweig, der dadurch nicht mehr
  übersetzbar war. Beide Panel-Varianten wieder gegengebaut.

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

P4_Display_Firmware/Changelog.txt

@@ -1,3 +1,94 @@
+Version 1.6.13:
+- Ursache des schwarzen Bildes gefunden: ein haengender I2C-Bus. Trifft ein Reset des P4
+  mitten in eine laufende Uebertragung, bleibt der angesprochene Baustein in seiner
+  Bitausgabe stehen und haelt die Datenleitung dauerhaft auf Masse. Damit schlaegt jede
+  weitere Uebertragung fehl. Bridge und Touch-Controller haengen an Dauerstrom, weshalb
+  dieser Zustand jeden Reset und sogar das Flashen ueberlebt - nur echtes Stromabziehen
+  loest ihn. Die Folge: Die Bridge wird nie initialisiert, das Bild bleibt schwarz, und
+  weil sie ihre Register behaelt, brennt die Hintergrundbeleuchtung weiter.
+- Vor dem Anlegen des I2C-Busses wird die Taktleitung jetzt von Hand getaktet, bis der
+  Baustein die Datenleitung freigibt, danach folgt eine Stopp-Bedingung. Das ist der in
+  der I2C-Spezifikation vorgesehene Weg und laeuft bei freiem Bus wirkungslos durch.
+- Fehlerbehebung: Schreibzugriffe auf die Bridge liefen stillschweigend ins Leere. Genau
+  deshalb war der Fehler so schwer zu finden - das Bild blieb ohne jeden Hinweis schwarz.
+  Sie melden Fehler jetzt im Klartext.
+- Fehlerbehebung: In pins_config.h stand seit 1.6.10 "#elsealle" statt "#else". Der
+  Tippfehler traf nur den Zweig des 4,3-Zoll-Panels, das damit nicht mehr uebersetzbar war.
+  Beide Panel-Varianten sind jetzt wieder gegengebaut.
+
+Version 1.6.12:
+- Die Abschaltfolge für die Bridge aus 1.6.11 ist zurückgenommen. Sie hat den Fehler nicht
+  behoben, sondern verschlimmert: Nach dem Nullsetzen von C0/C2/AC kam das Panel auch
+  direkt nach dem Flashen nicht mehr hoch, die normale Einschaltfolge holt es aus diesem
+  Zustand nicht zurück. Der Start entspricht damit wieder 1.6.10.
+- Stattdessen zwei Diagnosen, um die Ursache des schwarzen Bildes nach einem Reset
+  einzugrenzen: Der Startgrund wird beim Hochlauf ausgegeben (1 = Kaltstart, alles andere
+  Warmstart), und das Anfordern der Stromversorgung für die MIPI-Strecke wird jetzt auf
+  Fehler geprüft - bislang lief dieser Schritt als einziger ungeprüft durch.
+
+Version 1.6.10:
+- WS7_PARTIAL_REFRESH ist jetzt auch im Auslieferungsstand 1. Am Gerät geprüft: kein
+  Flackern, und der Gewinn ist der größte der drei Tempo-Maßnahmen - vor allem während
+  eines Bezugs, wo die Zeitanzeige zehnmal je Sekunde weiterläuft.
+- Der Hintergrund der Seiten ist deckend statt durchsichtig. Sichtbar ändert sich nichts,
+  die Ebenen darunter haben dieselbe Farbe; LVGL muss sie beim Scrollen aber nicht mehr
+  mitzeichnen und durchmischen.
+- Neuer Schalter WS7_PERF_MONITOR in pins_config.h: blendet unten rechts Bilder je Sekunde
+  und Prozessorlast ein. Damit lässt sich beantworten, ob weitere Mühe beim Zeichnen
+  überhaupt noch etwas bringt, statt ins Blaue zu optimieren.
+
+Version 1.6.9:
+- Fehlerbehebung: Ein Tipp auf einen Knopf löste oft mehrfach aus. Der GT9271 lässt
+  gelegentlich einen Messzyklus aus oder liefert ein unplausibles Paket; der
+  Hersteller-Treiber kann "gerade nichts Neues" nicht von "Finger weg" unterscheiden und
+  meldet beides als losgelassen. Seit die Oberfläche mit 1.6.8 doppelt so oft abtastet,
+  wurde daraus sichtbar Drücken-Loslassen-Drücken-Loslassen.
+- Vor den Treiber ist deshalb ein Filter gesetzt: Er verwirft unplausible Rohwerte und hält
+  den letzten gültigen Berührpunkt noch WS7_TOUCH_HOLD_MS lang (Vorgabe 40 ms). Losgelassen
+  wird erst gemeldet, wenn wirklich nichts mehr kommt. Der Wert steht in pins_config.h;
+  0 schaltet die Überbrückung ab. Zu groß gewählt verzögert er das Loslassen und dämpft
+  den Schwung beim Wischen.
+- Der Filter hängt sich zur Laufzeit in den Treiber ein, die Herstellerdateien bleiben
+  unverändert.
+
+Version 1.6.8:
+- Scrollen und Wischen laufen am Waveshare-Panel flüssiger: LVGL rechnet dort jetzt im
+  16-Millisekunden-Takt statt alle 33 ms. Dieser Wert steuert nicht nur die Bildrate,
+  sondern auch, wie oft der Touch abgetastet und wie fein eine Animation gerechnet wird -
+  mit 30 Schritten je Sekunde war vor allem das Ziehen mit dem Finger sichtbar ruckelig.
+  Das Panel läuft mit 60 Hz, mehr als 16 ms bringt also nichts.
+- Kommt das Zeichnen einmal nicht hinterher, wird ein Bild später fertig; kaputt geht
+  dabei nichts. Die JC-Panels bleiben bei 33 ms.
+
+Version 1.6.7:
+- Der Menue-Knopf oben links fängt Tipps jetzt auch knapp daneben ab. Mit 48x36 Pixeln war
+  er auf dem 7-Zoll-Panel ein sehr kleines Ziel; gemessen wurde ein Fehlgriff von rund
+  10 Pixeln neben der Fläche, weshalb sich das Menü scheinbar nicht öffnen ließ. Optisch
+  bleibt der Knopf unverändert, die Trefferfläche wächst um 20 Pixel nach allen Seiten.
+  Sie wirkt nur innerhalb der Kopfleiste - dem Seiteninhalt darunter wird nichts genommen.
+- Die Touch-Zuordnung selbst war korrekt: Eine Messung mit WS7_TOUCH_DEBUG hat gezeigt,
+  dass Achsen und Spiegelung stimmen.
+
+Version 1.6.6:
+- Neuer Schalter WS7_TOUCH_DEBUG in pins_config.h: Auf 1 gesetzt, gibt jeder Berührpunkt
+  seine Rohwerte und die daraus errechnete Bildposition auf der seriellen Konsole aus.
+  Gedacht zum Einmessen des Touch, wenn Bedienelemente nicht reagieren - insbesondere am
+  oberen Bildrand, wo die 44 Pixel hohe Kopfleiste sitzt.
+
+Version 1.6.5:
+- Zwei der drei Tempo-Maßnahmen aus 1.6.4 haben sich am Gerät als untauglich erwiesen und
+  stehen wieder auf ihrem alten Wert. Sie bleiben als Schalter erhalten, mit Notiz, woran
+  sie gescheitert sind.
+  * WS7_COLOR_BITS wieder 24: Die Bridge dieses Panels nimmt kein RGB565 an, mit 16 Bit
+    kommt kein brauchbares Bild.
+  * WS7_PARTIAL_REFRESH wieder 0: Der Menü-Vorhang ließ sich nicht mehr öffnen. Er ist
+    halbtransparent und wird deshalb in den gerade aktiven Bildpuffer eingemischt; die
+    nächste Zustandsmeldung der S3 zeichnet aber in den zweiten Puffer, der ihn nie
+    bekommen hat - der Vorhang war also sofort wieder weg. Nutzbar wäre der Modus erst,
+    wenn der LVGL-Port die geänderten Bereiche zwischen beiden Puffern abgleicht.
+- WS7_PARALLEL_RENDER bleibt eingeschaltet: LVGL zeichnet weiterhin mit zwei Threads auf
+  beiden Prozessorkernen. Von den drei Maßnahmen trägt nur diese.
+
 Version 1.6.4:
 - Waveshare 7" (H): Drei Maßnahmen gegen die träge Bedienung am großen Panel. 1280x720
   sind 2,4-mal so viele Pixel wie beim 4,3-Zöller, und bis 1.6.3 wurde bei jeder Änderung

P4_Display_Firmware/README.md

@@ -44,13 +44,13 @@ Vendor-Pakete: 4,3" im Ordner `JC4880P443C_I_W/`, 7,0" im Repo-Ordner
 | Haupt-MCU (UI) | ESP32-P4 | ESP32-P4 | ESP32-P4 (32 MB PSRAM, 32 MB Flash) |
 | WLAN/BT (hier ungenutzt) | ESP32-C6 | ESP32-C6 | — (kein Funkmodul) |
 | Display | ST7701, 480×800 IPS, MIPI-DSI | JD9165, 1024×600 IPS, MIPI-DSI | 1280×720 IPS, MIPI-DSI 2 Lanes über Waveshare-Bridge |
-| Farbformat | RGB565 | RGB565 | RGB565 oder RGB888, `WS7_COLOR_BITS` |
+| Farbformat | RGB565 | RGB565 | **RGB888** — die Bridge nimmt kein RGB565 (geprüft) |
 | LVGL-Ausrichtung | Rotation **270°** (PPA) → 800×480 | **keine** Rotation, nativ 1024×600 | **keine** Rotation, nativ 1280×720 |
 | Touch | GT911, I²C SDA=GPIO7 / SCL=GPIO8 | GT911, I²C SDA=GPIO7 / SCL=GPIO8 | **GT9271** (I²C-Adresse **0x14**), SDA=GPIO7 / SCL=GPIO8 |
 | 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 | 2 × 1,8 MB ≈ 3,7 MB PSRAM (RGB565, Teil-Neuzeichnen) |
+| Framebuffer | 3 × 750 KB ≈ 2,3 MB PSRAM | 3 × 1,2 MB ≈ 3,6 MB PSRAM | 3 × 2,8 MB ≈ 8,3 MB PSRAM |
 | 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** |
 
@@ -81,11 +81,15 @@ Das Panel hängt hinter einer Waveshare-Bridge und braucht **keinen** Hersteller
 [`pins_config.h`](pins_config.h) halten die Bedienung flüssig; jeder ist einzeln
 zurückstellbar, falls er Ärger macht:
 
-| Schalter | Vorgabe | Wirkung | Wenn es klemmt |
+| Schalter | Vorgabe | Wirkung | Stand am Gerät |
 |---|---|---|---|
-| `WS7_COLOR_BITS` | `16` | RGB565 halbiert die Datenmenge je Bild gegenüber RGB888. Sichtbar nur in den dunklen Verläufen der Rundanzeige. | Bild falschfarbig oder dunkel → die Bridge will kein RGB565, auf `24` stellen |
-| `WS7_PARTIAL_REFRESH` | `1` | Nur geänderte Bildbereiche neu zeichnen statt jedes Mal des ganzen Bildes (LVGL-Direct-Mode, zwei Puffer). Größter Gewinn. | Flackern in Flächen, die selten neu gezeichnet werden → auf `0` |
-| `WS7_PARALLEL_RENDER` | `1` | LVGL zeichnet mit zwei Threads auf beiden Prozessorkernen. | P4 startet nicht mehr (zwei zusätzliche Stacks im knappen internen RAM) → zuerst hier auf `0` |
+| `WS7_COLOR_BITS` | `24` | `16` (RGB565) würde die Datenmenge je Bild halbieren. | **Geht an diesem Panel nicht** — die Bridge nimmt nur RGB888 |
+| `WS7_PARTIAL_REFRESH` | `0` | `1` zeichnet nur geänderte Bereiche neu (LVGL-Direct-Mode, zwei wechselnde Puffer). Wäre der größte Gewinn. | **Geht so nicht** — der halbtransparente Menü-Vorhang landet nur in einem der beiden Puffer und ist sofort wieder weg |
+| `WS7_PARALLEL_RENDER` | `1` | LVGL zeichnet mit zwei Threads auf beiden Prozessorkernen. | Aktiv |
+
+> Damit `WS7_PARTIAL_REFRESH` nutzbar würde, müsste der LVGL-Port die geänderten Bereiche
+> zwischen beiden Puffern abgleichen. Die Maschinerie dafür liegt in `lvgl_port_v9.c`,
+> wird aber nur bei gedrehter Ausgabe übersetzt.
 
 Die Farbtiefe muss für Sketch **und** LVGL-Bibliothek dieselbe sein; `lv_conf.h` liest
 `WS7_COLOR_BITS` deshalb direkt aus `pins_config.h`.

P4_Display_Firmware/board_bringup.c

@@ -29,6 +29,8 @@
 #include "esp_cache.h"
 #include "esp_heap_caps.h"
 #include "esp_rom_sys.h"
+#include "esp_timer.h"
+#include "esp_system.h"
 #include "esp_ldo_regulator.h"
 #include "esp_lcd_panel_io.h"
 #include "esp_lcd_panel_ops.h"
@@ -69,6 +71,68 @@
 
 static i2c_master_bus_handle_t s_i2c_handle = NULL;
 
+#if JC_PANEL_TYPE == WS_PANEL_7H
+// ------------------------------------------------------------------------------------
+//  Haengenden I2C-Bus befreien
+//
+//  Trifft ein Reset des P4 mitten in eine laufende I2C-Uebertragung, bleibt der
+//  angesprochene Baustein in seiner Bitausgabe stehen und haelt die Datenleitung dauerhaft
+//  auf Masse. Der Bus ist damit tot: Jede weitere Uebertragung schlaegt fehl. Da Bridge und
+//  Touch-Controller an Dauerstrom haengen, ueberlebt dieser Zustand jeden Reset und sogar
+//  das Flashen - nur echtes Stromabziehen loest ihn.
+//
+//  Folge: Die Bridge wird nie initialisiert, das Bild bleibt schwarz, und weil sie ihre
+//  Register behaelt, brennt die Hintergrundbeleuchtung weiter. Genau dieses Bild.
+//
+//  Abhilfe ist der uebliche Weg aus der I2C-Spezifikation: Vor dem Anlegen des Busses die
+//  Taktleitung von Hand takten, bis der Baustein sein letztes Bit ausgegeben hat und die
+//  Datenleitung wieder loslaesst, danach eine Stopp-Bedingung erzeugen.
+static void ws7_i2c_bus_recover(void)
+{
+    gpio_config_t io = {
+        .pin_bit_mask = BIT64(BSP_I2C_SDA) | BIT64(BSP_I2C_SCL),
+        .mode = GPIO_MODE_INPUT_OUTPUT_OD,     // offener Kollektor, Zustand lesbar
+        .pull_up_en = GPIO_PULLUP_ENABLE,
+        .pull_down_en = GPIO_PULLDOWN_DISABLE,
+        .intr_type = GPIO_INTR_DISABLE,
+    };
+    gpio_config(&io);
+    gpio_set_level(BSP_I2C_SDA, 1);
+    gpio_set_level(BSP_I2C_SCL, 1);
+    esp_rom_delay_us(10);
+
+    if (gpio_get_level(BSP_I2C_SDA) != 0) {
+        return;                                 // Bus frei - nichts zu tun
+    }
+
+    esp_rom_printf("[Panel] I2C-Bus haengt - wird freigetaktet\n");
+
+    // Bis zu neun Takte: so viele braucht ein Baustein hoechstens, um ein angefangenes
+    // Byte samt Bestaetigungsbit zu Ende zu schieben.
+    for (int i = 0; i < 9 && gpio_get_level(BSP_I2C_SDA) == 0; i++) {
+        gpio_set_level(BSP_I2C_SCL, 0);
+        esp_rom_delay_us(5);
+        gpio_set_level(BSP_I2C_SCL, 1);
+        esp_rom_delay_us(5);
+    }
+
+    // Stopp-Bedingung: Datenleitung geht bei hoher Taktleitung von Masse auf High.
+    gpio_set_level(BSP_I2C_SDA, 0);
+    esp_rom_delay_us(5);
+    gpio_set_level(BSP_I2C_SCL, 1);
+    esp_rom_delay_us(5);
+    gpio_set_level(BSP_I2C_SDA, 1);
+    esp_rom_delay_us(10);
+
+    esp_rom_printf("[Panel] I2C-Bus nach Freitakten: %s\n",
+                   gpio_get_level(BSP_I2C_SDA) ? "frei" : "weiterhin belegt");
+
+    // Pins wieder freigeben, damit der I2C-Treiber sie uebernehmen kann.
+    gpio_reset_pin(BSP_I2C_SDA);
+    gpio_reset_pin(BSP_I2C_SCL);
+}
+#endif  // JC_PANEL_TYPE == WS_PANEL_7H
+
 // Klartext-Meldung auf der seriellen Konsole. Bewusst esp_rom_printf: das laeuft auch bei
 // abgeschaltetem Core-Debug-Level und ohne Heap, also genau dann, wenn es darauf ankommt.
 #define BSP_STEP(call, what)                                                             \
@@ -106,7 +170,13 @@ static void ws7_bridge_write(uint8_t reg, uint8_t value)
 {
     if (s_bridge_handle == NULL) return;
     uint8_t buf[2] = { reg, value };
-    i2c_master_transmit(s_bridge_handle, buf, sizeof(buf), 100);
+    esp_err_t err = i2c_master_transmit(s_bridge_handle, buf, sizeof(buf), 100);
+    if (err != ESP_OK) {
+        // Frueher lief das stillschweigend ins Leere. Schlaegt es fehl, wird die Bridge
+        // nie initialisiert und das Bild bleibt schwarz - ohne jeden Hinweis darauf.
+        esp_rom_printf("[Panel] Bridge-Register 0x%02X liess sich nicht schreiben: %s\n",
+                       reg, esp_err_to_name(err));
+    }
 }
 
 // Bridge am I2C-Bus anmelden und Grundinitialisierung schreiben. Muss VOR dem
@@ -184,6 +254,63 @@ IRAM_ATTR static bool on_vsync(esp_lcd_panel_handle_t panel,
 }
 
 #if JC_PANEL_TYPE == WS_PANEL_7H
+
+// ------------------------------------------------------------------------------------
+//  Aussetzer des GT9271 ueberbruecken
+//
+//  Der Controller laesst gelegentlich einen Messzyklus aus und liefert ab und zu ein
+//  unplausibles Paket - die Werks-Testskizze hat solche Pakete ebenfalls verworfen. Der
+//  Hersteller-Treiber kann "gerade nichts Neues" nicht von "Finger weg" unterscheiden und
+//  meldet beides als losgelassen. Seit die Oberflaeche doppelt so oft abtastet, faellt das
+//  auf: Aus einem Tipp wird Druecken-Loslassen-Druecken-Loslassen, der Knopf loest also
+//  mehrfach aus.
+//
+//  Deshalb liegt hier ein eigener get_xy-Aufsatz vor dem Treiber: Er verwirft unplausible
+//  Rohwerte und haelt den letzten gueltigen Punkt kurz fest. Losgelassen wird erst
+//  gemeldet, wenn WS7_TOUCH_HOLD_MS lang wirklich nichts mehr kam.
+// ------------------------------------------------------------------------------------
+static bool (*s_touch_get_xy_orig)(esp_lcd_touch_handle_t, uint16_t *, uint16_t *,
+                                   uint16_t *, uint8_t *, uint8_t) = NULL;
+static uint16_t s_touch_last_x = 0;
+static uint16_t s_touch_last_y = 0;
+static int64_t  s_touch_last_us = 0;
+static bool     s_touch_down = false;
+
+static bool ws7_touch_get_xy(esp_lcd_touch_handle_t tp, uint16_t *x, uint16_t *y,
+                             uint16_t *strength, uint8_t *point_num, uint8_t max_point_num)
+{
+    bool got = s_touch_get_xy_orig(tp, x, y, strength, point_num, max_point_num);
+    int64_t now = esp_timer_get_time();
+
+    // Unplausible Rohwerte gelten als Aussetzer, nicht als Beruehrung.
+    if (got && *point_num > 0 &&
+        (x[0] > WS7_TOUCH_RAW_X_MAX || y[0] > WS7_TOUCH_RAW_Y_MAX)) {
+        got = false;
+        *point_num = 0;
+    }
+
+    if (got && *point_num > 0) {
+        s_touch_last_x = x[0];
+        s_touch_last_y = y[0];
+        s_touch_last_us = now;
+        s_touch_down = true;
+        return true;
+    }
+
+#if WS7_TOUCH_HOLD_MS > 0
+    if (s_touch_down && (now - s_touch_last_us) < ((int64_t)WS7_TOUCH_HOLD_MS * 1000)) {
+        x[0] = s_touch_last_x;
+        y[0] = s_touch_last_y;
+        if (strength) strength[0] = 0;
+        *point_num = 1;
+        return true;
+    }
+#endif
+
+    s_touch_down = false;
+    return false;
+}
+
 // Der GT9271 des Waveshare-Panels meldet im Hochformat (Roh-X 0..720, Roh-Y 0..1280),
 // das Bild laeuft aber im Querformat 1280x720. Achsentausch (und bei Bedarf Spiegelung)
 // deshalb ueber den process_coordinates-Hook, der VOR den swap/mirror-Flags von
@@ -211,6 +338,10 @@ static void ws7_touch_map(esp_lcd_touch_handle_t tp, uint16_t *x, uint16_t *y,
         // Bild wird um 180 Grad gedreht ausgegeben -> Touch muss mitdrehen.
         px = (uint16_t)(LCD_H_RES - 1 - px);
         py = (uint16_t)(LCD_V_RES - 1 - py);
+#endif
+#if WS7_TOUCH_DEBUG
+        esp_rom_printf("[Touch] roh X=%u Y=%u -> Bild X=%u Y=%u\n",
+                       (unsigned)x[i], (unsigned)y[i], (unsigned)px, (unsigned)py);
 #endif
         x[i] = px;
         y[i] = py;
@@ -245,6 +376,10 @@ bool jc_board_bringup(void)
 {
     jc_backlight_init();
 
+#if JC_PANEL_TYPE == WS_PANEL_7H
+    ws7_i2c_bus_recover();          // haengenden Bus vom letzten Reset befreien
+#endif
+
     i2c_master_bus_config_t i2c_bus_conf = {
         .clk_source = I2C_CLK_SRC_DEFAULT,
         .sda_io_num = BSP_I2C_SDA,
@@ -258,12 +393,17 @@ bool jc_board_bringup(void)
     ws7_bridge_pre_init();
 #endif
 
+    // Reset-Grund mitschreiben: 1 = Kaltstart (Strom weg), alles andere ist ein Warmstart.
+    // Nur beim Warmstart behalten die Bausteine ihren alten Zustand - genau dann traten
+    // die schwarzen Bilder auf.
+    esp_rom_printf("[Panel] Startgrund: %d (1 = Kaltstart)\n", (int)esp_reset_reason());
+
     static esp_ldo_channel_handle_t phy_pwr_chan = NULL;
     esp_ldo_channel_config_t ldo_cfg = {
         .chan_id = BSP_MIPI_DSI_PHY_PWR_LDO_CHAN,
         .voltage_mv = BSP_MIPI_DSI_PHY_PWR_LDO_VOLTAGE_MV,
     };
-    esp_ldo_acquire_channel(&ldo_cfg, &phy_pwr_chan);
+    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_panel_io_handle_t io = NULL;
@@ -463,6 +603,12 @@ bool jc_board_bringup(void)
         esp_rom_printf("[Panel] Touch-Controller nicht gefunden - Anzeige laeuft ohne Bedienung.\n");
         tp_handle = NULL;
     }
+#if JC_PANEL_TYPE == WS_PANEL_7H
+    if (tp_handle) {                       // Aussetzer-Filter vor den Treiber haengen
+        s_touch_get_xy_orig = tp_handle->get_xy;
+        tp_handle->get_xy = ws7_touch_get_xy;
+    }
+#endif
 
     lvgl_port_interface_t interface =
         (dpi_config.flags.use_dma2d) ? LVGL_PORT_INTERFACE_MIPI_DSI_DMA

P4_Display_Firmware/config.h

@@ -126,7 +126,7 @@
 // -------------------------------------------------------------------------------------
 //  Firmware
 // -------------------------------------------------------------------------------------
-#define DISPLAY_FW_VERSION       "1.6.4"   // Firmware-Stand der P4-Display-Steuerung (Info-Seite)
+#define DISPLAY_FW_VERSION       "1.6.10"   // Firmware-Stand der P4-Display-Steuerung (Info-Seite)
 
 // -------------------------------------------------------------------------------------
 //  Protokoll

P4_Display_Firmware/lv_conf.h

@@ -91,7 +91,17 @@
  *====================*/
 
 /*Default display refresh, input device read and animation step period.*/
+/* Der Wert bestimmt nicht nur die Bildrate, sondern auch, wie oft der Touch abgetastet
+ * und wie fein eine Animation gerechnet wird. 33 ms sind 30 Schritte je Sekunde - beim
+ * Ziehen mit dem Finger deutlich als Stocken zu sehen. Das Waveshare-Panel laeuft mit
+ * 60 Hz, deshalb dort 16 ms; mehr als das Panel zeigen kann, bringt nichts. Kommt das
+ * Zeichnen nicht hinterher, wird es einfach seltener fertig - kaputt geht dabei nichts.
+ * Die JC-Panels bleiben bei 33 ms, damit sich ihr Verhalten nicht aendert. */
+#if JC_PANEL_TYPE == WS_PANEL_7H
+#define LV_DEF_REFR_PERIOD  16      /*[ms]*/
+#else
 #define LV_DEF_REFR_PERIOD  33      /*[ms]*/
+#endif
 
 /*Default Dot Per Inch. Used to initialize default sizes such as widgets sized, style paddings.
  *(Not so important, you can adjust it to modify default sizes and spaces)*/
@@ -905,14 +915,19 @@
 #define LV_USE_SNAPSHOT 0
 
 /*1: Enable system monitor component*/
+/* Nur fuer die Messeinblendung des Waveshare-Panels, siehe WS7_PERF_MONITOR. */
+#if (JC_PANEL_TYPE == WS_PANEL_7H) && WS7_PERF_MONITOR
+#define LV_USE_SYSMON   1
+#else
 #define LV_USE_SYSMON   0
+#endif
 #if LV_USE_SYSMON
     /*Get the idle percentage. E.g. uint32_t my_get_idle(void);*/
     #define LV_SYSMON_GET_IDLE lv_timer_get_idle
 
     /*1: Show CPU usage and FPS count
      * Requires `LV_USE_SYSMON = 1`*/
-    #define LV_USE_PERF_MONITOR 0
+    #define LV_USE_PERF_MONITOR 1
     #if LV_USE_PERF_MONITOR
         #define LV_USE_PERF_MONITOR_POS LV_ALIGN_BOTTOM_RIGHT
 

P4_Display_Firmware/pins_config.h

@@ -16,19 +16,22 @@
 //  1280x720 ist gegenueber den JC-Panels ein Vielfaches an Pixeln. Die drei Schalter
 //  senken die Last und lassen sich einzeln zurueckstellen, falls einer Aerger macht.
 //
-//  WS7_COLOR_BITS      16 = RGB565: halbe Datenmenge je Bild, dafuer 65.000 statt
-//                           16,7 Mio. Farben (sichtbar nur in den dunklen Verlaeufen der
-//                           Rundanzeige). 24 = RGB888 wie bis 1.6.3.
-//                           ACHTUNG: Nimmt die Bridge des Panels kein RGB565 an, bleibt
-//                           das Bild falschfarbig oder aus -> wieder auf 24 stellen.
+//  WS7_COLOR_BITS      24 = RGB888. GEPRUEFT: Die Bridge dieses Panels nimmt KEIN RGB565
+//                           an - mit 16 kommt kein brauchbares Bild. Einstellbar bleibt es
+//                           nur, weil andere Waveshare-Panels an derselben Firmware
+//                           haengen koennten.
 //  WS7_PARTIAL_REFRESH  1 = nur geaenderte Bildbereiche neu zeichnen statt jedes Mal des
-//                           gesamten Bildes. Groesster Tempogewinn. Am 4,3-Zoll-Panel
-//                           hatte dieser Modus geflackert; tritt das hier auch auf, auf 0.
+//                           gesamten Bildes (LVGL-Direct-Mode, zwei wechselnde Puffer).
+//                           Groesster Tempogewinn, besonders waehrend eines Bezugs: Die
+//                           Zeitanzeige laeuft zehnmal je Sekunde weiter, und mit 0 wuerde
+//                           dafuer jedes Mal das ganze Bild neu gezeichnet. Am Geraet
+//                           geprueft, kein Flackern - am 4,3-Zoll-Panel hatte derselbe
+//                           Modus dagegen geflackert.
 //  WS7_PARALLEL_RENDER  1 = LVGL zeichnet mit zwei Threads auf beiden Prozessorkernen.
 //                           Kostet zwei zusaetzliche Task-Stacks im knappen internen RAM;
 //                           startet der P4 nicht mehr, hier zuerst auf 0 stellen.
 // -------------------------------------------------------------------------------------
-#define WS7_COLOR_BITS        16
+#define WS7_COLOR_BITS        24
 #define WS7_PARTIAL_REFRESH    1
 #define WS7_PARALLEL_RENDER    1
 
@@ -123,6 +126,20 @@
 #define WS7_TOUCH_MIRROR_RAW_X 1
 #define WS7_TOUCH_MIRROR_RAW_Y 0
 
+// Zeigt unten rechts Bilder je Sekunde und Prozessorlast ein. Nur zum Messen gedacht:
+// Der Wert sagt, ob sich weitere Muehe beim Zeichnen ueberhaupt lohnt.
+#define WS7_PERF_MONITOR 0
+
+// Zum Einmessen des Touch: 1 gibt jeden Beruehrpunkt roh und umgerechnet auf der
+// seriellen Konsole aus. Danach wieder auf 0, sonst laeuft das Log dauernd mit.
+#define WS7_TOUCH_DEBUG 0
+
+// So lange gilt eine Beruehrung nach dem letzten gueltigen Messwert noch als anliegend.
+// Ueberbrueckt Aussetzer des GT9271, die sonst als Loslassen durchgehen und einen Tipp in
+// mehrere Klicks zerlegen. 0 schaltet die Ueberbrueckung ab. Nicht zu gross waehlen: Der
+// Wert verzoegert das Loslassen und daempft dadurch den Schwung beim Wischen.
+#define WS7_TOUCH_HOLD_MS 40
+
 #elif JC_PANEL_TYPE == JC_PANEL_70
 
 #define LCD_H_RES 1024

P4_Display_Firmware/ui.cpp

@@ -453,7 +453,11 @@ 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));
     lv_obj_set_pos(p, 0, 0);
-    lv_obj_set_style_bg_opa(p, LV_OPA_TRANSP, 0);
+    // Deckend in der Hintergrundfarbe statt durchsichtig. Sieht identisch aus - die Ebenen
+    // darunter haben dieselbe Farbe -, erspart LVGL beim Scrollen aber, sie mitzuzeichnen
+    // und durchzumischen.
+    lv_obj_set_style_bg_color(p, COL_BG, 0);
+    lv_obj_set_style_bg_opa(p, LV_OPA_COVER, 0);
     lv_obj_set_style_border_width(p, 0, 0);
     lv_obj_set_style_pad_all(p, 8, 0);
     lv_obj_set_style_pad_row(p, 6, 0);
@@ -3171,6 +3175,11 @@ void ui_init(ProtocolClient* client) {
 
     lv_obj_t* burger = th_button(header, LV_SYMBOL_LIST, COL_CARD2, COL_TEXT);
     lv_obj_set_size(burger, 48, 36);
+    // Der Knopf bleibt optisch klein, faengt Tipps aber auch knapp daneben ab. 48x36 Pixel
+    // sind auf dem 7-Zoll-Panel ein sehr kleines Ziel; gemessen wurde ein Fehlgriff von
+    // rund 10 Pixeln. Die Vergroesserung wirkt nur innerhalb der Kopfleiste - darunter
+    // beschneidet sie die Leiste selbst, sodass dem Seiteninhalt nichts weggenommen wird.
+    lv_obj_set_ext_click_area(burger, 20);
     lv_obj_add_event_cb(burger, hamburger_cb, LV_EVENT_CLICKED, nullptr);
 
     g_hdrTitle = lv_label_create(header);