[v1] arm64: dts: ti: k3-am62d2-evm: Fix regulator properties

[PATCH] arm64: dts: ti: k3-am62d2-evm: Fix regulator properties

Posted by Paresh Bhagat 4 months, 3 weeks ago

Fix missing supply for regulators TLV7103318QDSERQ1 and TPS22918DBVR.
Correct padconfig and gpio for TLV7103318QDSERQ1.

Reference Docs
Datasheet - https://www.ti.com/lit/ug/sprujd4/sprujd4.pdf
Schematics - https://www.ti.com/lit/zip/sprcal5

Fixes: 1544bca2f188e ("arm64: dts: ti: Add support for AM62D2-EVM")
Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
---

Boot logs
https://gist.github.com/paresh-bhagat12/9a4a1aec5119ceedb386743921432c91

Tag used
next-20250915

 arch/arm64/boot/dts/ti/k3-am62d2-evm.dts | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts b/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
index 83af889e790a..d202484eec3f 100644
--- a/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
+++ b/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
@@ -146,6 +146,7 @@ vdd_mmc1: regulator-4 {
 		regulator-name = "vdd_mmc1";
 		regulator-min-microvolt = <3300000>;
 		regulator-max-microvolt = <3300000>;
+		vin-supply = <&vcc_3v3_sys>;
 		regulator-boot-on;
 		enable-active-high;
 		gpio = <&exp1 3 GPIO_ACTIVE_HIGH>;
@@ -165,14 +166,16 @@ vcc_3v3_sys: regulator-5 {
 	};
 
 	vddshv_sdio: regulator-6 {
+		/* output of TLV7103318QDSERQ1 */
 		compatible = "regulator-gpio";
 		regulator-name = "vddshv_sdio";
 		pinctrl-names = "default";
 		pinctrl-0 = <&vddshv_sdio_pins_default>;
 		regulator-min-microvolt = <1800000>;
 		regulator-max-microvolt = <3300000>;
+		vin-supply = <&vcc_5v0>;
 		regulator-boot-on;
-		gpios = <&main_gpio1 31 GPIO_ACTIVE_HIGH>;
+		gpios = <&main_gpio0 59 GPIO_ACTIVE_HIGH>;
 		states = <1800000 0x0>,
 			 <3300000 0x1>;
 		bootph-all;
@@ -334,7 +337,7 @@ AM62DX_IOPAD(0x01d4, PIN_INPUT, 7) /* (C15) UART0_RTSn.GPIO1_23 */
 
 	vddshv_sdio_pins_default: vddshv-sdio-default-pins {
 		pinctrl-single,pins = <
-			AM62DX_IOPAD(0x1f4, PIN_OUTPUT, 7) /* (M19) GPMC0_CLK.GPIO1_31 */
+			AM62DX_IOPAD(0x00f0, PIN_INPUT, 7) /* (Y21) GPIO0_59 */
 		>;
 		bootph-all;
 	};
-- 
2.34.1

Re: [PATCH] arm64: dts: ti: k3-am62d2-evm: Fix regulator properties

Posted by Shree Ramamoorthy 3 months, 3 weeks ago

Hi Paresh,

On 9/16/25 4:07 AM, Paresh Bhagat wrote:
> Fix missing supply for regulators TLV7103318QDSERQ1 and TPS22918DBVR.
> Correct padconfig and gpio for TLV7103318QDSERQ1.
>
> Reference Docs
> Datasheet - https://www.ti.com/lit/ug/sprujd4/sprujd4.pdf
> Schematics - https://www.ti.com/lit/zip/sprcal5
>
> Fixes: 1544bca2f188e ("arm64: dts: ti: Add support for AM62D2-EVM")
> Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
> ---
>
> Boot logs
> https://gist.github.com/paresh-bhagat12/9a4a1aec5119ceedb386743921432c91

Reviewed-by: Shree Ramamoorthy <s-ramamoorthy@ti.com>

>
> Tag used
> next-20250915
>
>   arch/arm64/boot/dts/ti/k3-am62d2-evm.dts | 7 +++++--
>   1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts b/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
> index 83af889e790a..d202484eec3f 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am62d2-evm.dts
> @@ -146,6 +146,7 @@ vdd_mmc1: regulator-4 {
>   		regulator-name = "vdd_mmc1";
>   		regulator-min-microvolt = <3300000>;
>   		regulator-max-microvolt = <3300000>;
> +		vin-supply = <&vcc_3v3_sys>;
>   		regulator-boot-on;
>   		enable-active-high;
>   		gpio = <&exp1 3 GPIO_ACTIVE_HIGH>;
> @@ -165,14 +166,16 @@ vcc_3v3_sys: regulator-5 {
>   	};
>   
>   	vddshv_sdio: regulator-6 {
> +		/* output of TLV7103318QDSERQ1 */
>   		compatible = "regulator-gpio";
>   		regulator-name = "vddshv_sdio";
>   		pinctrl-names = "default";
>   		pinctrl-0 = <&vddshv_sdio_pins_default>;
>   		regulator-min-microvolt = <1800000>;
>   		regulator-max-microvolt = <3300000>;
> +		vin-supply = <&vcc_5v0>;
>   		regulator-boot-on;
> -		gpios = <&main_gpio1 31 GPIO_ACTIVE_HIGH>;
> +		gpios = <&main_gpio0 59 GPIO_ACTIVE_HIGH>;
>   		states = <1800000 0x0>,
>   			 <3300000 0x1>;
>   		bootph-all;
> @@ -334,7 +337,7 @@ AM62DX_IOPAD(0x01d4, PIN_INPUT, 7) /* (C15) UART0_RTSn.GPIO1_23 */
>   
>   	vddshv_sdio_pins_default: vddshv-sdio-default-pins {
>   		pinctrl-single,pins = <
> -			AM62DX_IOPAD(0x1f4, PIN_OUTPUT, 7) /* (M19) GPMC0_CLK.GPIO1_31 */
> +			AM62DX_IOPAD(0x00f0, PIN_INPUT, 7) /* (Y21) GPIO0_59 */
>   		>;
>   		bootph-all;
>   	};

-- 
Best,
Shree Ramamoorthy
PMIC Software Engineer