[PATCH 0/2] ARM: dts: stm32: add SDIO pinctrl sleep support on stm32f7

[PATCH 0/2] ARM: dts: stm32: add SDIO pinctrl sleep support on stm32f7

[Ben Wolsieffer]

Add support for switching the SDIO pins on the STM32F7 series to analog
mode when not in use to save power.

I don't have any of the disco/eval boards to test, but I have tested
this on an Emcraft STM32F746 SOM with a UCL-SOM-BSB carrier (using an
out-of-tree DT) and our custom carrier board.

Ben Wolsieffer (2):
  ARM: dts: stm32: add stm32f7 SDIO sleep pins
  ARM: dts: stm32: add SDIO pinctrl sleep support on stm32f7 boards

 arch/arm/boot/dts/st/stm32746g-eval.dts   |  3 ++-
 arch/arm/boot/dts/st/stm32f7-pinctrl.dtsi | 22 ++++++++++++++++++++++
 arch/arm/boot/dts/st/stm32f746-disco.dts  |  3 ++-
 arch/arm/boot/dts/st/stm32f769-disco.dts  |  3 ++-
 4 files changed, 28 insertions(+), 3 deletions(-)

-- 
2.42.0

[PATCH 1/2] ARM: dts: stm32: add stm32f7 SDIO sleep pins

[Ben Wolsieffer]

Add SDIO sleep pin definitions that place the pins in analog mode to
save power.

Signed-off-by: Ben Wolsieffer <ben.wolsieffer@hefring.com>
---
 arch/arm/boot/dts/st/stm32f7-pinctrl.dtsi | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/arch/arm/boot/dts/st/stm32f7-pinctrl.dtsi b/arch/arm/boot/dts/st/stm32f7-pinctrl.dtsi
index 9f65403295ca..26f91ca0d458 100644
--- a/arch/arm/boot/dts/st/stm32f7-pinctrl.dtsi
+++ b/arch/arm/boot/dts/st/stm32f7-pinctrl.dtsi
@@ -253,6 +253,17 @@ pins2 {
 				};
 			};
 
+			sdio_pins_sleep_a: sdio-pins-sleep-a-0 {
+				pins {
+					pinmux = <STM32_PINMUX('C', 8, ANALOG)>, /* SDMMC1 D0 */
+						 <STM32_PINMUX('C', 9, ANALOG)>, /* SDMMC1 D1 */
+						 <STM32_PINMUX('C', 10, ANALOG)>, /* SDMMC1 D2 */
+						 <STM32_PINMUX('C', 11, ANALOG)>, /* SDMMC1 D3 */
+						 <STM32_PINMUX('C', 12, ANALOG)>, /* SDMMC1 CLK */
+						 <STM32_PINMUX('D', 2, ANALOG)>; /* SDMMC1 CMD */
+				};
+			};
+
 			sdio_pins_b: sdio-pins-b-0 {
 				pins {
 					pinmux = <STM32_PINMUX('G', 9, AF11)>, /* SDMMC2 D0 */
@@ -284,6 +295,17 @@ pins2 {
 				};
 			};
 
+			sdio_pins_sleep_b: sdio-pins-sleep-b-0 {
+				pins {
+					pinmux = <STM32_PINMUX('G', 9, ANALOG)>, /* SDMMC2 D0 */
+						 <STM32_PINMUX('G', 10, ANALOG)>, /* SDMMC2 D1 */
+						 <STM32_PINMUX('B', 3, ANALOG)>, /* SDMMC2 D2 */
+						 <STM32_PINMUX('B', 4, ANALOG)>, /* SDMMC2 D3 */
+						 <STM32_PINMUX('D', 6, ANALOG)>, /* SDMMC2 CLK */
+						 <STM32_PINMUX('D', 7, ANALOG)>; /* SDMMC2 CMD */
+				};
+			};
+
 			can1_pins_a: can1-0 {
 				pins1 {
 					pinmux = <STM32_PINMUX('A', 12, AF9)>; /* CAN1_TX */
-- 
2.42.0

[PATCH 2/2] ARM: dts: stm32: add SDIO pinctrl sleep support on stm32f7 boards

[Ben Wolsieffer]

Use the new analog mode SDIO pin definitions on the STM32F7 boards.

Signed-off-by: Ben Wolsieffer <ben.wolsieffer@hefring.com>
---
 arch/arm/boot/dts/st/stm32746g-eval.dts  | 3 ++-
 arch/arm/boot/dts/st/stm32f746-disco.dts | 3 ++-
 arch/arm/boot/dts/st/stm32f769-disco.dts | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/st/stm32746g-eval.dts b/arch/arm/boot/dts/st/stm32746g-eval.dts
index a293e65141c6..e9ac37b6eca0 100644
--- a/arch/arm/boot/dts/st/stm32746g-eval.dts
+++ b/arch/arm/boot/dts/st/stm32746g-eval.dts
@@ -188,9 +188,10 @@ &sdio1 {
 	status = "okay";
 	vmmc-supply = <&mmc_vcard>;
 	broken-cd;
-	pinctrl-names = "default", "opendrain";
+	pinctrl-names = "default", "opendrain", "sleep";
 	pinctrl-0 = <&sdio_pins_a>;
 	pinctrl-1 = <&sdio_pins_od_a>;
+	pinctrl-2 = <&sdio_pins_sleep_a>;
 	bus-width = <4>;
 };
 
diff --git a/arch/arm/boot/dts/st/stm32f746-disco.dts b/arch/arm/boot/dts/st/stm32f746-disco.dts
index c11616ed5fc6..b50461d676a7 100644
--- a/arch/arm/boot/dts/st/stm32f746-disco.dts
+++ b/arch/arm/boot/dts/st/stm32f746-disco.dts
@@ -103,9 +103,10 @@ &sdio1 {
 	status = "okay";
 	vmmc-supply = <&mmc_vcard>;
 	cd-gpios = <&gpioc 13 GPIO_ACTIVE_LOW>;
-	pinctrl-names = "default", "opendrain";
+	pinctrl-names = "default", "opendrain", "sleep";
 	pinctrl-0 = <&sdio_pins_a>;
 	pinctrl-1 = <&sdio_pins_od_a>;
+	pinctrl-2 = <&sdio_pins_sleep_a>;
 	bus-width = <4>;
 };
 
diff --git a/arch/arm/boot/dts/st/stm32f769-disco.dts b/arch/arm/boot/dts/st/stm32f769-disco.dts
index b038d0ed39e8..5d12ae25b327 100644
--- a/arch/arm/boot/dts/st/stm32f769-disco.dts
+++ b/arch/arm/boot/dts/st/stm32f769-disco.dts
@@ -131,9 +131,10 @@ &sdio2 {
 	vmmc-supply = <&mmc_vcard>;
 	cd-gpios = <&gpioi 15 GPIO_ACTIVE_LOW>;
 	broken-cd;
-	pinctrl-names = "default", "opendrain";
+	pinctrl-names = "default", "opendrain", "sleep";
 	pinctrl-0 = <&sdio_pins_b>;
 	pinctrl-1 = <&sdio_pins_od_b>;
+	pinctrl-2 = <&sdio_pins_sleep_b>;
 	bus-width = <4>;
 };
 
-- 
2.42.0

Re: [PATCH 0/2] ARM: dts: stm32: add SDIO pinctrl sleep support on stm32f7

[Alexandre TORGUE]

Hi Ben

On 10/2/23 19:13, Ben Wolsieffer wrote:
> Add support for switching the SDIO pins on the STM32F7 series to analog
> mode when not in use to save power.
> 
> I don't have any of the disco/eval boards to test, but I have tested
> this on an Emcraft STM32F746 SOM with a UCL-SOM-BSB carrier (using an
> out-of-tree DT) and our custom carrier board.
> 
> Ben Wolsieffer (2):
>    ARM: dts: stm32: add stm32f7 SDIO sleep pins
>    ARM: dts: stm32: add SDIO pinctrl sleep support on stm32f7 boards
> 
>   arch/arm/boot/dts/st/stm32746g-eval.dts   |  3 ++-
>   arch/arm/boot/dts/st/stm32f7-pinctrl.dtsi | 22 ++++++++++++++++++++++
>   arch/arm/boot/dts/st/stm32f746-disco.dts  |  3 ++-
>   arch/arm/boot/dts/st/stm32f769-disco.dts  |  3 ++-
>   4 files changed, 28 insertions(+), 3 deletions(-)
> 

Series applied on stm32-next.

Regards
Alex