Add missing peripheral support on STM32F1

[benjaminbjornsson]

This PR aims at resolving #79809 which requests adding support for these peripherals to the STM32F1 series:

• I2S
• FSMC
• SDIO

[benjaminbjornsson]

I successfully built samples/drivers/i2s/output using waveshare_open103z with these local changes:

diff --git a/boards/waveshare/open103z/waveshare_open103z.dts b/boards/waveshare/open103z/waveshare_open103z.dts
index 72dd744c4c2..41ae01b380f 100644
--- a/boards/waveshare/open103z/waveshare_open103z.dts
+++ b/boards/waveshare/open103z/waveshare_open103z.dts
@@ -82,6 +82,7 @@
 		led3 = &led_4;
 		sw0 = &button;
 		watchdog0 = &iwdg;
+		i2s-tx = &i2s2;
 	};
 };
 
@@ -105,6 +106,14 @@
 	/* usbpre not set: USB clock = 72 / 1.5: 48MHz */
 };
 
+&dma1 {
+	status = "okay";
+};
+
+&dma2 {
+	status = "okay";
+};
+
 &usart1 {
 	pinctrl-0 = <&usart1_tx_pa9 &usart1_rx_pa10>;
 	pinctrl-names = "default";
@@ -130,6 +139,18 @@
 	pinctrl-0 = <&spi2_nss_master_pb12 &spi2_sck_master_pb13
 		     &spi2_miso_master_pb14 &spi2_mosi_master_pb15>;
 	pinctrl-names = "default";
+	status = "disabled";
+};
+
+&i2s2 {
+	pinctrl-0 = <&i2s2_ws_pb12 &i2s2_sd_pb15>;
+	pinctrl-names = "default";
+	status = "okay";
+};
+
+&i2s3 {
+	pinctrl-0 = <&i2s3_ws_pa15 &i2s3_sd_pb5>;
+	pinctrl-names = "default";
 	status = "okay";
 };

[zephyrbot]

The following west manifest projects have been modified in this Pull Request:

Name	Old Revision	New Revision	Diff
hal_stm32	zephyrproject-rtos/hal_stm32@019d825 (main)	zephyrproject-rtos/hal_stm32#239	zephyrproject-rtos/hal_stm32#239/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.

[benjaminbjornsson]

Successfully built tests/drivers/disk/disk_access using waveshare_open103z with these local changes:

diff --git a/boards/waveshare/open103z/waveshare_open103z.dts b/boards/waveshare/open103z/waveshare_open103z.dts
index 72dd744c4c2..0f9866d119e 100644
--- a/boards/waveshare/open103z/waveshare_open103z.dts
+++ b/boards/waveshare/open103z/waveshare_open103z.dts
@@ -82,6 +82,7 @@
                led3 = &led_4;
                sw0 = &button;
                watchdog0 = &iwdg;
+               sdhc0 = &sdmmc1;
        };
 };
 
@@ -189,3 +190,14 @@ zephyr_udc0: &usb {
        pinctrl-names = "default";
        status = "okay";
 };
+
+&sdmmc1 {
+       status = "okay";
+       pinctrl-0 = <&sdio_d0_pc8
+                    &sdio_d1_pc9
+                    &sdio_d2_pc10
+                    &sdio_d3_pc11
+                    &sdio_ck_pc12
+                    &sdio_cmd_pd2>;
+       pinctrl-names = "default";
+};

[erwango]

Why not DT_NODE_HAS_PROP(DT_DRV_INST(0), reset) ?

[erwango]

Nit: Using IS_ENABLED() to preserve readability (may have the consequence of having .reset always present in the structure, but if we consider we're building a SDMMC application, so it can be seen as neglectible).

This being said proposed code is aligned with DMA handling, so keeping it this way could be ok as well. This is readability vs consistency ..