]> granicus.if.org Git - esp-idf/commitdiff
modify GPIO number check
authorWangjialin <wangjialin@espressif.com>
Mon, 24 Oct 2016 01:38:34 +0000 (09:38 +0800)
committerWangjialin <wangjialin@espressif.com>
Mon, 24 Oct 2016 01:38:34 +0000 (09:38 +0800)
components/driver/uart.c

index 157e0c49c73363339735362ad281eeaaafd8dafa..f585c0965ddb3e56082a24275e78b9514f80b056 100644 (file)
@@ -312,10 +312,10 @@ esp_err_t uart_isr_register(uart_port_t uart_num, uint8_t uart_intr_num, void (*
 esp_err_t uart_set_pin(uart_port_t uart_num, int tx_io_num, int rx_io_num, int rts_io_num, int cts_io_num)\r
 {\r
     UART_CHECK((uart_num < UART_NUM_MAX), "uart_num error");\r
-    UART_CHECK((tx_io_num < 0 || tx_io_num < GPIO_PIN_COUNT || GPIO_PIN_MUX_REG[tx_io_num] != 0), "tx_io_num error");\r
-    UART_CHECK((rx_io_num < 0 || rx_io_num < GPIO_PIN_COUNT || GPIO_PIN_MUX_REG[rx_io_num] != 0), "rx_io_num error");\r
-    UART_CHECK((rts_io_num < 0 || rts_io_num < GPIO_PIN_COUNT || GPIO_PIN_MUX_REG[rts_io_num] != 0), "rts_io_num error");\r
-    UART_CHECK((cts_io_num < 0 || cts_io_num < GPIO_PIN_COUNT || GPIO_PIN_MUX_REG[cts_io_num] != 0), "cts_io_num error");\r
+    UART_CHECK((tx_io_num < 0 || (GPIO_IS_VALID_OUTPUT_GPIO(tx_io_num))), "tx_io_num error");\r
+    UART_CHECK((rx_io_num < 0 || (GPIO_IS_VALID_GPIO(rx_io_num))), "rx_io_num error");\r
+    UART_CHECK((rts_io_num < 0 || (GPIO_IS_VALID_OUTPUT_GPIO(rts_io_num))), "rts_io_num error");\r
+    UART_CHECK((cts_io_num < 0 || (GPIO_IS_VALID_GPIO(cts_io_num))), "cts_io_num error");\r
 \r
     int tx_sig, rx_sig, rts_sig, cts_sig;\r
     switch(uart_num) {\r