[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241120103946.49779-1-andrei.simion@microchip.com>
Date: Wed, 20 Nov 2024 12:39:47 +0200
From: Andrei Simion <andrei.simion@...rochip.com>
To: <cristian.birsan@...rochip.com>
CC: <alexandre.belloni@...tlin.com>, <claudiu.beznea@...on.dev>,
<conor+dt@...nel.org>, <devicetree@...r.kernel.org>, <krzk+dt@...nel.org>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
<mihai.sain@...rochip.com>, <nicolas.ferre@...rochip.com>, <robh@...nel.org>,
Andrei Simion <andrei.simion@...rochip.com>
Subject: Re: [PATCH 2/2] ARM: dts: microchip: sama5d27_wlsom1_ek: Add no-1-8-v property to sdmmc0 node
> From: Cristian Birsan <cristian.birsan@...rochip.com>
>
> Add no-1-8-v property to sdmmc0 node to keep VDDSDMMC power rail at 3.3V.
> This property will stop the LDO regulator from switching to 1.8V when the
> MMC core detects an UHS SD Card. VDDSDMMC power rail is used by all the
> SDMMC interface pins in GPIO mode (PA0 - PA13).
>
> On this board, PA10 is used as GPIO to enable the power switch controlling
> USB Vbus for the USB Host. The change is needed to fix the PA10 voltage
> level to 3.3V instead of 1.8V.
>
> Fixes: 5d4c3cfb63fe ("ARM: dts: at91: sama5d27_wlsom1: add SAMA5D27 wlsom1 and wlsom1-ek")
> Suggested-by: Mihai Sain <mihai.sain@...rochip.com>
> Signed-off-by: Cristian Birsan <cristian.birsan@...rochip.com>
Hi,
Tested-by: Andrei Simion <andrei.simion@...rochip.com>
> ---
> arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts | 1 +
> 1 file changed, 1 insertion(+)
>
> 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..35a933eec573 100644
> --- a/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts
> +++ b/arch/arm/boot/dts/microchip/at91-sama5d27_wlsom1_ek.dts
> @@ -197,6 +197,7 @@ qspi1_flash: flash@0 {
>
> &sdmmc0 {
> bus-width = <4>;
> + no-1-8-v;
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_sdmmc0_default>;
> status = "okay";
Best Regards,
--
Andrei Simion
Powered by blists - more mailing lists