preprocessor messages review

Author: SuGlider

cores/esp32/esp32-hal-touch.c

@@ -13,15 +13,15 @@
 // limitations under the License.
 
 #include "soc/soc_caps.h"
-
-#if !defined(SOC_TOUCH_VERSION_1) && !defined(SOC_TOUCH_VERSION_2)
-#warning CONFIG_IDF_TARGET is not supported for Touch IDF driver!
-#endif
-
 #if SOC_TOUCH_SENSOR_NUM > 0
+
 #include "driver/touch_sensor.h"
 #include "esp32-hal-touch.h"
 
+#if !defined(SOC_TOUCH_VERSION_1) && !defined(SOC_TOUCH_VERSION_2)
+#error Touch IDF driver Not supported!
+#endif
+
 /*
     Internal Private Touch Data Structure and Functions
 */

cores/esp32/esp32-hal-touch.h

@@ -25,9 +25,13 @@ extern "C" {
 #endif
 
 #include "soc/soc_caps.h"
+#include "esp32-hal.h"
 
 #if SOC_TOUCH_SENSOR_NUM > 0
-#include "esp32-hal.h"
+
+#if !defined(CONFIG_IDF_TARGET_ESP32) && !defined(CONFIG_IDF_TARGET_ESP32S2) && !defined(CONFIG_IDF_TARGET_ESP32S3)
+#warning This SoC may not support the current Touch Sensor implementation!
+#endif
 
 #if SOC_TOUCH_VERSION_1 // ESP32
 typedef uint16_t touch_value_t;
@@ -84,7 +88,6 @@ void touchInterruptSetThresholdDirection(bool mustbeLower);
 bool touchInterruptGetLastStatus(uint8_t pin);
 #endif
 
-
 #endif // SOC_TOUCH_SENSOR_NUM > 0
 
 #ifdef __cplusplus