]> granicus.if.org Git - esp-idf/commitdiff
gpio driver: Fix gpio_set_level validation of gpio_num argument
authorAngus Gratton <angus@espressif.com>
Thu, 22 Dec 2016 02:05:19 +0000 (13:05 +1100)
committerAngus Gratton <angus@espressif.com>
Tue, 27 Dec 2016 23:17:56 +0000 (10:17 +1100)
Closes #125 https://github.com/espressif/esp-idf/issues/125

components/driver/gpio.c
components/driver/include/driver/gpio.h

index 1a38620dbb525991bb28914067090d5ff23dddde..3201372bc1dd096b870ac9e5f21b27f13fb0ae56 100644 (file)
@@ -161,7 +161,7 @@ static esp_err_t gpio_output_enable(gpio_num_t gpio_num)
 
 esp_err_t gpio_set_level(gpio_num_t gpio_num, uint32_t level)
 {
-    GPIO_CHECK(GPIO_IS_VALID_GPIO(gpio_num), "GPIO number error", ESP_ERR_INVALID_ARG);
+    GPIO_CHECK(GPIO_IS_VALID_OUTPUT_GPIO(gpio_num), "GPIO output gpio_num error", ESP_ERR_INVALID_ARG);
     if (level) {
         if (gpio_num < 32) {
             GPIO.out_w1ts = (1 << gpio_num);
index 83d38068342335843cb7fa867ec34181dd8d08c2..4f2a8007241e22f275e268422c418896698f94fe 100644 (file)
@@ -269,7 +269,7 @@ esp_err_t gpio_intr_disable(gpio_num_t gpio_num);
  *
  * @return
  *     - ESP_OK Success
- *     - GPIO_IS_VALID_GPIO GPIO number error
+ *     - ESP_ERR_INVALID_ARG GPIO number error
  *
  */
 esp_err_t gpio_set_level(gpio_num_t gpio_num, uint32_t level);