fix(P4): Schwarzes Bild nach Reset am Waveshare-Panel (1.6.11)

Nach einem Reset des P4 blieb das Bild schwarz, die Hintergrundbeleuchtung
brannte; nur direkt nach dem Flashen lief es.

Die Bridge des Panels hat keine Reset-Leitung und hängt an Dauerstrom. Ein
Reset des P4 setzt sie nicht zurück - ihre Register standen noch auf den
Werten der vorigen Sitzung, während der P4 seine DSI-Strecke komplett neu
aufbaut. Beim Flashen wird die Versorgung getrennt, deshalb trat der Fehler
dort nicht auf.

ws7_bridge_pre_init() schaltet die Bridge jetzt zuerst definiert ab
(AB/AA, AD, dann AC/C2/C0), wartet WS7_BRIDGE_RESET_MS und schreibt erst
danach die Grundinitialisierung.

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,41 @@
+Version 1.6.11:
+- Fehlerbehebung: Nach einem Reset des P4 blieb das Bild schwarz, während die
+  Hintergrundbeleuchtung brannte; erst ein erneutes Flashen half. Die Bridge des
+  Waveshare-Panels hat keine Reset-Leitung und hängt an Dauerstrom - ein Reset des P4
+  setzt sie also nicht zurück, ihre Register standen noch auf den Werten der vorigen
+  Sitzung. Der P4 baut seine DSI-Strecke aber komplett neu auf, und ein frischer DSI-Start
+  auf einer bereits laufenden Bridge liefert kein Bild. Beim Flashen wird die Versorgung
+  getrennt, deshalb war der Fehler dort nicht zu sehen.
+- Der Start schaltet die Bridge jetzt zuerst definiert ab (Beleuchtung, dann Ausgabe,
+  dann Grundregister), wartet WS7_BRIDGE_RESET_MS und fährt sie erst danach hoch. Der
+  Wert steht in pins_config.h (Vorgabe 120 ms) und ist die erste Stellschraube, falls
+  nach einem Reset doch einmal ein schwarzes Bild auftaucht.
+
+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,

P4_Display_Firmware/README.md

@@ -67,6 +67,11 @@ Das Panel hängt hinter einer Waveshare-Bridge und braucht **keinen** Hersteller
 - Reihenfolge: Bridge-Grundinitialisierung (`C0`/`C2`/`AC`) → DSI → DCS `MADCTL`/`Sleep Out`/
   `Display On` → DPI-Start → **erst dann** Hintergrundbeleuchtung. Wird das Licht früher
   eingeschaltet, blitzt beim Start Bildrauschen auf.
+- **Nach einem Reset:** Die Bridge hat keine Reset-Leitung und hängt an Dauerstrom — ein
+  Reset des P4 setzt sie **nicht** zurück. Der Bringup schaltet sie deshalb zuerst
+  definiert ab, wartet `WS7_BRIDGE_RESET_MS` und fährt sie dann neu hoch. Ohne das trifft
+  ein frischer DSI-Start auf eine noch laufende Bridge: Bild schwarz, Beleuchtung an.
+  Nach dem Flashen fällt das nicht auf, weil dabei die Versorgung getrennt wird.
 - **Einbaulage:** `WS7_ROTATE_180` in [`pins_config.h`](pins_config.h) dreht Bild **und**
   Touch um 180°, falls das Panel auf dem Kopf montiert wird. Vorgabe ist **0** — in der
   jetzigen Einbaulage steht das Bild richtig herum. Die Drehung macht der

P4_Display_Firmware/board_bringup.c

@@ -29,6 +29,7 @@
 #include "esp_cache.h"
 #include "esp_heap_caps.h"
 #include "esp_rom_sys.h"
+#include "esp_timer.h"
 #include "esp_ldo_regulator.h"
 #include "esp_lcd_panel_io.h"
 #include "esp_lcd_panel_ops.h"
@@ -109,8 +110,16 @@ static void ws7_bridge_write(uint8_t reg, uint8_t value)
     i2c_master_transmit(s_bridge_handle, buf, sizeof(buf), 100);
 }
 
-// Bridge am I2C-Bus anmelden und Grundinitialisierung schreiben. Muss VOR dem
-// DSI-Bringup laufen; die Hintergrundbeleuchtung bleibt dabei noch aus.
+// Bridge in einen definierten Grundzustand bringen und neu initialisieren.
+//
+// WICHTIG - der Grund fuer das Abschalten am Anfang: Die Bridge hat keine Reset-Leitung
+// und haengt an Dauerstrom. Ein Reset des P4 setzt sie also NICHT zurueck; ihre Register
+// stehen noch so, wie die vorige Sitzung sie hinterlassen hat. Der P4 baut seine
+// DSI-Strecke aber komplett neu auf. Trifft ein frischer DSI-Start auf eine bereits
+// laufende Bridge, bleibt das Bild schwarz, waehrend die Hintergrundbeleuchtung brennt.
+// Nach dem Flashen faellt das nicht auf, weil dabei die Versorgung getrennt wird.
+//
+// Deshalb erst ausschalten (Beleuchtung, dann Bridge), kurz warten, dann normal hochfahren.
 static void ws7_bridge_pre_init(void)
 {
     i2c_device_config_t dev_cfg = {
@@ -122,9 +131,22 @@ static void ws7_bridge_pre_init(void)
         s_bridge_handle = NULL;
         return;
     }
+
+    // --- definierter Aus-Zustand (umgekehrte Reihenfolge zum Einschalten) ---
+    ws7_bridge_write(0xAB, 0xFF);   // Helligkeit 0
+    ws7_bridge_write(0xAA, 0x01);   // uebernehmen
+    ws7_bridge_write(0xAD, 0x00);   // Bridge-Ausgabe aus
+    vTaskDelay(pdMS_TO_TICKS(20));
+    ws7_bridge_write(0xAC, 0x00);
+    ws7_bridge_write(0xC2, 0x00);
+    ws7_bridge_write(0xC0, 0x00);
+    vTaskDelay(pdMS_TO_TICKS(WS7_BRIDGE_RESET_MS));
+
+    // --- Grundinitialisierung wie im Linux-Treiber ---
     ws7_bridge_write(0xC0, 0x01);
     ws7_bridge_write(0xC2, 0x01);
     ws7_bridge_write(0xAC, 0x01);
+    vTaskDelay(pdMS_TO_TICKS(20));
 }
 
 void jc_backlight_init(void)
@@ -184,6 +206,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
@@ -467,6 +546,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.8"   // Firmware-Stand der P4-Display-Steuerung (Info-Seite)
+#define DISPLAY_FW_VERSION       "1.6.11"   // Firmware-Stand der P4-Display-Steuerung (Info-Seite)
 
 // -------------------------------------------------------------------------------------
 //  Protokoll

P4_Display_Firmware/lv_conf.h

@@ -915,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

@@ -20,20 +20,19 @@
 //                           an - mit 16 kommt kein brauchbares Bild. Einstellbar bleibt es
 //                           nur, weil andere Waveshare-Panels an derselben Firmware
 //                           haengen koennten.
-//  WS7_PARTIAL_REFRESH  0 = Vollbild neu zeichnen. GEPRUEFT: Mit 1 (nur geaenderte
-//                           Bereiche, zwei wechselnde Bildpuffer) laesst sich der
-//                           Menue-Vorhang nicht mehr oeffnen. Er ist halbtransparent und
-//                           wird in den gerade aktiven Puffer eingemischt; die naechste
-//                           Zustandsmeldung der S3 zeichnet in den anderen, der ihn nie
-//                           bekommen hat. Am 4,3-Zoll-Panel hatte derselbe Modus
-//                           geflackert. Nutzbar erst, wenn der LVGL-Port die geaenderten
-//                           Bereiche zwischen beiden Puffern abgleicht.
+//  WS7_PARTIAL_REFRESH  1 = nur geaenderte Bildbereiche neu zeichnen statt jedes Mal des
+//                           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        24
-#define WS7_PARTIAL_REFRESH    0
+#define WS7_PARTIAL_REFRESH    1
 #define WS7_PARALLEL_RENDER    1
 
 #define EXAMPLE_LVGL_PORT_TASK_MAX_DELAY_MS 500  //range 2 to 2000
@@ -115,6 +114,12 @@
 // nicht bekannt; die 180-Grad-Drehung macht deshalb der PPA (WS7_ROTATE_180).
 #define WS7_MADCTL           0x00
 
+// So lange bleibt die Bridge beim Start stromlos geschaltet, bevor sie neu
+// initialisiert wird. Sie hat keine Reset-Leitung und ueberlebt einen Reset des P4;
+// ohne diese Pause trifft ein frischer DSI-Start auf eine noch laufende Bridge und das
+// Bild bleibt schwarz. Bei schwarzem Bild nach Reset zuerst diesen Wert erhoehen.
+#define WS7_BRIDGE_RESET_MS  120
+
 // Der GT9271 meldet im Hochformat: Roh-X 0..720, Roh-Y 0..1280. Fuer das Querformat
 // werden die Achsen getauscht. Die Spiegelung geschieht VOR dem Tausch, also noch auf den
 // Rohwerten: Roh-X wird zur Bildschirmhoehe, Roh-Y zur Bildschirmbreite.
@@ -127,10 +132,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
@@ -139,7 +154,7 @@
 #define LCD_RST 27
 #define LCD_LED 23
 
-#else
+#elsealle
 
 #define LCD_H_RES 480
 #define LCD_V_RES 800

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);