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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <19b16368-9ec8-40d8-9389-1dc48f28dd2a@ti.com>
Date: Thu, 16 Oct 2025 12:53:36 -0500
From: Shree Ramamoorthy <s-ramamoorthy@...com>
To: Paresh Bhagat <p-bhagat@...com>, <nm@...com>, <vigneshr@...com>,
        <praneeth@...com>
CC: <kristo@...nel.org>, <robh@...nel.org>, <krzk+dt@...nel.org>,
        <conor+dt@...nel.org>, <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <khasim@...com>, <v-singh1@...com>, <afd@...com>, <bb@...com>
Subject: Re: [PATCH] arm64: dts: ti: k3-am62d2-evm: Fix regulator properties

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@...com>
> ---
>
> Boot logs
> https://gist.github.com/paresh-bhagat12/9a4a1aec5119ceedb386743921432c91

Reviewed-by: Shree Ramamoorthy <s-ramamoorthy@...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