From 1ed4cf724b786bc6e14408033e2365f32c0a4de3 Mon Sep 17 00:00:00 2001 From: Wesley Ellis Date: Sat, 13 Jun 2026 14:54:36 -0400 Subject: [PATCH] fix(heltec_t114): correct swapped P0.X comments on Wire1 I2C pins The PIN_WIRE1_SDA/SCL comments were swapped and wrong: pin 7 was labelled P0.8 and pin 8 labelled P0.7. Per g_ADigitalPinMap (1:1 for pins >= 2), Arduino pin 7 = P0.07 and pin 8 = P0.08. Wire1 is the environment-sensor bus (ENV_PIN_SDA/SCL), so the misleading comments can lead users to wire INA219/other sensors with SDA/SCL reversed, which prevents the sensor from responding and can hang the I2C scan in EnvironmentSensorManager::begin() at boot. Comment-only change; no electrical/behavioral change. Co-Authored-By: Claude Opus 4.8 --- variants/heltec_t114/variant.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/variants/heltec_t114/variant.h b/variants/heltec_t114/variant.h index bfb4484d13..9ab90c255d 100644 --- a/variants/heltec_t114/variant.h +++ b/variants/heltec_t114/variant.h @@ -61,8 +61,8 @@ #define PIN_WIRE_SDA (26) // P0.26 #define PIN_WIRE_SCL (27) // P0.27 -#define PIN_WIRE1_SDA (7) // P0.8 -#define PIN_WIRE1_SCL (8) // P0.7 +#define PIN_WIRE1_SDA (7) // P0.07 +#define PIN_WIRE1_SCL (8) // P0.08 //////////////////////////////////////////////////////////////////////////////// // SPI pin definition