fix(P4): Tipp auf einen Knopf löst nicht mehr mehrfach aus (1.6.9)

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
Abtastrate mit 1.6.8 von 33 auf 16 ms gestiegen ist, wurde daraus sichtbar
Drücken-Loslassen-Drücken-Loslassen.

Vor den Treiber ist jetzt ein eigener get_xy-Aufsatz gesetzt: Er verwirft
unplausible Rohwerte und hält den letzten gültigen Punkt noch
WS7_TOUCH_HOLD_MS lang (Vorgabe 40 ms). Der Aufsatz wird zur Laufzeit in das
Touch-Handle eingehängt, die Herstellerdateien bleiben unverändert.

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,35 @@
+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.

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"
@@ -184,6 +185,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 +525,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.6"   // Firmware-Stand der P4-Display-Steuerung (Info-Seite)
+#define DISPLAY_FW_VERSION       "1.6.9"   // 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)*/

P4_Display_Firmware/pins_config.h

@@ -33,7 +33,7 @@
 //                           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
@@ -131,6 +131,12 @@
 // 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

@@ -3171,6 +3171,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);