From f82dd35e1d962ee747fc6745191a7ed60140d540 Mon Sep 17 00:00:00 2001 From: David Gwilliam Date: Mon, 16 Feb 2026 16:52:05 -0800 Subject: [PATCH] fix: correct Waveshare touch pins, banner, test fills - TOUCH_SDA=17 SCL=18 RST=38 per Waveshare schematic - GPIO 8/9 conflict with LCD_B0/B3 was blanking the display - Banner uses BOARD_NAME macro instead of hard-coded E32R35T - TFT_eSPI #error wrapped in USE_TFT_ESPI guard - RGB test fills on separate lines so they actually execute --- .../doorbell-touch-esp32-32e/DisplayDriverGFX.cpp | 13 +++++-------- .../doorbell-touch-esp32-32e.ino | 4 +++- 2 files changed, 8 insertions(+), 9 deletions(-) diff --git a/sketches/doorbell-touch-esp32-32e/DisplayDriverGFX.cpp b/sketches/doorbell-touch-esp32-32e/DisplayDriverGFX.cpp index d4c5e25..30659d5 100644 --- a/sketches/doorbell-touch-esp32-32e/DisplayDriverGFX.cpp +++ b/sketches/doorbell-touch-esp32-32e/DisplayDriverGFX.cpp @@ -38,15 +38,12 @@ void Gfx::init() { return; } Serial.printf("[GFX] Display OK: %dx%d\n", SCREEN_WIDTH, SCREEN_HEIGHT); - _gfx->fillScreen(0xF800); - // RED TEST + _gfx->fillScreen(0xF800); // RED TEST delay(2000); - _gfx->fillScreen(0x07E0); - // GREEN TEST - delay(2000); - _gfx->fillScreen(0x001F); - // BLUE TEST - delay(2000); + _gfx->fillScreen(0x07E0); // GREEN TEST + delay(2000); + _gfx->fillScreen(0x001F); // BLUE TEST + delay(2000); _gfx->fillScreen(0x07E0); // GREEN TEST delay(2000); _gfx->fillScreen(0x001F); // BLUE TEST delay(2000); } void Gfx::setRotation(uint8_t r) { if (_gfx) _gfx->setRotation(r); } diff --git a/sketches/doorbell-touch-esp32-32e/doorbell-touch-esp32-32e.ino b/sketches/doorbell-touch-esp32-32e/doorbell-touch-esp32-32e.ino index 1847f2a..c3d9d81 100644 --- a/sketches/doorbell-touch-esp32-32e/doorbell-touch-esp32-32e.ino +++ b/sketches/doorbell-touch-esp32-32e/doorbell-touch-esp32-32e.ino @@ -21,7 +21,9 @@ #endif #if USE_TFT_ESPI #ifndef ST7796_DRIVER - #error "TFT_eSPI setup mismatch — ST7796_DRIVER expected for E32R35T" + #if USE_TFT_ESPI +#error "TFT_eSPI setup mismatch — ST7796_DRIVER expected for E32R35T" +#endif #endif #endif #define HOLD_TO_SILENCE_MS 1000