fix(P4): Menü-Knopf fängt Tipps auch knapp daneben ab (1.6.7)

Eine Messung mit WS7_TOUCH_DEBUG hat gezeigt, dass die Touch-Zuordnung stimmt:
Der Tipp auf den Menü-Knopf landete bei X=66/Y=27, der Knopf liegt aber bei
X 8..56. Mit 48x36 Pixeln ist er auf dem 7-Zoll-Panel ein sehr kleines Ziel.

lv_obj_set_ext_click_area vergrößert die Trefferfläche um 20 Pixel, ohne den
Knopf optisch zu ändern. Die Vergrößerung wirkt nur innerhalb der Kopfleiste,
weil diese ihre Kinder beschneidet - dem Seiteninhalt darunter wird nichts
weggenommen, und bei offenem Menü liegt ohnehin der Vorhang darüber.

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

CLAUDE.md

@@ -11,17 +11,17 @@ für zwei Mikrocontroller, die per UART (230400 8N1, JSON-Zeilen) verbunden sind
   Änderungshistorie: `Changelog.txt` (Root), Version steht in `String version = "…"`.
 
 - **ESP32-P4 – UART-Touch-Display (spiegelt den S3-Zustand).**
-  Ordner: `JC_Display_Firmware/` (Hauptsketch `JC_Display_Firmware.ino`,
+  Ordner: `P4_Display_Firmware/` (Hauptsketch `P4_Display_Firmware.ino`,
   UI in `ui.cpp`/`ui.h`, UART-Protokoll in `protocol_client.*`, LVGL 9.x).
   Hat **kein eigenes WLAN** (der ESP32-C6-Co-Prozessor ist ungenutzt); alle Daten
   kommen über UART vom S3. Eigene Version `DISPLAY_FW_VERSION` +
-  `JC_Display_Firmware/Changelog.txt`.
+  `P4_Display_Firmware/Changelog.txt`.
 
 Wenn ich „S3/Controller/Maschine" sage, ist `Dual_PID_FastHeatUp.ino` gemeint;
-„P4/Display" ist der Ordner `JC_Display_Firmware/`.
+„P4/Display" ist der Ordner `P4_Display_Firmware/`.
 
 **Nicht anfassen:** Der Ordner `Sicherungen/` enthält nur datierte Versions-Backups
-(`.ino`) — niemals dort editieren. Unter `JC_Display_Firmware/JC4880P443C_I_W/` liegen
+(`.ino`) — niemals dort editieren. Unter `P4_Display_Firmware/JC4880P443C_I_W/` liegen
 reine Hersteller-Demos/Beispiele (Board-Support), ebenfalls nicht als Projektcode behandeln.
 
 # Agent Instructions
@@ -33,8 +33,8 @@ reine Hersteller-Demos/Beispiele (Board-Support), ebenfalls nicht als Projektcod
 ## Git-Workflow
 
 - Host: Gitea `git.mueller.black`, Repo `thomas/Dual-PID`, User `thomas`, CLI `tea`.
-- Aenderungen immer ueber Feature-Branch + Merge-Request (`tea pr create`), nie direkt auf `main` committen oder pushen.
-- Pull/Merge-Requests dürfen immer selbstständig gemergt werden (Auto-Merge, kein grünes CI erforderlich, via `tea pr merge` oder Gitea-API); danach den Merge kurz melden.
+- Aenderungen immer ueber Feature-Branch + Merge-Request, nie direkt auf `main` committen oder pushen.
+- PR anlegen **und** mergen bevorzugt über die **Gitea-REST-API** (PowerShell `Invoke-RestMethod` auf `https://git.mueller.black/api/v1/repos/thomas/Dual-PID/pulls[/<nr>/merge]`, Token aus `C:\Users\Thomas\AppData\Local\tea\config.yml`, nur in Variable, nie ausgeben), da die `tea`-CLI gegen `git.mueller.black` oft hängt und nie zurückkehrt. `tea pr create`/`tea pr merge` nur als Ausweichweg. PRs dürfen selbstständig gemergt werden (Auto-Merge, kein grünes CI erforderlich); danach den Merge kurz melden.
 - Kein Force-Push, kein History-Rewrite auf geteilten Branches.
 - Commit-Identitaet bleibt `raw-designs` / `Thomas@raw-designs.de` (globale Git-Config, nicht aendern).
 - Commit-Messages im Conventional-Commits-Format (feat/fix/docs/refactor/...).

Doku/P4-Display_UART-Protokoll.md

@@ -1,4 +1,4 @@
-# JC4880P443C-I-W Touch-Display — UART-Protokoll & Architektur
+# P4-Touch-Display — UART-Protokoll & Architektur
 
 Status: Entwurf v1.0 der Doku · Firmware-Protokollversion: **1** (vorhanden, abgeschaltet) ·
 Ziel-Protokollversion: **2** (geplante Erweiterungen)
@@ -16,11 +16,11 @@ Bewusste Designentscheidung (siehe Projektbesprechung):
 
 - Das **Webinterface bleibt auf der Hauptplatine** (ESP32-S3). Es bleibt unverändert
   die zentrale Steuerungs- und Konnektivitätsinstanz.
-- Das **JC-Display ist ein vollwertiges Touch-Bediengerät**, das über UART angebunden
+- Das **P4-Display ist ein vollwertiges Touch-Bediengerät**, das über UART angebunden
   ist und denselben Funktionsumfang bietet wie das Webinterface — aber **kein** eigenes
   WLAN/Webserver betreibt.
 - Das **OLED (SH1106G) bleibt optional** und wird wie bisher über ein **Compile-Flag**
-  gewählt (`ENABLE_DISPLAY`). Es ist unabhängig vom JC-Display.
+  gewählt (`ENABLE_DISPLAY`). Es ist unabhängig vom P4-Display.
 
 ```
   +----------------------------+            UART (GPIO43/44)            +---------------------------+
@@ -34,13 +34,13 @@ Bewusste Designentscheidung (siehe Projektbesprechung):
   |  - OLED (optional)         |                                       |   ST7701 / GT911 / ES8311 |
   +----------------------------+                                       +---------------------------+
 
-  * Der ESP32-C6 auf dem JC-Display (WLAN/BT) wird in diesem Projekt NICHT für die
+  * Der ESP32-C6 auf dem P4-Display (WLAN/BT) wird in diesem Projekt NICHT für die
     Maschinensteuerung benötigt. WLAN/Webinterface laufen weiterhin auf der S3.
 ```
 
 ### Rollenverteilung
 
-| Aufgabe | S3 (Hauptplatine) | P4 (JC-Display) |
+| Aufgabe | S3 (Hauptplatine) | P4 (P4-Display) |
 |---|---|---|
 | Echtzeit-Regelung (PID, SSR, Sicherheit) | **ja** | nein |
 | Sensoren (NTC, MAX6675, Waage) | **ja** | nein |
@@ -76,7 +76,7 @@ schickt Bedienwünsche als Kommandos. So bleiben Web-UI, OLED und Touch-Display
 ```
 
 > **Hinweis (bewusst akzeptiert):** GPIO43/44 sind beim ESP32-S3 zugleich die Standard-
-> Pins der seriellen Debug-Konsole (UART0). Solange das JC-Display dort angeschlossen ist,
+> Pins der seriellen Debug-Konsole (UART0). Solange das P4-Display dort angeschlossen ist,
 > steht die klassische serielle Debug-Ausgabe an diesen Pins nicht parallel zur Verfügung.
 > Debug/Logging erfolgt dann über USB-CDC bzw. das Webinterface. Diese Pins wurden
 > ausdrücklich beibehalten, weil sie auf der Platine herausgeführt sind.
@@ -519,7 +519,7 @@ Für ein vollwertiges 480×800-Touch-UI fehlen v1 noch einige Daten/Funktionen.
 ## 9. OLED-Koexistenz (Compile-Flag)
 
 - OLED bleibt über `ENABLE_DISPLAY` (Compile-Flag) wählbar — unverändert.
-- Das JC-Display wird über `TOUCH_UART_ENABLED` (Compile-Flag) aktiviert.
+- Das P4-Display wird über `TOUCH_UART_ENABLED` (Compile-Flag) aktiviert.
 - **Geplante Änderung:** beide Flags strikt **unabhängig** halten, damit alle Kombinationen
   baubar sind:
 
@@ -562,7 +562,7 @@ Puffergrößen/Chunking, optionaler CRC + Handshake, `action`-Stringliste dokume
 **Löschen:** Nichts.
 
 **Fully-Kiosk bleibt erhalten** (Entscheidung) — als eigenständige, optionale Funktion
-neben dem JC-Display. Lastbewertung der S3:
+neben dem P4-Display. Lastbewertung der S3:
 - Einziger Aufruf `sendFullyKioskLoadUrl()` wird **nur bei Standby-Zustandswechsel**
   getriggert (`if (standbyModeActive != lastStandbyModeActive)`), **nicht periodisch**
   (wenige Male pro Tag).
@@ -586,10 +586,10 @@ neben dem JC-Display. Lastbewertung der S3:
 6. **[offen, optional]** CRC + ggf. Puffer-Feinjustage (§8.3) — bewusst nicht als toter,
    ungetesteter Code eingebaut; bei Bedarf später.
 7. **[offen, durch dich]** `TOUCH_UART_ENABLED` auf 1 setzen + Build verifizieren, **sobald**
-   das JC-Display physisch an Serial1 (GPIO43/44) hängt. Bewusst noch **0**, damit die
+   das P4-Display physisch an Serial1 (GPIO43/44) hängt. Bewusst noch **0**, damit die
    serielle Debug-Konsole (GPIO43/44) im aktuellen Betrieb erhalten bleibt.
 
-### Phase B — P4-Seite (JC-Display, Projekt `JC_Display_Firmware/`)
+### Phase B — P4-Seite (P4-Display, Projekt `P4_Display_Firmware/`)
 Grundgerüst angelegt (Arduino-ESP32 P4 + LVGL 9 + ArduinoJson 7). Aufteilung:
 `config.h`, `machine_state.h`, `protocol_client.*`, `ui.*`, `display_hal.*`, `.ino`, `README.md`.
 
@@ -603,7 +603,7 @@ Grundgerüst angelegt (Arduino-ESP32 P4 + LVGL 9 + ArduinoJson 7). Aufteilung:
    (`display_hal.cpp`, Schalter `JC_USE_REAL_PANEL`): MIPI-DSI/ST7701, GT911 (I²C SDA=7/SCL=8),
    Backlight GPIO23, LCD-Reset GPIO5; LVGL-Port mit eigenem Task + Lock. Stub-Modus bleibt für
    Schreibtisch-Tests. Vendor-BSP-Dateien sind in den Sketch zu kopieren (siehe Projekt-README).
-5. **[festgelegt]** UART-Pins am JC-Display: **P4 GPIO50 (TX) / GPIO51 (RX)** vom Expand-IO-Header
+5. **[festgelegt]** UART-Pins am P4-Display: **P4 GPIO50 (TX) / GPIO51 (RX)** vom Expand-IO-Header
    JP1 (Schaltplan `4_USB&IO.png`). Gegenseite S3 unverändert GPIO43/44.
 6. **[offen]** Weitere Screens: Einstellungen (Brew/Service/Sensor), Profil-Auswahl/-Editor,
    PID/AutoTune-Bedienung; Ausrichtung Hochformat (Rotation 0 in `pins_config.h`).

JC_Display_Firmware/pins_config.h

@@ -1,96 +0,0 @@
-#pragma once
-
-#include "config.h"   // liefert JC_PANEL_TYPE (C-sicher, keine Rueckabhaengigkeit)
-
-#define EXAMPLE_LVGL_PORT_TASK_MAX_DELAY_MS 500  //range 2 to 2000
-#define EXAMPLE_LVGL_PORT_TASK_MIN_DELAY_MS 5     //range 1 to 100
-#define EXAMPLE_LVGL_PORT_TASK_PRIORITY     4
-#define EXAMPLE_LVGL_PORT_TASK_STACK_SIZE_KB  32  //KB (erhoeht: komplexe UI, Listen-Rendering, JSON in Callbacks)
-#define EXAMPLE_LVGL_PORT_TASK_CORE         -1  //range -1 to 1
-#define EXAMPLE_LVGL_PORT_TICK              2   //ragne 1 to 100
-
-#define EXAMPLE_LVGL_PORT_AVOID_TEAR_ENABLE   1
-
-#ifdef  EXAMPLE_LVGL_PORT_AVOID_TEAR_ENABLE
-// 2 = Triple-Buffer + LVGL Full-Refresh: zeichnet bei jeder Aenderung das komplette
-//     Bild, daher zeigen statische Flaechen (z.B. Tab-Leiste) nie alten Bufferinhalt.
-//     (Mode 3 = Direct-Mode flackert bei nie neu gezeichneten Bereichen.)
-//     Bei Rotation 90 nutzen beide Modi ohnehin 3 Framebuffer -> kein Mehrverbrauch.
-#define EXAMPLE_LVGL_PORT_AVOID_TEAR_MODE   2   //range 1 to 3
-
-#if JC_PANEL_TYPE == WS_PANEL_7H
-// Waveshare 7" (H): Panel ist nativ 1280x720 Querformat -> keine Rotation, kein PPA noetig.
-#define EXAMPLE_LVGL_PORT_ROTATION_DEGREE_ 0    // 0,90,180,270
-#define EXAMPLE_LVGL_PORT_PPA_ROTATION_ENABLE 0
-#elif JC_PANEL_TYPE == JC_PANEL_70
-// 7,0" JD9165: Panel ist nativ 1024x600 Querformat -> keine Rotation, kein PPA noetig.
-#define EXAMPLE_LVGL_PORT_ROTATION_DEGREE_ 0    // 0,90,180,270
-#define EXAMPLE_LVGL_PORT_PPA_ROTATION_ENABLE 0
-#else
-// 4,3" ST7701: Panel ist physisch 480x800 Hochformat.
-#define EXAMPLE_LVGL_PORT_ROTATION_DEGREE_ 270  // 0,90,180,270 (270 = Querformat 800x480, um 180 Grad gedreht vs. 90)
-#define EXAMPLE_LVGL_PORT_PPA_ROTATION_ENABLE 1
-#endif
-#endif
-
-
-// LCD_H_RES/LCD_V_RES = NATIVE Panel-Aufloesung (vor der LVGL-Rotation).
-#if JC_PANEL_TYPE == WS_PANEL_7H
-
-#define LCD_H_RES 1280
-#define LCD_V_RES 720
-
-// Das Waveshare-Panel hat weder einen Reset-Pin noch eine PWM-Hintergrundbeleuchtung:
-// Reset macht die Bridge selbst, die Helligkeit laeuft ueber deren I2C-Register.
-#define LCD_RST (-1)
-#define LCD_LED (-1)
-
-// ---- MIPI-DSI-Parameter des Waveshare 7inch DSI LCD (H) -----------------------------
-// Uebernommen aus dem funktionierenden Funktionstest-Sketch. Wichtig ist die Legacy-
-// Referenztaktquelle PLL_F20M: nur damit liefert der ESP32-P4 Rev. 1.3 / ECO2 ein Bild.
-// Das Panel akzeptiert nur RGB888 (24 Bit) - deshalb rechnet LVGL bei diesem Panel
-// ebenfalls mit 24 Bit (lv_conf.h).
-#define WS7_DSI_LANES        2
-#define WS7_DSI_LANE_MBPS    1250
-#define WS7_DPI_CLOCK_MHZ    80    // 1472 x 912 Gesamtraster -> ca. 60 Hz
-#define WS7_HSYNC_BACK       64
-#define WS7_HSYNC_PULSE      64
-#define WS7_HSYNC_FRONT      64
-#define WS7_VSYNC_BACK       64
-#define WS7_VSYNC_PULSE      64
-#define WS7_VSYNC_FRONT      64
-
-// Bridge (Helligkeit/Panel-Freigabe) und Touch-Controller am selben I2C-Bus wie GT911.
-#define WS7_BRIDGE_I2C_ADDR  0x45
-#define WS7_TOUCH_I2C_ADDR   0x14   // GT9271
-
-// Der GT9271 meldet im Hochformat: Roh-X 0..720, Roh-Y 0..1280. Fuer das Querformat
-// werden die Achsen getauscht. Zeigt der Cursor spiegelverkehrt, hier die passende
-// Achse auf 1 setzen (die Spiegelung geschieht VOR dem Tausch, auf den Rohwerten).
-#define WS7_TOUCH_RAW_X_MAX  720
-#define WS7_TOUCH_RAW_Y_MAX  1280
-#define WS7_TOUCH_MIRROR_RAW_X 0
-#define WS7_TOUCH_MIRROR_RAW_Y 0
-
-#elif JC_PANEL_TYPE == JC_PANEL_70
-
-#define LCD_H_RES 1024
-#define LCD_V_RES 600
-
-#define LCD_RST 27
-#define LCD_LED 23
-
-#else
-
-#define LCD_H_RES 480
-#define LCD_V_RES 800
-
-#define LCD_RST 5
-#define LCD_LED 23
-
-#endif
-
-#define TP_I2C_SDA 7
-#define TP_I2C_SCL 8
-#define TP_RST -1
-#define TP_INT -1

P4_Display_Firmware/Changelog.txt

@@ -1,3 +1,79 @@
+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
+  das komplette Bild in 24 Bit neu gezeichnet.
+  * Farbtiefe RGB565 statt RGB888 (WS7_COLOR_BITS): halbiert die Datenmenge je Bild.
+    Feinere Farbabstufungen entfallen, sichtbar allenfalls in den dunklen Verläufen der
+    Rundanzeige - die JC-Panels laufen seit jeher so.
+  * Nur geänderte Bildbereiche neu zeichnen statt des ganzen Bildes
+    (WS7_PARTIAL_REFRESH, LVGL-Direct-Mode mit zwei Puffern). Eine sich ändernde
+    Temperaturanzeige kostet damit ein paar tausend statt 921.600 Pixel.
+  * LVGL zeichnet mit zwei Threads auf beiden Prozessorkernen (WS7_PARALLEL_RENDER).
+- Jeder der drei Schalter steht in pins_config.h und lässt sich einzeln zurückstellen;
+  die README nennt zu jedem, woran man erkennt, dass er zurück muss. Die JC-Panels sind
+  von allen dreien nicht betroffen.
+- Nebeneffekt: Der Framebuffer-Bedarf sinkt von rund 8,3 MB auf 3,7 MB PSRAM.
+
+Version 1.6.3:
+- Waveshare ESP32-P4-Pico: UART zur Hauptplatine liegt jetzt auf TX = GPIO20 und
+  RX = GPIO21 (vorher GPIO17/GPIO18). Die JC-Panels bleiben bei GPIO33/GPIO31.
+- Das Bild steht in der tatsächlichen Einbaulage richtig herum, die 180-Grad-Drehung aus
+  1.6.1 ist deshalb wieder abgeschaltet (WS7_ROTATE_180 = 0). Sie bleibt als Schalter
+  erhalten, falls das Panel später anders montiert wird.
+- Der Touch war dagegen oben/unten spiegelverkehrt: Der Digitizer sitzt gegenüber dem Bild
+  um die Hochachse gedreht. WS7_TOUCH_MIRROR_RAW_X steht dafür jetzt auf 1. Bild- und
+  Touch-Ausrichtung sind damit zwei getrennte Schalter, weil sie an diesem Panel eben
+  nicht zusammenfallen.
+
+Version 1.6.2:
+- Kommt das Panel nicht hoch, stürzt der P4 nicht mehr wortlos ab. Die Bringup-Schritte
+  melden ihren Fehler jetzt im Klartext auf der seriellen Konsole (auch bei abgeschaltetem
+  Core-Debug-Level), zusammen mit PSRAM-Größe, freiem PSRAM und dem tatsächlich benötigten
+  Framebuffer-Bedarf. Vorher endete ein fehlgeschlagener Panel-Aufbau in einem
+  Speicherauszug, aus dem die Ursache nicht hervorging.
+- Schlägt der Panel-Aufbau fehl, läuft die Firmware ohne Anzeige weiter, statt ein zweites
+  Mal abzustürzen: UART-Verbindung und OTA bleiben offen, die S3 kann also neu flashen.
+  Alle Oberflächenfunktionen prüfen dafür, ob das Dashboard überhaupt aufgebaut wurde.
+- Fehlerbehebung: Findet der Touch-Controller nicht statt, lieferte der Hersteller-Treiber
+  einen bereits freigegebenen Zeiger zurück, der ungeprüft weiterverwendet wurde. Das
+  Display läuft in diesem Fall jetzt einfach ohne Bedienung weiter.
+
+Version 1.6.1:
+- Waveshare 7" (H): Bild und Touch lassen sich um 180 Grad drehen, wenn das Panel auf dem
+  Kopf eingebaut ist. Schalter WS7_ROTATE_180 in pins_config.h, Vorgabe ist gedreht.
+  Die Drehung übernimmt der PPA-Grafikbeschleuniger des ESP32-P4 - die Software-Rotation
+  des LVGL-Ports rechnet nur in RGB565 und wäre für dieses Panel unbrauchbar.
+- Der Touch dreht mit, sonst läge die Bedienung spiegelbildlich zum Bild.
+
 Version 1.6.0:
 - Neu: Dieselbe Firmware läuft jetzt auch auf einem Waveshare ESP32-P4-Pico mit dem
   Waveshare 7inch DSI LCD (H) (1280x720). Umgeschaltet wird über „JC_PANEL_TYPE" in