linux-kernel - Re: [PATCH 1/5] arm64: dts: ti: j721e-sk: Add DT nodes for power regulators

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 for Android: free password hash cracker in your pocket

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

Message-ID: <ecdd7f17-e7c5-40fa-972b-4f30bcb5090e@ti.com>
Date: Tue, 8 Apr 2025 14:15:17 +0530
From: Yemike Abhilash Chandra <y-abhilashchandra@...com>
To: Nishanth Menon <nm@...com>
CC: <vigneshr@...com>, <kristo@...nel.org>, <robh@...nel.org>,
        <krzk+dt@...nel.org>, <conor+dt@...nel.org>, <vaishnav.a@...com>,
        <jai.luthra@...ux.dev>, <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <imx@...ts.linux.dev>,
        <linux-kernel@...r.kernel.org>, <u-kumar1@...com>
Subject: Re: [PATCH 1/5] arm64: dts: ti: j721e-sk: Add DT nodes for power
 regulators

Hi Nishanth,

On 07/04/25 19:11, Nishanth Menon wrote:
> On 17:10-20250401, Yemike Abhilash Chandra wrote:
>> Add device tree nodes for two power regulators on the J721E SK board.
>> vsys_5v0: A fixed regulator representing the 5V supply output from the
>> LM61460 and vdd_sd_dv: A GPIO-controlled TLV71033 regulator.
>>
>> J721E-SK schematics: https://www.ti.com/lit/zip/sprr438
>> Signed-off-by: Yemike Abhilash Chandra <y-abhilashchandra@...com>
>> ---
>>   arch/arm64/boot/dts/ti/k3-j721e-sk.dts | 31 ++++++++++++++++++++++++++
>>   1 file changed, 31 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-sk.dts b/arch/arm64/boot/dts/ti/k3-j721e-sk.dts
>> index 440ef57be294..4965957e6545 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j721e-sk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-j721e-sk.dts
>> @@ -184,6 +184,17 @@ vsys_3v3: fixedregulator-vsys3v3 {
>>   		regulator-boot-on;
>>   	};
>>   
>> +	vsys_5v0: fixedregulator-vsys5v0 {
>> +		/* Output of LM61460 */
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vsys_5v0";
>> +		regulator-min-microvolt = <5000000>;
>> +		regulator-max-microvolt = <5000000>;
>> +		vin-supply = <&vusb_main>;
>> +		regulator-always-on;
>> +		regulator-boot-on;
>> +	};
>> +
>>   	vdd_mmc1: fixedregulator-sd {
>>   		compatible = "regulator-fixed";
>>   		pinctrl-names = "default";
>> @@ -211,6 +222,20 @@ vdd_sd_dv_alt: gpio-regulator-tps659411 {
>>   			 <3300000 0x1>;
>>   	};
>>   
>> +	vdd_sd_dv: gpio-regulator-TLV71033 {
> 
> Where is this used?

This node will serve as the vin-supply for other regulator nodes defined 
within
the device tree overlay. It is intended to be utilized when the overlay 
is applied.
The corresponding usage of this node can be found in Patch 3 of this series.

Thanks and Regards
Yemike Abhilash Chandra
> 
>> +		compatible = "regulator-gpio";
>> +		pinctrl-names = "default";
>> +		pinctrl-0 = <&vdd_sd_dv_pins_default>;
>> +		regulator-name = "tlv71033";
>> +		regulator-min-microvolt = <1800000>;
>> +		regulator-max-microvolt = <3300000>;
>> +		regulator-boot-on;
>> +		vin-supply = <&vsys_5v0>;
>> +		gpios = <&main_gpio0 118 GPIO_ACTIVE_HIGH>;
>> +		states = <1800000 0x0>,
>> +			 <3300000 0x1>;
>> +	};
>> +
>>   	transceiver1: can-phy1 {
>>   		compatible = "ti,tcan1042";
>>   		#phy-cells = <0>;
>> @@ -613,6 +638,12 @@ J721E_WKUP_IOPAD(0xd4, PIN_OUTPUT, 7) /* (G26) WKUP_GPIO0_9 */
>>   		>;
>>   	};
>>   
>> +	vdd_sd_dv_pins_default: vdd-sd-dv-default-pins {
>> +		pinctrl-single,pins = <
>> +			J721E_IOPAD(0x1dc, PIN_INPUT, 7) /* (Y1) SPI1_CLK.GPIO0_118 */
>> +		>;
>> +	};
>> +
>>   	wkup_uart0_pins_default: wkup-uart0-default-pins {
>>   		pinctrl-single,pins = <
>>   			J721E_WKUP_IOPAD(0xa0, PIN_INPUT, 0) /* (J29) WKUP_UART0_RXD */
>> -- 
>> 2.34.1
>>
>