]> granicus.if.org Git - esp-idf/commitdiff
bugfix/sdspi_host.c: wrong CD/WP pin configuration
authorFlavio Bayer <admin@anoniamto.com>
Fri, 13 Apr 2018 19:34:23 +0000 (16:34 -0300)
committerAngus Gratton <gus@projectgus.com>
Mon, 23 Apr 2018 01:32:42 +0000 (11:32 +1000)
Looks like the configuration of CP and WP pins are wrong, since the check for `gpio_cd` and `gpio_wp` in `slot_config` seems to be swapped.

components/driver/sdspi_host.c

index 5c9db7e46ab23120abbea86725dccf9fa0a67cec..f47b590031fcb727863d04adc01daaf1aadd5bba 100644 (file)
@@ -316,16 +316,16 @@ esp_err_t sdspi_host_init_slot(int slot, const sdspi_slot_config_t* slot_config)
     };\r
     if (slot_config->gpio_cd != SDSPI_SLOT_NO_CD) {\r
         io_conf.pin_bit_mask |= (1 << slot_config->gpio_cd);\r
-        s_slots[slot].gpio_wp = slot_config->gpio_wp;\r
+        s_slots[slot].gpio_cd = slot_config->gpio_cd;\r
     } else {\r
-        s_slots[slot].gpio_wp = GPIO_UNUSED;\r
+        s_slots[slot].gpio_cd = GPIO_UNUSED;\r
     }\r
 \r
     if (slot_config->gpio_wp != SDSPI_SLOT_NO_WP) {\r
         io_conf.pin_bit_mask |= (1 << slot_config->gpio_wp);\r
-        s_slots[slot].gpio_cd = slot_config->gpio_cd;\r
+        s_slots[slot].gpio_wp = slot_config->gpio_wp;\r
     } else {\r
-        s_slots[slot].gpio_cd = GPIO_UNUSED;\r
+        s_slots[slot].gpio_wp = GPIO_UNUSED;\r
     }\r
 \r
     if (io_conf.pin_bit_mask != 0) {\r