drivers: disk: sdmmc_stm32: Add support for F1 series

Extend sdmmc driver to support the F1 series.

This includes removing the RCC peripheral reset when building
for F1 since this series doensn't have reset register for AHB
peripherals.
Therefore, it was neccesary to add an F1 specific
compatible that removes the required resets property,
as well as if-def the usage of reset in the device driver.

Signed-off-by: Benjamin Björnsson <[email protected]>

Author: benjaminbjornsson

drivers/disk/Kconfig.sdmmc

@@ -37,7 +37,7 @@ config SDMMC_STM32
 	select USE_STM32_HAL_MMC if SDMMC_STM32_EMMC
 	select USE_STM32_HAL_MMC_EX if SDMMC_STM32_EMMC && SOC_SERIES_STM32L4X
 	select USE_STM32_LL_SDMMC
-	select USE_STM32_HAL_DMA if (SOC_SERIES_STM32L4X || SOC_SERIES_STM32F7X || SOC_SERIES_STM32F4X)
+	select USE_STM32_HAL_DMA if (SOC_SERIES_STM32L4X || SOC_SERIES_STM32F7X || SOC_SERIES_STM32F4X || SOC_SERIES_STM32F1X)
 	select DMA if $(DT_STM32_SDMMC_HAS_DMA) && (SOC_SERIES_STM32F4X || SOC_SERIES_STM32F7X)
 	select PINCTRL
 	select RESET

drivers/disk/sdmmc_stm32.c

@@ -21,6 +21,7 @@
 LOG_MODULE_REGISTER(stm32_sdmmc, CONFIG_SDMMC_LOG_LEVEL);
 
 #define STM32_SDMMC_USE_DMA DT_NODE_HAS_PROP(DT_DRV_INST(0), dmas)
+#define STM32_SDMMC_USE_RESET !DT_NODE_HAS_COMPAT(DT_DRV_INST(0), st_stm32f1_sdmmc)
 
 #if STM32_SDMMC_USE_DMA
 #include <zephyr/drivers/dma.h>
@@ -84,7 +85,9 @@ struct stm32_sdmmc_priv {
 	struct gpio_dt_spec pe;
 	struct stm32_pclken *pclken;
 	const struct pinctrl_dev_config *pcfg;
+#if STM32_SDMMC_USE_RESET
 	const struct reset_dt_spec reset;
+#endif
 
 #if STM32_SDMMC_USE_DMA
 	struct sdmmc_dma_stream dma_rx;
@@ -302,11 +305,13 @@ static int stm32_sdmmc_access_init(struct disk_info *disk)
 		return err;
 	}
 
+#if STM32_SDMMC_USE_RESET
 	err = reset_line_toggle_dt(&priv->reset);
 	if (err) {
 		LOG_ERR("failed to reset peripheral");
 		return err;
 	}
+#endif
 
 #ifdef CONFIG_SDMMC_STM32_EMMC
 	err = HAL_MMC_Init(&priv->hsd);
@@ -690,10 +695,12 @@ static int disk_stm32_sdmmc_init(const struct device *dev)
 		return -ENODEV;
 	}
 
+#if STM32_SDMMC_USE_RESET
 	if (!device_is_ready(priv->reset.dev)) {
 		LOG_ERR("reset control device not ready");
 		return -ENODEV;
 	}
+#endif
 
 	/* Configure dt provided device signals when available */
 	err = pinctrl_apply_state(priv->pcfg, PINCTRL_STATE_DEFAULT);
@@ -812,7 +819,9 @@ static struct stm32_sdmmc_priv stm32_sdmmc_priv_1 = {
 #endif
 	.pclken = pclken_sdmmc,
 	.pcfg = PINCTRL_DT_INST_DEV_CONFIG_GET(0),
+#if STM32_SDMMC_USE_RESET
 	.reset = RESET_DT_SPEC_INST_GET(0),
+#endif
 	SDMMC_DMA_CHANNEL(rx, RX)
 	SDMMC_DMA_CHANNEL(tx, TX)
 };

dts/bindings/mmc/st,stm32f1-sdmmc.yaml

@@ -0,0 +1,15 @@
+# Copyright (c) 2024 Benjamin Björnsson [email protected]
+
+# SPDX-License-Identifier: Apache-2.0
+
+description: |
+        ST STM32F1 family SDIO
+        Remove the resets property since there's no reset register
+        for AHB peripherals on F1 series.
+
+compatible: "st,stm32f1-sdmmc"
+
+include:
+  - name: st,stm32-sdmmc.yaml
+    property-blocklist:
+      - resets