* [PATCH 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes
2024-01-23 8:31 [PATCH 0/8] Add power-supply properties for sdmmc nodes on Microchip boards Mihai Sain
@ 2024-01-23 8:31 ` Mihai Sain
2024-01-23 8:46 ` Krzysztof Kozlowski
2024-01-23 8:31 ` [PATCH 2/8] ARM: dts: microchip: sam9x60ek: " Mihai Sain
` (6 subsequent siblings)
7 siblings, 1 reply; 13+ messages in thread
From: Mihai Sain @ 2024-01-23 8:31 UTC (permalink / raw)
To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, devicetree, linux-arm-kernel,
linux-kernel
Cc: Mihai Sain
The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
---
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
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes
2024-01-23 8:31 ` [PATCH 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes Mihai Sain
@ 2024-01-23 8:46 ` Krzysztof Kozlowski
2024-01-23 9:07 ` Mihai.Sain
0 siblings, 1 reply; 13+ messages in thread
From: Krzysztof Kozlowski @ 2024-01-23 8:46 UTC (permalink / raw)
To: Mihai Sain, claudiu.beznea, robh+dt, krzysztof.kozlowski+dt,
conor+dt, nicolas.ferre, alexandre.belloni, devicetree,
linux-arm-kernel, linux-kernel
On 23/01/2024 09:31, Mihai Sain wrote:
> The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
> Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
>
> Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
> ---
> 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>;
For this and all other patches:
It is surprising that you use the same supply for both. Looks like
inaccurate description. It is possible, but I think unlikely because
usually VQMMC has lower voltage than 3.3V.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 13+ messages in thread* RE: [PATCH 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes
2024-01-23 8:46 ` Krzysztof Kozlowski
@ 2024-01-23 9:07 ` Mihai.Sain
2024-01-23 9:58 ` Krzysztof Kozlowski
2024-02-03 16:55 ` claudiu beznea
0 siblings, 2 replies; 13+ messages in thread
From: Mihai.Sain @ 2024-01-23 9:07 UTC (permalink / raw)
To: krzysztof.kozlowski, claudiu.beznea, robh+dt,
krzysztof.kozlowski+dt, conor+dt, Nicolas.Ferre,
alexandre.belloni, devicetree, linux-arm-kernel, linux-kernel
Hi Krzysztof,
I'm sorry for quoting (email is sent from Outlook).
The sdmmc controller from SAM9 MPUs doesn't support the IO voltage signaling/switching required by the UHS sd-card.
On the electrical schematics of the boards the controller and sd-card power pins are connected to 3V3 regulator.
On the SAMA5 MPUs there is support for IO voltage switching but since we have issues with the tuning procedure required but the UHS cards,
we want to keep vqmmc at 3V3 in order to use high-speed mode.
Best regards,
Mihai Sain
----------------------------------------------------------------------
> 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>;
For this and all other patches:
It is surprising that you use the same supply for both. Looks like inaccurate description. It is possible, but I think unlikely because usually VQMMC has lower voltage than 3.3V.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 13+ messages in thread* Re: [PATCH 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes
2024-01-23 9:07 ` Mihai.Sain
@ 2024-01-23 9:58 ` Krzysztof Kozlowski
2024-02-03 16:55 ` claudiu beznea
1 sibling, 0 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2024-01-23 9:58 UTC (permalink / raw)
To: Mihai.Sain, claudiu.beznea, robh+dt, krzysztof.kozlowski+dt,
conor+dt, Nicolas.Ferre, alexandre.belloni, devicetree,
linux-arm-kernel, linux-kernel
On 23/01/2024 10:07, Mihai.Sain@microchip.com wrote:
> Hi Krzysztof,
>
> I'm sorry for quoting (email is sent from Outlook).
> The sdmmc controller from SAM9 MPUs doesn't support the IO voltage signaling/switching required by the UHS sd-card.
> On the electrical schematics of the boards the controller and sd-card power pins are connected to 3V3 regulator.
> On the SAMA5 MPUs there is support for IO voltage switching but since we have issues with the tuning procedure required but the UHS cards,
> we want to keep vqmmc at 3V3 in order to use high-speed mode.
OK, it is fine for me.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes
2024-01-23 9:07 ` Mihai.Sain
2024-01-23 9:58 ` Krzysztof Kozlowski
@ 2024-02-03 16:55 ` claudiu beznea
1 sibling, 0 replies; 13+ messages in thread
From: claudiu beznea @ 2024-02-03 16:55 UTC (permalink / raw)
To: Mihai.Sain, krzysztof.kozlowski, robh+dt, krzysztof.kozlowski+dt,
conor+dt, Nicolas.Ferre, alexandre.belloni, devicetree,
linux-arm-kernel, linux-kernel
Hi, Mihai,
On 23.01.2024 11:07, Mihai.Sain@microchip.com wrote:
> Hi Krzysztof,
>
> I'm sorry for quoting (email is sent from Outlook).
> The sdmmc controller from SAM9 MPUs doesn't support the IO voltage signaling/switching required by the UHS sd-card.
> On the electrical schematics of the boards the controller and sd-card power pins are connected to 3V3 regulator.
> On the SAMA5 MPUs there is support for IO voltage switching but since we have issues with the tuning procedure required but the UHS cards,
> we want to keep vqmmc at 3V3 in order to use high-speed mode.
Can you capture this information in each patch description?
>
> Best regards,
> Mihai Sain
>
> ----------------------------------------------------------------------
>
>> 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>;
>
> For this and all other patches:
>
> It is surprising that you use the same supply for both. Looks like inaccurate description. It is possible, but I think unlikely because usually VQMMC has lower voltage than 3.3V.
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/8] ARM: dts: microchip: sam9x60ek: Add power-supply properties for sdmmc nodes
2024-01-23 8:31 [PATCH 0/8] Add power-supply properties for sdmmc nodes on Microchip boards Mihai Sain
2024-01-23 8:31 ` [PATCH 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes Mihai Sain
@ 2024-01-23 8:31 ` Mihai Sain
2024-01-23 8:31 ` [PATCH 3/8] ARM: dts: microchip: sama5d27_som1_ek: " Mihai Sain
` (5 subsequent siblings)
7 siblings, 0 replies; 13+ messages in thread
From: Mihai Sain @ 2024-01-23 8:31 UTC (permalink / raw)
To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, devicetree, linux-arm-kernel,
linux-kernel
Cc: Mihai Sain
The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
---
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
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 3/8] ARM: dts: microchip: sama5d27_som1_ek: Add power-supply properties for sdmmc nodes
2024-01-23 8:31 [PATCH 0/8] Add power-supply properties for sdmmc nodes on Microchip boards Mihai Sain
2024-01-23 8:31 ` [PATCH 1/8] ARM: dts: microchip: sam9x60_curiosity: Add power-supply properties for sdmmc nodes Mihai Sain
2024-01-23 8:31 ` [PATCH 2/8] ARM: dts: microchip: sam9x60ek: " Mihai Sain
@ 2024-01-23 8:31 ` Mihai Sain
2024-01-23 8:31 ` [PATCH 4/8] ARM: dts: microchip: sama5d27_wlsom1: Add power-supply property for sdmmc1 node Mihai Sain
` (4 subsequent siblings)
7 siblings, 0 replies; 13+ messages in thread
From: Mihai Sain @ 2024-01-23 8:31 UTC (permalink / raw)
To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, devicetree, linux-arm-kernel,
linux-kernel
Cc: Mihai Sain
The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
---
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
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 4/8] ARM: dts: microchip: sama5d27_wlsom1: Add power-supply property for sdmmc1 node
2024-01-23 8:31 [PATCH 0/8] Add power-supply properties for sdmmc nodes on Microchip boards Mihai Sain
` (2 preceding siblings ...)
2024-01-23 8:31 ` [PATCH 3/8] ARM: dts: microchip: sama5d27_som1_ek: " Mihai Sain
@ 2024-01-23 8:31 ` Mihai Sain
2024-01-23 8:31 ` [PATCH 5/8] ARM: dts: microchip: sama5d27_wlsom1_ek: Add power-supply property for sdmmc0 node Mihai Sain
` (3 subsequent siblings)
7 siblings, 0 replies; 13+ messages in thread
From: Mihai Sain @ 2024-01-23 8:31 UTC (permalink / raw)
To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, devicetree, linux-arm-kernel,
linux-kernel
Cc: Mihai Sain
The sdmmc1 controller is powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc1 node.
Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
---
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
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 5/8] ARM: dts: microchip: sama5d27_wlsom1_ek: Add power-supply property for sdmmc0 node
2024-01-23 8:31 [PATCH 0/8] Add power-supply properties for sdmmc nodes on Microchip boards Mihai Sain
` (3 preceding siblings ...)
2024-01-23 8:31 ` [PATCH 4/8] ARM: dts: microchip: sama5d27_wlsom1: Add power-supply property for sdmmc1 node Mihai Sain
@ 2024-01-23 8:31 ` Mihai Sain
2024-01-23 8:31 ` [PATCH 6/8] ARM: dts: microchip: sama5d29_curiosity: Add power-supply properties for sdmmc nodes Mihai Sain
` (2 subsequent siblings)
7 siblings, 0 replies; 13+ messages in thread
From: Mihai Sain @ 2024-01-23 8:31 UTC (permalink / raw)
To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, devicetree, linux-arm-kernel,
linux-kernel
Cc: Mihai Sain
The sdmmc0 controller is powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc0 node.
Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
---
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
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 6/8] ARM: dts: microchip: sama5d29_curiosity: Add power-supply properties for sdmmc nodes
2024-01-23 8:31 [PATCH 0/8] Add power-supply properties for sdmmc nodes on Microchip boards Mihai Sain
` (4 preceding siblings ...)
2024-01-23 8:31 ` [PATCH 5/8] ARM: dts: microchip: sama5d27_wlsom1_ek: Add power-supply property for sdmmc0 node Mihai Sain
@ 2024-01-23 8:31 ` Mihai Sain
2024-01-23 8:31 ` [PATCH 7/8] ARM: dts: microchip: sama5d2_icp: Add power-supply property for sdmmc0 node Mihai Sain
2024-01-23 8:31 ` [PATCH 8/8] ARM: dts: microchip: sama5d2_xplained: " Mihai Sain
7 siblings, 0 replies; 13+ messages in thread
From: Mihai Sain @ 2024-01-23 8:31 UTC (permalink / raw)
To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, devicetree, linux-arm-kernel,
linux-kernel
Cc: Mihai Sain
The sdmmc0 and sdmmc1 controllers are powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc nodes.
Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
---
arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts
index 6b02b7bcfd49..4a86597d089a 100644
--- a/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts
+++ b/arch/arm/boot/dts/microchip/at91-sama5d29_curiosity.dts
@@ -504,6 +504,8 @@ &sdmmc0 {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_sdmmc0_default>;
disable-wp;
+ vmmc-supply = <&vdd_3v3>;
+ vqmmc-supply = <&vdd_3v3>;
status = "okay";
};
@@ -512,6 +514,8 @@ &sdmmc1 {
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_sdmmc1_default>;
disable-wp;
+ vmmc-supply = <&vdd_3v3>;
+ vqmmc-supply = <&vdd_3v3>;
status = "okay";
};
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 7/8] ARM: dts: microchip: sama5d2_icp: Add power-supply property for sdmmc0 node
2024-01-23 8:31 [PATCH 0/8] Add power-supply properties for sdmmc nodes on Microchip boards Mihai Sain
` (5 preceding siblings ...)
2024-01-23 8:31 ` [PATCH 6/8] ARM: dts: microchip: sama5d29_curiosity: Add power-supply properties for sdmmc nodes Mihai Sain
@ 2024-01-23 8:31 ` Mihai Sain
2024-01-23 8:31 ` [PATCH 8/8] ARM: dts: microchip: sama5d2_xplained: " Mihai Sain
7 siblings, 0 replies; 13+ messages in thread
From: Mihai Sain @ 2024-01-23 8:31 UTC (permalink / raw)
To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, devicetree, linux-arm-kernel,
linux-kernel
Cc: Mihai Sain
The sdmmc0 controller is powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc0 node.
Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
---
arch/arm/boot/dts/microchip/at91-sama5d2_icp.dts | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/boot/dts/microchip/at91-sama5d2_icp.dts b/arch/arm/boot/dts/microchip/at91-sama5d2_icp.dts
index 999adeca6f33..adcb3240e5f5 100644
--- a/arch/arm/boot/dts/microchip/at91-sama5d2_icp.dts
+++ b/arch/arm/boot/dts/microchip/at91-sama5d2_icp.dts
@@ -712,6 +712,8 @@ &sdmmc0 {
bus-width = <4>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_sdmmc0_default>;
+ vmmc-supply = <&vdd_io_reg>; // 3.3V
+ vqmmc-supply = <&vdd_io_reg>; // 3.3V
status = "okay";
};
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 8/8] ARM: dts: microchip: sama5d2_xplained: Add power-supply property for sdmmc0 node
2024-01-23 8:31 [PATCH 0/8] Add power-supply properties for sdmmc nodes on Microchip boards Mihai Sain
` (6 preceding siblings ...)
2024-01-23 8:31 ` [PATCH 7/8] ARM: dts: microchip: sama5d2_icp: Add power-supply property for sdmmc0 node Mihai Sain
@ 2024-01-23 8:31 ` Mihai Sain
7 siblings, 0 replies; 13+ messages in thread
From: Mihai Sain @ 2024-01-23 8:31 UTC (permalink / raw)
To: claudiu.beznea, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, devicetree, linux-arm-kernel,
linux-kernel
Cc: Mihai Sain
The sdmmc0 controller is powered from 3.3V regulator.
Add vmmc-supply and vqmmc-supply properties to sdmmc0 node.
Signed-off-by: Mihai Sain <mihai.sain@microchip.com>
---
arch/arm/boot/dts/microchip/at91-sama5d2_xplained.dts | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/boot/dts/microchip/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/microchip/at91-sama5d2_xplained.dts
index 6680031387e8..9b7e56790a5a 100644
--- a/arch/arm/boot/dts/microchip/at91-sama5d2_xplained.dts
+++ b/arch/arm/boot/dts/microchip/at91-sama5d2_xplained.dts
@@ -67,6 +67,8 @@ sdmmc0: sdio-host@a0000000 {
pinctrl-0 = <&pinctrl_sdmmc0_default>;
non-removable;
mmc-ddr-3_3v;
+ vmmc-supply = <&vdd_3v3_reg>;
+ vqmmc-supply = <&vdd_3v3_reg>;
status = "okay";
};
--
2.43.0
^ permalink raw reply related [flat|nested] 13+ messages in thread