Mirrors_Framework/esphome
1
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-05-21 17:39:00 +08:00
Code Issues Actions 13 Packages Projects Releases Wiki Activity

[esp32_touch] Fix zero readings on GPIO33/GPIO32 (channels 8/9)

Browse Source 
Work around ESP-IDF bug where touch_ll_enable_channel_mask() and
touch_ll_set_charge_speed() don't apply the channel 8/9 bit swap
required by the ESP32 hardware. The legacy driver's equivalent
functions (touch_ll_set_channel_mask, etc.) correctly applied
TOUCH_LL_BITS_SWAP, but the new unified API omits it.

This caused channels 8 (GPIO33) and 9 (GPIO32) to not be properly
enabled for measurement, resulting in reads always returning zero.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Jonathan Swoboda
2026-02-17 14:26:47 -05:00
parent cf2489c54f
commit 5818d03378
1 changed files with 19 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
esphome/components/esp32_touch/esp32_touch.cpp
+19
View File
@@ -7,6 +7,12 @@
#include <cinttypes>
#if defined(USE_ESP32_VARIANT_ESP32) && ESP_IDF_VERSION <= ESP_IDF_VERSION_VAL(5, 5, 2)
// Workaround for ESP-IDF bug: touch_ll_enable_channel_mask() doesn't apply the
// channel 8/9 bit swap. Fixed in ESP-IDF v5.5.3 via MR !43531.
#include <hal/touch_sensor_ll.h>
#endif
namespace esphome::esp32_touch {
static const char *const TAG = "esp32_touch";
@@ -203,6 +209,19 @@ void ESP32TouchComponent::setup() {
return;
}
#if defined(USE_ESP32_VARIANT_ESP32) && ESP_IDF_VERSION <= ESP_IDF_VERSION_VAL(5, 5, 2)
// Workaround for ESP-IDF bug: touch_ll_enable_channel_mask() doesn't apply the
// bit swap for channels 8 and 9. The ESP32 hardware has these two channels
// physically swapped in the SENS peripheral registers. The legacy API applied
// TOUCH_LL_BITS_SWAP() but the new unified API omits it, causing channels 8
// (GPIO33) and 9 (GPIO32) to not be enabled for measurement (reads return zero).
// Fixed in ESP-IDF v5.5.3 via MR !43531.
{
uint16_t worken = SENS.sar_touch_enable.touch_pad_worken;
SENS.sar_touch_enable.touch_pad_worken = TOUCH_LL_BITS_SWAP(worken);
}
#endif
// Do initial oneshot scans to populate baseline values
for (int i = 0; i < 3; i++) {
touch_sensor_trigger_oneshot_scanning(this->sens_handle_, 2000);