SDMMC - Peripheral manager implementation (#8289)

* sdmmc periman implemented

* save pins when SOC_SDMMC_USE_IOMUX

Author: P-R-O-C-H-Y

Diff for: libraries/SD_MMC/src/SD_MMC.cpp

@@ -24,10 +24,10 @@
 #include "sdmmc_cmd.h"
 #include "soc/sdmmc_pins.h"
 #include "ff.h"
+#include "esp32-hal-periman.h"
 
 using namespace fs;
 
-
 SDMMCFS::SDMMCFS(FSImplPtr impl)
     : FS(impl), _card(nullptr)
 {
@@ -40,7 +40,25 @@ SDMMCFS::SDMMCFS(FSImplPtr impl)
     _pin_d2 = SDMMC_D2;
     _pin_d3 = SDMMC_D3;
 #endif // BOARD_HAS_1BIT_SDMMC
-#endif // defined(SOC_SDMMC_USE_GPIO_MATRIX) && defined(BOARD_HAS_SDMMC)
+
+#elif SOC_SDMMC_USE_IOMUX
+    _pin_clk = SDMMC_SLOT1_IOMUX_PIN_NUM_CLK;
+    _pin_cmd = SDMMC_SLOT1_IOMUX_PIN_NUM_CMD;
+    _pin_d0 = SDMMC_SLOT1_IOMUX_PIN_NUM_D0;
+#ifndef BOARD_HAS_1BIT_SDMMC
+    _pin_d1 = SDMMC_SLOT1_IOMUX_PIN_NUM_D1;
+    _pin_d2 = SDMMC_SLOT1_IOMUX_PIN_NUM_D2;
+    _pin_d3 = SDMMC_SLOT1_IOMUX_PIN_NUM_D3;
+#endif // BOARD_HAS_1BIT_SDMMC
+#endif
+}
+
+bool SDMMCFS::sdmmcDetachBus(void * bus_pointer){
+    SDMMCFS *bus = (SDMMCFS *) bus_pointer;
+    if(bus->_card) {
+        bus->end();
+    }
+    return true;
 }
 
 bool SDMMCFS::setPins(int clk, int cmd, int d0)
@@ -68,12 +86,12 @@ bool SDMMCFS::setPins(int clk, int cmd, int d0, int d1, int d2, int d3)
     // Since SDMMCFS::begin hardcodes the usage of slot 1, only check if
     // the pins match slot 1 pins.
     bool pins_ok = (clk == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_CLK) &&
-        (cmd == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_CMD) &&
-        (d0 == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_D0) &&
-        (((d1 == -1) && (d2 == -1) && (d3 == -1)) ||
-         ((d1 == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_D1) &&
-         (d2 == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_D2) &&
-         (d3 == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_D3)));
+                   (cmd == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_CMD) &&
+                   (d0 == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_D0) &&
+                   (((d1 == -1) && (d2 == -1) && (d3 == -1)) ||
+                   ((d1 == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_D1) &&
+                   (d2 == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_D2) &&
+                   (d3 == (int)SDMMC_SLOT1_IOMUX_PIN_NUM_D3)));
     if (!pins_ok) {
         log_e("SDMMCFS: specified pins are not supported by this chip.");
         return false;
@@ -89,11 +107,13 @@ bool SDMMCFS::begin(const char * mountpoint, bool mode1bit, bool format_if_mount
     if(_card) {
         return true;
     }
+    perimanSetBusDeinit(ESP32_BUS_TYPE_SDMMC, SDMMCFS::sdmmcDetachBus);
+
     //mount
     sdmmc_slot_config_t slot_config = SDMMC_SLOT_CONFIG_DEFAULT();
 #ifdef SOC_SDMMC_USE_GPIO_MATRIX
     // SoC supports SDMMC pin configuration via GPIO matrix.
-    // Chech that the pins have been set either in the constructor or setPins function.
+    // Check that the pins have been set either in the constructor or setPins function.
     if (_pin_cmd == -1 || _pin_clk == -1 || _pin_d0 == -1
         || (!mode1bit && (_pin_d1 == -1 || _pin_d2 == -1 || _pin_d3 == -1))) {
         log_e("SDMMCFS: some SD pins are not set");
@@ -108,6 +128,16 @@ bool SDMMCFS::begin(const char * mountpoint, bool mode1bit, bool format_if_mount
     slot_config.d3 = (gpio_num_t) _pin_d3;
     slot_config.width = 4;
 #endif // SOC_SDMMC_USE_GPIO_MATRIX
+
+    if(!perimanSetPinBus(_pin_cmd, ESP32_BUS_TYPE_INIT, NULL)){ return false; }
+    if(!perimanSetPinBus(_pin_clk, ESP32_BUS_TYPE_INIT, NULL)){ return false; }
+    if(!perimanSetPinBus(_pin_d0,  ESP32_BUS_TYPE_INIT, NULL)){ return false; }
+    if(!mode1bit) {
+        if(!perimanSetPinBus(_pin_d1, ESP32_BUS_TYPE_INIT, NULL)){ return false; }
+        if(!perimanSetPinBus(_pin_d2, ESP32_BUS_TYPE_INIT, NULL)){ return false; }
+        if(!perimanSetPinBus(_pin_d3, ESP32_BUS_TYPE_INIT, NULL)){ return false; }
+    }
+    
     sdmmc_host_t host = SDMMC_HOST_DEFAULT();
     host.flags = SDMMC_HOST_FLAG_4BIT;
     host.slot = SDMMC_HOST_SLOT_1;
@@ -119,6 +149,7 @@ bool SDMMCFS::begin(const char * mountpoint, bool mode1bit, bool format_if_mount
         host.flags = SDMMC_HOST_FLAG_1BIT; //use 1-line SD mode
         slot_config.width = 1;
     }
+    _mode1bit = mode1bit;
 
     esp_vfs_fat_sdmmc_mount_config_t mount_config = {
         .format_if_mount_failed = format_if_mount_failed,
@@ -142,7 +173,21 @@ bool SDMMCFS::begin(const char * mountpoint, bool mode1bit, bool format_if_mount
         return false;
     }
     _impl->mountpoint(mountpoint);
+
+    if(!perimanSetPinBus(_pin_cmd, ESP32_BUS_TYPE_SDMMC, (void *)(this))){ goto err; }
+    if(!perimanSetPinBus(_pin_clk, ESP32_BUS_TYPE_SDMMC, (void *)(this))){ goto err; }
+    if(!perimanSetPinBus(_pin_d0,  ESP32_BUS_TYPE_SDMMC, (void *)(this))){ goto err; }
+    if(!mode1bit) {
+        if(!perimanSetPinBus(_pin_d1, ESP32_BUS_TYPE_SDMMC, (void *)(this))){ goto err; }
+        if(!perimanSetPinBus(_pin_d2, ESP32_BUS_TYPE_SDMMC, (void *)(this))){ goto err; }
+        if(!perimanSetPinBus(_pin_d3, ESP32_BUS_TYPE_SDMMC, (void *)(this))){ goto err; }
+    }
     return true;
+
+err:
+    log_e("Failed to set all pins bus to SDMMC");
+    SDMMCFS::sdmmcDetachBus((void *)(this));
+    return false;
 }
 
 void SDMMCFS::end()
@@ -151,6 +196,14 @@ void SDMMCFS::end()
         esp_vfs_fat_sdcard_unmount(_impl->mountpoint(), _card);
         _impl->mountpoint(NULL);
         _card = NULL;
+        perimanSetPinBus(_pin_cmd, ESP32_BUS_TYPE_INIT, NULL);
+        perimanSetPinBus(_pin_clk, ESP32_BUS_TYPE_INIT, NULL);
+        perimanSetPinBus(_pin_d0,  ESP32_BUS_TYPE_INIT, NULL);
+        if(!_mode1bit) {
+            perimanSetPinBus(_pin_d1, ESP32_BUS_TYPE_INIT, NULL);
+            perimanSetPinBus(_pin_d2, ESP32_BUS_TYPE_INIT, NULL);
+            perimanSetPinBus(_pin_d3, ESP32_BUS_TYPE_INIT, NULL);
+        }
     }
 }
 

Diff for: libraries/SD_MMC/src/SD_MMC.h

@@ -36,15 +36,13 @@ class SDMMCFS : public FS
 {
 protected:
     sdmmc_card_t* _card;
-
-#ifdef SOC_SDMMC_USE_GPIO_MATRIX
     int8_t _pin_clk = -1;
     int8_t _pin_cmd = -1;
     int8_t _pin_d0 = -1;
     int8_t _pin_d1 = -1;
     int8_t _pin_d2 = -1;
     int8_t _pin_d3 = -1;
-#endif
+    bool _mode1bit = false;
 
 public:
     SDMMCFS(FSImplPtr impl);
@@ -56,6 +54,9 @@ class SDMMCFS : public FS
     uint64_t cardSize();
     uint64_t totalBytes();
     uint64_t usedBytes();
+    
+private:
+    static bool sdmmcDetachBus(void * bus_pointer);
 };
 
 }