[PATCH 0/2] ARM: dts: stm32: add dfsdm & sai pin muxings on stm32mp13

[PATCH 0/2] ARM: dts: stm32: add dfsdm & sai pin muxings on stm32mp13

[Olivier Moysan]

Add pin muxings for STM32 DFSDM and SAI audio peripherals on STM32MP13
Disco boards expansion connector.

Olivier Moysan (2):
  ARM: dts: stm32: add pins muxing for sai1 on stm32mp13
  ARM: dts: stm32: add pins muxing for dfsdm on stm32mp13

 arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi | 60 +++++++++++++++++++++
 1 file changed, 60 insertions(+)

-- 
2.25.1

[PATCH 1/2] ARM: dts: stm32: add pins muxing for sai1 on stm32mp13

[Olivier Moysan]

These pins are used for SAI1 on STM32MP13 Disco boards expansion
connector.
Pins mapping:
- EXP_GPIO18	SAI1_SCKA (PA4)
- EXP_GPIO19	SAI1_FSA  (PF11)
- EXP_GPIO20	SAI1_SDA  (PA5)
- EXP_GPIO21	SAI1_SDB  (PA0)

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi b/arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi
index 27e0c3826789..5e3e627ceb54 100644
--- a/arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi
+++ b/arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi
@@ -114,6 +114,27 @@ pins {
 		};
 	};
 
+	sai1a_pins_a: sai1a-0 {
+		pins {
+			pinmux = <STM32_PINMUX('A', 4, AF12)>, /* SAI1_SCK_A */
+				 <STM32_PINMUX('A', 0, AF6)>, /* SAI1_SD_B */
+				 <STM32_PINMUX('A', 5, AF6)>, /* SAI1_SD_A */
+				 <STM32_PINMUX('F', 11, AF6)>; /* SAI1_FS_A */
+			slew-rate = <0>;
+			drive-push-pull;
+			bias-disable;
+		};
+	};
+
+	sai1a_sleep_pins_a: sai1a-sleep-0 {
+		pins {
+			pinmux = <STM32_PINMUX('A', 4, ANALOG)>, /* SAI1_SCK_A */
+				 <STM32_PINMUX('A', 0, ANALOG)>, /* SAI1_SD_B */
+				 <STM32_PINMUX('A', 5, ANALOG)>, /* SAI1_SD_A */
+				 <STM32_PINMUX('F', 11, ANALOG)>; /* SAI1_FS_A */
+		};
+	};
+
 	sdmmc1_b4_pins_a: sdmmc1-b4-0 {
 		pins {
 			pinmux = <STM32_PINMUX('C', 8, AF12)>, /* SDMMC1_D0 */
-- 
2.25.1

[PATCH 2/2] ARM: dts: stm32: add pins muxing for dfsdm on stm32mp13

[Olivier Moysan]

These pins are used for DFSDM on STM32MP13x Disco boards expansion
connector.
Pins mapping:
- EXP_GPIO27	DFSDM1_CKOUT
- EXP_GPIO22	DFSDM1_DATIN3
- EXP_GPIO4	 DFSDM1_DATIN1	(shared with TIM3_CH4)

Signed-off-by: Olivier Moysan <olivier.moysan@foss.st.com>
---
 arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi | 39 +++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi b/arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi
index 5e3e627ceb54..50490053f03d 100644
--- a/arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi
+++ b/arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi
@@ -13,6 +13,45 @@ pins {
 		};
 	};
 
+	dfsdm_clkout_pins_a: dfsdm-clkout-pins-0 {
+		pins {
+			pinmux = <STM32_PINMUX('C', 3, AF3)>; /* DFSDM_CKOUT */
+			bias-disable;
+			drive-push-pull;
+			slew-rate = <0>;
+		};
+	};
+
+	dfsdm_clkout_sleep_pins_a: dfsdm-clkout-sleep-pins-0 {
+		pins {
+			pinmux = <STM32_PINMUX('C', 3, ANALOG)>; /* DFSDM_CKOUT */
+		};
+	};
+
+	dfsdm_datin1_pins_a: dfsdm-datin1-pins-0 {
+		pins {
+			pinmux = <STM32_PINMUX('B', 1, AF6)>; /* DFSDM_DATIN1 */
+		};
+	};
+
+	dfsdm_datin1_sleep_pins_a: dfsdm-datin1-sleep-pins-0 {
+		pins {
+			pinmux = <STM32_PINMUX('B', 1, ANALOG)>; /* DFSDM_DATIN1 */
+		};
+	};
+
+	dfsdm_datin3_pins_a: dfsdm-datin3-pins-0 {
+		pins {
+			pinmux = <STM32_PINMUX('F', 13, AF6)>; /* DFSDM_DATIN3 */
+		};
+	};
+
+	dfsdm_datin3_sleep_pins_a: dfsdm-datin3-sleep-pins-0 {
+		pins {
+			pinmux = <STM32_PINMUX('F', 13, ANALOG)>; /* DFSDM_DATIN3 */
+		};
+	};
+
 	i2c1_pins_a: i2c1-0 {
 		pins {
 			pinmux = <STM32_PINMUX('D', 12, AF5)>, /* I2C1_SCL */
-- 
2.25.1

Re: [PATCH 0/2] ARM: dts: stm32: add dfsdm & sai pin muxings on stm32mp13

[Alexandre TORGUE]

Hi Olivier

On 9/19/23 18:30, Olivier Moysan wrote:
> Add pin muxings for STM32 DFSDM and SAI audio peripherals on STM32MP13
> Disco boards expansion connector.
> 
> Olivier Moysan (2):
>    ARM: dts: stm32: add pins muxing for sai1 on stm32mp13
>    ARM: dts: stm32: add pins muxing for dfsdm on stm32mp13
> 
>   arch/arm/boot/dts/st/stm32mp13-pinctrl.dtsi | 60 +++++++++++++++++++++
>   1 file changed, 60 insertions(+)
> 

Why do you push only pins muxing ? I prefer to merge pins when I'm sure 
that they'll be used on a board. In your case, if those pins rely on a 
connector then declare this descriptor (as we did for mp15).

regards
Alex