2024-02-05 08:02:11

by Mihai Sain

[permalink] [raw]
Subject: [PATCH v2 0/8] Add power-supply properties for sdmmc nodes on Microchip boards

This patch series adds power-supply properties for sdmmc nodes on Microchip boards.

Changes in v2:
--------------

* Update the commit description for each board.
* The goal is to keep the vqmmc at 3V3 in order to use the sd high-speed mode.

Mihai Sain (8):
ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes
ARM: dts: microchip: sam9x60ek: Add power-supply properties for sdmmc nodes
ARM: dts: microchip: sama5d27_som1_ek: Add power-supply properties for sdmmc nodes
ARM: dts: microchip: sama5d27_wlsom1: Add power-supply property for sdmmc1 node
ARM: dts: microchip: sama5d27_wlsom1_ek: Add power-supply property for sdmmc0 node
ARM: dts: microchip: sama5d29_curiosity: Add power-supply properties for sdmmc nodes
ARM: dts: microchip: sama5d2_icp: Add power-supply property for sdmmc0 node
ARM: dts: microchip: sama5d2_xplained: Add power-supply property for sdmmc0 node

arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts | 4 ++++
arch/arm/boot/dts/microchip/at91-sam9x60ek.dts | 4 ++++
arch/arm/boot/dts/microchip/at91-sama5d27_som1_ek.dts | 4 ++++
arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1.dtsi | 2 ++
arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts | 2 ++
arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts | 4 ++++
arch/arm/boot/dts/microchip/at91-sama5d2_icp.dts | 2 ++
arch/arm/boot/dts/microchip/at91-sama5d2_xplained.dts | 2 ++
8 files changed, 24 insertions(+)

--
2.43.0



2024-02-05 08:04:25

by Mihai Sain

[permalink] [raw]
Subject: [PATCH v2 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes

The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
The sdmmc controller from SAM9X60 MPU doesn't support the IO voltage signaling/switching required by the UHS sd-card.
In order to use the sd high-speed mode, keep vqmmc at 3V3.

Signed-off-by: Mihai Sain <[email protected]>
---
arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
index c6fbdd29019f..457c54dde0b7 100644
--- a/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
+++ b/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
@@ -452,6 +452,8 @@ &sdmmc0 {
pinctrl-0 = <&pinctrl_sdmmc0_default &pinctrl_sdmmc0_cd>;
cd-gpios = <&pioA 25 GPIO_ACTIVE_LOW>;
disable-wp;
+ vmmc-supply = <&vdd1_3v3>;
+ vqmmc-supply = <&vdd1_3v3>;
status = "okay";
};

@@ -460,6 +462,8 @@ &sdmmc1 {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_sdmmc1_default>;
disable-wp;
+ vmmc-supply = <&vdd1_3v3>;
+ vqmmc-supply = <&vdd1_3v3>;
status = "okay";
};

--
2.43.0


2024-02-05 08:04:59

by Mihai Sain

[permalink] [raw]
Subject: [PATCH v2 2/8] ARM: dts: microchip: sam9x60ek: Add power-supply properties for sdmmc nodes

The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
The sdmmc controller from SAM9X60 MPU doesn't support the IO voltage signaling/switching required by the UHS sd-card.
In order to use the sd high-speed mode, keep vqmmc at 3V3.

Signed-off-by: Mihai Sain <[email protected]>
---
arch/arm/boot/dts/microchip/at91-sam9x60ek.dts | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/microchip/at91-sam9x60ek.dts b/arch/arm/boot/dts/microchip/at91-sam9x60ek.dts
index f3cbb675cea4..b19a0956dc97 100644
--- a/arch/arm/boot/dts/microchip/at91-sam9x60ek.dts
+++ b/arch/arm/boot/dts/microchip/at91-sam9x60ek.dts
@@ -560,6 +560,8 @@ &sdmmc0 {
status = "okay";
cd-gpios = <&pioA 23 GPIO_ACTIVE_LOW>;
disable-wp;
+ vmmc-supply = <&vdd1_3v3>;
+ vqmmc-supply = <&vdd1_3v3>;
};

&sdmmc1 {
@@ -568,6 +570,8 @@ &sdmmc1 {
pinctrl-0 = <&pinctrl_sdmmc1_default>;
no-1-8-v;
non-removable;
+ vmmc-supply = <&vdd1_3v3>;
+ vqmmc-supply = <&vdd1_3v3>;
status = "disabled"; /* Conflict with flx4. */
};

--
2.43.0


2024-02-05 08:05:08

by Mihai Sain

[permalink] [raw]
Subject: [PATCH v2 4/8] ARM: dts: microchip: sama5d27_wlsom1: Add power-supply property for sdmmc1 node

The sdmmc1 controller is powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc1 node.
The sdmmc controller from SAMA5D2 MPU has support for IO voltage signaling/switching required by the UHS sd-card.
In order to avoid the issues from the tuning procedure required by the UHS cards,
keep the vqmmc at 3V3 to use the sd high-speed mode.

Signed-off-by: Mihai Sain <[email protected]>
---
arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1.dtsi | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1.dtsi b/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1.dtsi
index 4617805c7748..96819ea24cbd 100644
--- a/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1.dtsi
+++ b/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1.dtsi
@@ -368,6 +368,8 @@ &sdmmc1 {
no-1-8-v;
non-removable;
bus-width = <4>;
+ vmmc-supply = <&vdd_3v3>;
+ vqmmc-supply = <&vdd_3v3>;
status = "okay";

wilc: wifi@0 {
--
2.43.0


2024-02-05 08:05:24

by Mihai Sain

[permalink] [raw]
Subject: [PATCH v2 5/8] ARM: dts: microchip: sama5d27_wlsom1_ek: Add power-supply property for sdmmc0 node

The sdmmc0 controller is powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc0 node.
The sdmmc controller from SAMA5D2 MPU has support for IO voltage signaling/switching required by the UHS sd-card.
In order to avoid the issues from the tuning procedure required by the UHS cards,
keep the vqmmc at 3V3 to use the sd high-speed mode.

Signed-off-by: Mihai Sain <[email protected]>
---
arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts b/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts
index 15239834d886..7b36e1970bb7 100644
--- a/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts
+++ b/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts
@@ -199,6 +199,8 @@ &sdmmc0 {
bus-width = <4>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_sdmmc0_default>;
+ vmmc-supply = <&vdd_3v3>;
+ vqmmc-supply = <&vdd_3v3>;
status = "okay";
};

--
2.43.0


2024-02-05 08:07:14

by Mihai Sain

[permalink] [raw]
Subject: [PATCH v2 3/8] ARM: dts: microchip: sama5d27_som1_ek: Add power-supply properties for sdmmc nodes

The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
The sdmmc controller from SAMA5D2 MPU has support for IO voltage signaling/switching required by the UHS sd-card.
In order to avoid the issues from the tuning procedure required by the UHS cards,
keep the vqmmc at 3V3 to use the sd high-speed mode.

Signed-off-by: Mihai Sain <[email protected]>
---
arch/arm/boot/dts/microchip/at91-sama5d27_som1_ek.dts | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/microchip/at91-sama5d27_som1_ek.dts b/arch/arm/boot/dts/microchip/at91-sama5d27_som1_ek.dts
index f3ffb8f01d8a..255ee0640133 100644
--- a/arch/arm/boot/dts/microchip/at91-sama5d27_som1_ek.dts
+++ b/arch/arm/boot/dts/microchip/at91-sama5d27_som1_ek.dts
@@ -56,6 +56,8 @@ sdmmc0: sdio-host@a0000000 {
bus-width = <8>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_sdmmc0_default>;
+ vmmc-supply = <&vddin_3v3>;
+ vqmmc-supply = <&vddin_3v3>;
status = "okay";
};

@@ -63,6 +65,8 @@ sdmmc1: sdio-host@b0000000 {
bus-width = <4>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_sdmmc1_default>;
+ vmmc-supply = <&vddin_3v3>;
+ vqmmc-supply = <&vddin_3v3>;
status = "okay";
};

--
2.43.0


2024-02-06 06:01:31

by Claudiu

[permalink] [raw]
Subject: Re: [PATCH v2 2/8] ARM: dts: microchip: sam9x60ek: Add power-supply properties for sdmmc nodes

Hi, Mihai,

On 05.02.2024 10:00, Mihai Sain wrote:
> The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
> Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
> The sdmmc controller from SAM9X60 MPU doesn't support the IO voltage signaling/switching required by the UHS sd-card.

Please wrap the line at 75 chars (see [1]).

Thank you,
Claudiu Beznea

[1]
https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html#the-canonical-patch-format.

> In order to use the sd high-speed mode, keep vqmmc at 3V3.
>
> Signed-off-by: Mihai Sain <[email protected]>
> ---
> arch/arm/boot/dts/microchip/at91-sam9x60ek.dts | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm/boot/dts/microchip/at91-sam9x60ek.dts b/arch/arm/boot/dts/microchip/at91-sam9x60ek.dts
> index f3cbb675cea4..b19a0956dc97 100644
> --- a/arch/arm/boot/dts/microchip/at91-sam9x60ek.dts
> +++ b/arch/arm/boot/dts/microchip/at91-sam9x60ek.dts
> @@ -560,6 +560,8 @@ &sdmmc0 {
> status = "okay";
> cd-gpios = <&pioA 23 GPIO_ACTIVE_LOW>;
> disable-wp;
> + vmmc-supply = <&vdd1_3v3>;
> + vqmmc-supply = <&vdd1_3v3>;
> };
>
> &sdmmc1 {
> @@ -568,6 +570,8 @@ &sdmmc1 {
> pinctrl-0 = <&pinctrl_sdmmc1_default>;
> no-1-8-v;
> non-removable;
> + vmmc-supply = <&vdd1_3v3>;
> + vqmmc-supply = <&vdd1_3v3>;
> status = "disabled"; /* Conflict with flx4. */
> };
>