fix(gpio): remove GPIO_OUTPUT_SET macro for ESP32P4

Use GPIO_OUTPUT_SET led to compilation error

Closes #16050

Author: songruo

components/esp_driver_gpio/test_apps/gpio/sdkconfig.ci.iram_safe

@@ -8,3 +8,5 @@ CONFIG_COMPILER_OPTIMIZATION_CHECKS_SILENT=y
 CONFIG_COMPILER_OPTIMIZATION_ASSERTIONS_SILENT=y
 # GPIO test uses IPC call, the default stack size of IPC task can satisfy the -O0 optimization
 CONFIG_ESP_IPC_TASK_STACK_SIZE=2048
+# test gpio functionality with non-default option
+CONFIG_HAL_GPIO_USE_ROM_IMPL=n

components/esp_rom/esp32/include/esp32/rom/gpio.h

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2010-2022 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2010-2025 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -36,15 +36,6 @@ extern "C" {
 
 #define GPIO_REGID_TO_PINIDX(reg_id) ((reg_id) - GPIO_ID_PIN0)
 
-typedef enum {
-    GPIO_PIN_INTR_DISABLE = 0,
-    GPIO_PIN_INTR_POSEDGE = 1,
-    GPIO_PIN_INTR_NEGEDGE = 2,
-    GPIO_PIN_INTR_ANYEDGE = 3,
-    GPIO_PIN_INTR_LOLEVEL = 4,
-    GPIO_PIN_INTR_HILEVEL = 5
-} GPIO_INT_TYPE;
-
 #define GPIO_OUTPUT_SET(gpio_no, bit_value) \
         ((gpio_no < 32) ? gpio_output_set(bit_value<<gpio_no, (bit_value ? 0 : 1)<<gpio_no, 1<<gpio_no,0) : \
                          gpio_output_set_high(bit_value<<(gpio_no - 32), (bit_value ? 0 : 1)<<(gpio_no - 32), 1<<(gpio_no -32),0))
@@ -103,28 +94,6 @@ uint32_t gpio_input_get(void);
   */
 uint32_t gpio_input_get_high(void);
 
-/**
-  * @brief Set GPIO to wakeup the ESP32.
-  *        Please do not call this function in SDK.
-  *
-  * @param uint32_t i: gpio number.
-  *
-  * @param GPIO_INT_TYPE intr_state : only GPIO_PIN_INTR_LOLEVEL\GPIO_PIN_INTR_HILEVEL can be used
-  *
-  * @return None
-  */
-void gpio_pin_wakeup_enable(uint32_t i, GPIO_INT_TYPE intr_state);
-
-/**
-  * @brief disable GPIOs to wakeup the ESP32.
-  *        Please do not call this function in SDK.
-  *
-  * @param None
-  *
-  * @return None
-  */
-void gpio_pin_wakeup_disable(void);
-
 /**
   * @brief set gpio input to a signal, one gpio can input to several signals.
   *

components/esp_rom/esp32c2/include/esp32c2/rom/gpio.h

@@ -73,7 +73,7 @@ void gpio_output_set(uint32_t set_mask, uint32_t clear_mask, uint32_t enable_mas
 uint32_t gpio_input_get(void);
 
 /**
-  * @brief Set GPIO to wakeup the ESP32.
+  * @brief Set GPIO to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param uint32_t i: gpio number.
@@ -85,7 +85,7 @@ uint32_t gpio_input_get(void);
 void gpio_pin_wakeup_enable(uint32_t i, GPIO_INT_TYPE intr_state);
 
 /**
-  * @brief disable GPIOs to wakeup the ESP32.
+  * @brief disable GPIOs to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param None

components/esp_rom/esp32c3/include/esp32c3/rom/gpio.h

@@ -73,7 +73,7 @@ void gpio_output_set(uint32_t set_mask, uint32_t clear_mask, uint32_t enable_mas
 uint32_t gpio_input_get(void);
 
 /**
-  * @brief Set GPIO to wakeup the ESP32.
+  * @brief Set GPIO to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param uint32_t i: gpio number.
@@ -85,7 +85,7 @@ uint32_t gpio_input_get(void);
 void gpio_pin_wakeup_enable(uint32_t i, GPIO_INT_TYPE intr_state);
 
 /**
-  * @brief disable GPIOs to wakeup the ESP32.
+  * @brief disable GPIOs to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param None

components/esp_rom/esp32c5/include/esp32c5/rom/gpio.h

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2022-2024 Espressif Systems (Shanghai) CO LTD
+ * SPDX-FileCopyrightText: 2022-2025 Espressif Systems (Shanghai) CO LTD
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -33,62 +33,23 @@ extern "C" {
 
 #define GPIO_REGID_TO_PINIDX(reg_id) ((reg_id) - GPIO_ID_PIN0)
 
-typedef enum {
-    GPIO_PIN_INTR_DISABLE = 0,
-    GPIO_PIN_INTR_POSEDGE = 1,
-    GPIO_PIN_INTR_NEGEDGE = 2,
-    GPIO_PIN_INTR_ANYEDGE = 3,
-    GPIO_PIN_INTR_LOLEVEL = 4,
-    GPIO_PIN_INTR_HILEVEL = 5
-} GPIO_INT_TYPE;
-
-
-/**
-  * @brief Change GPIO(0-28) pin output by setting, clearing, or disabling pins, GPIO0<->BIT(0).
-  *         There is no particular ordering guaranteed; so if the order of writes is significant,
-  *         calling code should divide a single call into multiple calls.
-  *
-  * @param  uint32_t set_mask : the gpios that need high level.
-  *
-  * @param  uint32_t clear_mask : the gpios that need low level.
-  *
-  * @param  uint32_t enable_mask : the gpios that need be changed.
-  *
-  * @param  uint32_t disable_mask : the gpios that need disable output.
-  *
-  * @return None
-  */
-void gpio_output_set(uint32_t set_mask, uint32_t clear_mask, uint32_t enable_mask, uint32_t disable_mask);
 
 /**
-  * @brief Sample the value of GPIO input pins(0-28) and returns a bitmask.
-  * @param None
+  * @brief Set GPIO output level
   *
-  * @return uint32_t : bitmask for GPIO input pins, BIT(0) for GPIO0.
+  * @param gpio_num GPIO number
+  * @param level Output level, 0:low; 1:high
   */
-uint32_t gpio_input_get(void);
+void gpio_set_output_level(uint32_t gpio_num, uint32_t level);
 
 /**
-  * @brief Set GPIO to wakeup.
-  *        Please do not call this function in SDK.
-  *
-  * @param uint32_t i: gpio number.
+  * @brief Get GPIO input level
   *
-  * @param GPIO_INT_TYPE intr_state : only GPIO_PIN_INTR_LOLEVEL\GPIO_PIN_INTR_HILEVEL can be used
+  * @param gpio_num GPIO number
   *
-  * @return None
-  */
-void gpio_pin_wakeup_enable(uint32_t i, GPIO_INT_TYPE intr_state);
-
-/**
-  * @brief disable GPIOs to wakeup.
-  *        Please do not call this function in SDK.
-  *
-  * @param None
-  *
-  * @return None
+  * @return 0:the GPIO_input level is low; 1:the GPIO input level is high
   */
-void gpio_pin_wakeup_disable(void);
+uint32_t gpio_get_input_level(uint32_t gpio_num);
 
 /**
   * @brief set gpio input to a signal, one gpio can input to several signals.

components/esp_rom/esp32c6/include/esp32c6/rom/gpio.h

@@ -72,7 +72,7 @@ void gpio_output_set(uint32_t set_mask, uint32_t clear_mask, uint32_t enable_mas
 uint32_t gpio_input_get(void);
 
 /**
-  * @brief Set GPIO to wakeup the ESP32.
+  * @brief Set GPIO to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param uint32_t i: gpio number.
@@ -84,7 +84,7 @@ uint32_t gpio_input_get(void);
 void gpio_pin_wakeup_enable(uint32_t i, GPIO_INT_TYPE intr_state);
 
 /**
-  * @brief disable GPIOs to wakeup the ESP32.
+  * @brief disable GPIOs to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param None

components/esp_rom/esp32c61/include/esp32c61/rom/gpio.h

@@ -33,62 +33,23 @@ extern "C" {
 
 #define GPIO_REGID_TO_PINIDX(reg_id) ((reg_id) - GPIO_ID_PIN0)
 
-typedef enum {
-    GPIO_PIN_INTR_DISABLE = 0,
-    GPIO_PIN_INTR_POSEDGE = 1,
-    GPIO_PIN_INTR_NEGEDGE = 2,
-    GPIO_PIN_INTR_ANYEDGE = 3,
-    GPIO_PIN_INTR_LOLEVEL = 4,
-    GPIO_PIN_INTR_HILEVEL = 5
-} GPIO_INT_TYPE;
-
-
-/**
-  * @brief Change GPIO(0-29) pin output by setting, clearing, or disabling pins, GPIO0<->BIT(0).
-  *         There is no particular ordering guaranteed; so if the order of writes is significant,
-  *         calling code should divide a single call into multiple calls.
-  *
-  * @param  uint32_t set_mask : the gpios that need high level.
-  *
-  * @param  uint32_t clear_mask : the gpios that need low level.
-  *
-  * @param  uint32_t enable_mask : the gpios that need be changed.
-  *
-  * @param  uint32_t disable_mask : the gpios that need disable output.
-  *
-  * @return None
-  */
-void gpio_output_set(uint32_t set_mask, uint32_t clear_mask, uint32_t enable_mask, uint32_t disable_mask);
 
 /**
-  * @brief Sample the value of GPIO input pins(0-29) and returns a bitmask.
-  * @param None
+  * @brief Set GPIO output level
   *
-  * @return uint32_t : bitmask for GPIO input pins, BIT(0) for GPIO0.
+  * @param gpio_num GPIO number
+  * @param level Output level, 0:low; 1:high
   */
-uint32_t gpio_input_get(void);
+void gpio_set_output_level(uint32_t gpio_num, uint32_t level);
 
 /**
-  * @brief Set GPIO to wakeup.
-  *        Please do not call this function in SDK.
-  *
-  * @param uint32_t i: gpio number.
+  * @brief Get GPIO input level
   *
-  * @param GPIO_INT_TYPE intr_state : only GPIO_PIN_INTR_LOLEVEL\GPIO_PIN_INTR_HILEVEL can be used
+  * @param gpio_num GPIO number
   *
-  * @return None
-  */
-void gpio_pin_wakeup_enable(uint32_t i, GPIO_INT_TYPE intr_state);
-
-/**
-  * @brief disable GPIOs to wakeup.
-  *        Please do not call this function in SDK.
-  *
-  * @param None
-  *
-  * @return None
+  * @return 0:the GPIO_input level is low; 1:the GPIO input level is high
   */
-void gpio_pin_wakeup_disable(void);
+uint32_t gpio_get_input_level(uint32_t gpio_num);
 
 /**
   * @brief set gpio input to a signal, one gpio can input to several signals.

components/esp_rom/esp32h2/include/esp32h2/rom/gpio.h

@@ -73,7 +73,7 @@ void gpio_output_set(uint32_t set_mask, uint32_t clear_mask, uint32_t enable_mas
 uint32_t gpio_input_get(void);
 
 /**
-  * @brief Set GPIO to wakeup the ESP32.
+  * @brief Set GPIO to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param uint32_t i: gpio number.
@@ -85,7 +85,7 @@ uint32_t gpio_input_get(void);
 void gpio_pin_wakeup_enable(uint32_t i, GPIO_INT_TYPE intr_state);
 
 /**
-  * @brief disable GPIOs to wakeup the ESP32.
+  * @brief disable GPIOs to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param None

components/esp_rom/esp32h21/include/esp32h21/rom/gpio.h

@@ -73,7 +73,7 @@ void gpio_output_set(uint32_t set_mask, uint32_t clear_mask, uint32_t enable_mas
 uint32_t gpio_input_get(void);
 
 /**
-  * @brief Set GPIO to wakeup the ESP32H21.
+  * @brief Set GPIO to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param uint32_t i: gpio number.
@@ -85,7 +85,7 @@ uint32_t gpio_input_get(void);
 void gpio_pin_wakeup_enable(uint32_t i, GPIO_INT_TYPE intr_state);
 
 /**
-  * @brief disable GPIOs to wakeup the ESP32H21.
+  * @brief disable GPIOs to wakeup the chip.
   *        Please do not call this function in SDK.
   *
   * @param None