From: Wangjialin <wangjialin@espressif.com>
Date: Tue, 19 Sep 2017 11:40:49 +0000 (+0800)
Subject: bugfix(touch): add workaround for touch pad DAC hardware source mismatch on tp8 and... 
X-Git-Tag: v3.1-dev~147^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=449f9e10a33286b2dc2b804cf16a5fc48219f37d;p=esp-idf

bugfix(touch): add workaround for touch pad DAC hardware source mismatch on tp8 and tp9.
---

diff --git a/components/driver/include/driver/touch_pad.h b/components/driver/include/driver/touch_pad.h
index ce67a7dc72..9d14fe4e71 100644
--- a/components/driver/include/driver/touch_pad.h
+++ b/components/driver/include/driver/touch_pad.h
@@ -31,8 +31,8 @@ typedef enum {
     TOUCH_PAD_NUM5,     /*!< Touch pad channel 5 is GPIO12*/
     TOUCH_PAD_NUM6,     /*!< Touch pad channel 6 is GPIO14*/
     TOUCH_PAD_NUM7,     /*!< Touch pad channel 7 is GPIO27*/
-    TOUCH_PAD_NUM8,     /*!< Touch pad channel 8 is GPIO32*/
-    TOUCH_PAD_NUM9,     /*!< Touch pad channel 9 is GPIO33*/
+    TOUCH_PAD_NUM8,     /*!< Touch pad channel 8 is GPIO33*/
+    TOUCH_PAD_NUM9,     /*!< Touch pad channel 9 is GPIO32*/
     TOUCH_PAD_MAX,
 } touch_pad_t;
 
diff --git a/components/driver/rtc_module.c b/components/driver/rtc_module.c
index 0804721e4d..95370b1cb9 100644
--- a/components/driver/rtc_module.c
+++ b/components/driver/rtc_module.c
@@ -510,8 +510,16 @@ esp_err_t touch_pad_set_cnt_mode(touch_pad_t touch_num, touch_cnt_slope_t slope,
     portENTER_CRITICAL(&rtc_spinlock);
     //set tie opt value, high or low level seem no difference for counter
     RTCIO.touch_pad[touch_num].tie_opt = opt;
+
+    //workaround for touch pad DAC mismatch on tp8 and tp9
+    touch_pad_t touch_pad_wrap = touch_num;
+    if (touch_num == TOUCH_PAD_NUM9) {
+        touch_pad_wrap = TOUCH_PAD_NUM8;
+    } else if (touch_num == TOUCH_PAD_NUM8) {
+        touch_pad_wrap = TOUCH_PAD_NUM9;
+    }
     //touch sensor set slope for charging and discharging.
-    RTCIO.touch_pad[touch_num].dac = slope;
+    RTCIO.touch_pad[touch_pad_wrap].dac = slope;
     portEXIT_CRITICAL(&rtc_spinlock);
     return ESP_OK;
 }
@@ -521,7 +529,14 @@ esp_err_t touch_pad_get_cnt_mode(touch_pad_t touch_num, touch_cnt_slope_t *slope
     RTC_MODULE_CHECK((touch_num < TOUCH_PAD_MAX), "touch IO error", ESP_ERR_INVALID_ARG);
     portENTER_CRITICAL(&rtc_spinlock);
     if (slope) {
-        *slope = RTCIO.touch_pad[touch_num].dac;
+        //workaround for touch pad DAC mismatch on tp8 and tp9
+        touch_pad_t touch_pad_wrap = touch_num;
+        if (touch_num == TOUCH_PAD_NUM9) {
+            touch_pad_wrap = TOUCH_PAD_NUM8;
+        } else if (touch_num == TOUCH_PAD_NUM8) {
+            touch_pad_wrap = TOUCH_PAD_NUM9;
+        }
+        *slope = RTCIO.touch_pad[touch_pad_wrap].dac;
     }
     if (opt) {
         *opt = RTCIO.touch_pad[touch_num].tie_opt;