fix(P4): I2C-Störimpulsfilter - Ursache des schwarzen Bildes (1.6.16)

Der I2C-Scan vor und nach dem Panel-Start war entscheidend:

  vor:   0x14 (Touch)  0x18 (Audio)  0x45 (Bridge)
  nach:  0x18

Es fallen genau die beiden Bausteine weg, die am Displaykabel hängen; der
Audio-Baustein auf der Platine bleibt erreichbar, der Bus arbeitet also.
Sobald die DSI-Ausgabe läuft, stören deren Signale auf das Flachbandkabel
ein. Die Bridge liess sich danach nicht mehr ansprechen
(ESP_ERR_INVALID_STATE), das Panel blieb dunkel.

i2c_master_bus_config_t bekommt für WS_PANEL_7H jetzt glitch_ignore_cnt = 7
und enable_internal_pullup - beides setzt auch das Waveshare-Beispiel. In
unserem Bringup fehlte es, weil die Bus-Einrichtung von den JC-Panels
übernommen wurde, die kein Flachbandkabel dieser Länge haben.

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,31 @@
+Version 1.6.16:
+- Ursache des schwarzen Bildes gefunden. Der Start-Log zeigte es eindeutig: Vor dem
+  Panel-Start meldeten sich alle drei I2C-Bausteine (Touch 0x14, Audio 0x18, Bridge 0x45),
+  danach nur noch der Audio-Baustein. Es fielen also genau die beiden weg, die am
+  Displaykabel haengen - der Bus selbst arbeitete weiter. Sobald die DSI-Ausgabe laeuft,
+  stoeren deren schnelle Signale auf das lange Flachbandkabel ein, und ohne Filterung
+  bringen diese Stoerspitzen die I2C-Uebertragung aus dem Tritt. Die Bridge liess sich
+  daraufhin nicht mehr ansprechen, das Panel blieb dunkel.
+- Der I2C-Bus laeuft fuer das Waveshare-Panel jetzt mit Stoerimpulsfilter und internen
+  Abschlusswiderstaenden, so wie es auch das Waveshare-Beispiel tut. Beides fehlte, weil
+  die Bus-Einrichtung urspruenglich von den JC-Panels uebernommen wurde, die kein
+  Flachbandkabel dieser Laenge haben.
+- Das erklaert rueckwirkend auch, warum sich der Fehler so sprunghaft verhielt: Ob eine
+  Uebertragung durchkam, hing von der Stoerlage ab.
+
+Version 1.6.15:
+- Diagnose fuer das schwarze Bild, das der Log bisher nicht erklaeren konnte: Alle Schritte
+  liefen fehlerfrei durch, trotzdem blieb die Anzeige leer.
+- Die drei Weckbefehle an das Panel (MADCTL, Sleep Out, Display On) liefen bislang
+  ungeprueft durch und sind jetzt in die Fehlerausgabe einbezogen.
+- Neuer Schalter WS7_TEST_PATTERN in pins_config.h: zeigt statt der Oberflaeche
+  Prueffarbbalken an. Sie entstehen im DSI-Baustein selbst und benutzen weder Bildspeicher
+  noch LVGL. Damit laesst sich in einem Durchgang trennen, ob die Anzeigekette bis zum
+  Panel arbeitet oder ob der Fehler weiter oben liegt.
+- Das Ende des Hochlaufs wird gemeldet ("Bringup fertig" / "LVGL laeuft"). Bisher endete
+  die Ausgabe nach dem Panel-Start, und ein Haengenbleiben in LVGL war von einem
+  fehlenden Bild nicht zu unterscheiden.
+
 Version 1.6.14:
 - Der Touch-Controller wird jetzt erst angesprochen, nachdem die Bridge freigegeben und die
   Hintergrundbeleuchtung eingeschaltet ist. Der GT9271 haengt an derselben Versorgung und

P4_Display_Firmware/board_bringup.c

@@ -399,6 +399,16 @@ bool jc_board_bringup(void)
         .sda_io_num = BSP_I2C_SDA,
         .scl_io_num = BSP_I2C_SCL,
         .i2c_port = BSP_I2C_NUM,
+#if JC_PANEL_TYPE == WS_PANEL_7H
+        // Stoerimpulsfilter und interne Abschlusswiderstaende - beides setzt auch das
+        // Waveshare-Beispiel. Ohne den Filter brechen Bridge und Touch weg, sobald die
+        // DSI-Ausgabe laeuft: Deren schnelle Signale stoeren auf das lange Flachbandkabel
+        // ein, und ungefilterte Stoerspitzen bringen die I2C-Uebertragung aus dem Tritt.
+        // Der Audio-Baustein auf der Platine bleibt dabei erreichbar, weil er nicht am
+        // Displaykabel haengt - genau dieses Muster war im Start-Log zu sehen.
+        .glitch_ignore_cnt = 7,
+        .flags.enable_internal_pullup = true,
+#endif
     };
     i2c_new_master_bus(&i2c_bus_conf, &s_i2c_handle);
 
@@ -486,13 +496,23 @@ bool jc_board_bringup(void)
     {
         uint8_t zero = 0x00;
         uint8_t madctl = WS7_MADCTL;
-        esp_lcd_panel_io_tx_param(io, 0x36, &madctl, 1); // MADCTL
-        esp_lcd_panel_io_tx_param(io, 0x11, &zero, 1);   // Sleep Out
+        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));
-        esp_lcd_panel_io_tx_param(io, 0x29, &zero, 1);   // Display On
+        BSP_STEP(esp_lcd_panel_io_tx_param(io, 0x29, &zero, 1), "Panel-Befehl Display On");
         vTaskDelay(pdMS_TO_TICKS(20));
     }
     BSP_STEP(esp_lcd_panel_init(disp_panel), "DPI-Videoausgabe starten");
+#if WS7_TEST_PATTERN
+    // Prueffarbbalken: Sie entstehen im DSI-Baustein selbst und benutzen weder den
+    // Bildspeicher noch LVGL. Erscheinen sie, arbeitet die gesamte Anzeigekette bis zum
+    // Panel korrekt und der Fehler liegt weiter oben (Bildspeicher, LVGL, Oberflaeche).
+    // Bleibt das Bild schwarz, liegt es an DSI, Zeitbasis oder Panel.
+    BSP_STEP(esp_lcd_dpi_panel_set_pattern(disp_panel, MIPI_DSI_PATTERN_BAR_VERTICAL),
+             "Prueffarbbalken");
+    esp_rom_printf("[Panel] Pruefbild aktiv - erwartet werden senkrechte Farbbalken.\n");
+#endif
+
 #elif JC_PANEL_TYPE == JC_PANEL_70
     // ---------------- 7,0" JD9165 (1024x600) ----------------
     esp_lcd_dsi_bus_config_t bus_config = JD9165_PANEL_BUS_DSI_2CH_CONFIG();
@@ -638,7 +658,9 @@ bool jc_board_bringup(void)
     lvgl_port_interface_t interface =
         (dpi_config.flags.use_dma2d) ? LVGL_PORT_INTERFACE_MIPI_DSI_DMA
                                      : LVGL_PORT_INTERFACE_MIPI_DSI_NO_DMA;
+    esp_rom_printf("[Panel] Bringup fertig - LVGL wird gestartet\n");
     lvgl_port_init(disp_panel, tp_handle, interface);
+    esp_rom_printf("[Panel] LVGL laeuft\n");
     return true;
 }
 

P4_Display_Firmware/pins_config.h

@@ -145,6 +145,11 @@
 // auf I2C antwortet. Meldet sich 0x14 beim Start nicht, diesen Wert erhoehen.
 #define WS7_TOUCH_POWER_MS 120
 
+// Prueffarbbalken statt Oberflaeche anzeigen. Sie entstehen im DSI-Baustein selbst und
+// benutzen weder Bildspeicher noch LVGL. Damit laesst sich trennen, ob die Anzeigekette
+// bis zum Panel arbeitet (Balken sichtbar) oder nicht (schwarz). Nur zur Fehlersuche.
+#define WS7_TEST_PATTERN 0
+
 #elif JC_PANEL_TYPE == JC_PANEL_70
 
 #define LCD_H_RES 1024