| Message ID | 20240507122158.3739291-4-n-francis@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: ti: Add TPS6287 nodes | expand |
Hi Neha On 5/7/2024 5:51 PM, Neha Malcom Francis wrote: > Add DTS node for LP87334E PMIC and two TPS6287x high current buck > converters. > > LP87334E is responsible for supplying power to the MCU and MAIN domains > as well as to LPDDR4. The two TPS6287x supply power to the MAIN > domain for AVS and other core supplies. > > Signed-off-by: Neha Malcom Francis <n-francis@ti.com> > Link: https://www.ti.com/lit/pdf/slda060 > --- > [..] > + tps62873b: tps62873@43 { > + compatible = "ti,tps6287x"; > + bootph-pre-ram; I think, we don;t need bootph-pre-ram for fixed regulator, Please check once, if really bootph-pre-ram needed here > + reg = <0x43>; > + regulator-name = "VDD_CORE_0V8"; > + regulator-min-microvolt = <800000>; > + regulator-max-microvolt = <800000>; > + regulator-boot-on; > + regulator-always-on; > + }; > +}; > + > &mcu_uart0 { > status = "okay"; > pinctrl-names = "default";
Hi Udit On 08/05/24 10:25, Kumar, Udit wrote: > Hi Neha > > On 5/7/2024 5:51 PM, Neha Malcom Francis wrote: >> Add DTS node for LP87334E PMIC and two TPS6287x high current buck >> converters. >> >> LP87334E is responsible for supplying power to the MCU and MAIN domains >> as well as to LPDDR4. The two TPS6287x supply power to the MAIN >> domain for AVS and other core supplies. >> >> Signed-off-by: Neha Malcom Francis <n-francis@ti.com> >> Link: https://www.ti.com/lit/pdf/slda060 >> --- >> [..] >> + tps62873b: tps62873@43 { >> + compatible = "ti,tps6287x"; >> + bootph-pre-ram; > > I think, we don;t need bootph-pre-ram for fixed regulator, > > Please check once, if really bootph-pre-ram needed here > Yeah, there's no need for the bootloader needing this regulator at present. Will make the change, thanks! > >> + reg = <0x43>; >> + regulator-name = "VDD_CORE_0V8"; >> + regulator-min-microvolt = <800000>; >> + regulator-max-microvolt = <800000>; >> + regulator-boot-on; >> + regulator-always-on; >> + }; >> +}; >> + >> &mcu_uart0 { >> status = "okay"; >> pinctrl-names = "default";
diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts index d743f023cdd9..5cd8310ea1c3 100644 --- a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts @@ -414,6 +414,84 @@ &wkup_uart0 { pinctrl-0 = <&wkup_uart0_pins_default>; }; +&wkup_i2c0 { + bootph-all; + status = "okay"; + pinctrl-names = "default"; + pinctrl-0 = <&wkup_i2c0_pins_default>; + clock-frequency = <400000>; + + lp8733: pmic@60 { + compatible = "ti,lp8733"; + reg = <0x60>; + + buck0-in-supply = <&vsys_3v3>; + buck1-in-supply = <&vsys_3v3>; + ldo0-in-supply = <&vsys_3v3>; + ldo1-in-supply = <&vsys_3v3>; + + lp8733_regulators: regulators { + lp8733_buck0_reg: buck0 { + /* FB_B0 -> LP8733-BUCK1 - VDD_MCU_0V85 */ + regulator-name = "lp8733-buck0"; + regulator-min-microvolt = <850000>; + regulator-max-microvolt = <850000>; + regulator-always-on; + regulator-boot-on; + }; + + lp8733_buck1_reg: buck1 { + /* FB_B1 -> LP8733-BUCK2 - VDD_DDR_1V1 */ + regulator-name = "lp8733-buck1"; + regulator-min-microvolt = <1100000>; + regulator-max-microvolt = <1100000>; + regulator-always-on; + regulator-boot-on; + }; + + lp8733_ldo0_reg: ldo0 { + /* LDO0 -> LP8733-LDO1 - VDA_DLL_0V8 */ + regulator-name = "lp8733-ldo0"; + regulator-min-microvolt = <800000>; + regulator-max-microvolt = <800000>; + regulator-boot-on; + regulator-always-on; + }; + + lp8733_ldo1_reg: ldo1 { + /* LDO1 -> LP8733-LDO2 - VDA_LN_1V8 */ + regulator-name = "lp8733-ldo1"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; + regulator-boot-on; + }; + }; + }; + + tps62873a: tps62873@40 { + compatible = "ti,tps6287x"; + bootph-pre-ram; + reg = <0x40>; + regulator-name = "VDD_CPU_AVS"; + regulator-min-microvolt = <600000>; + regulator-max-microvolt = <900000>; + regulator-boot-on; + regulator-always-on; + }; + + tps62873b: tps62873@43 { + compatible = "ti,tps6287x"; + bootph-pre-ram; + reg = <0x43>; + regulator-name = "VDD_CORE_0V8"; + regulator-min-microvolt = <800000>; + regulator-max-microvolt = <800000>; + regulator-boot-on; + regulator-always-on; + }; +}; + &mcu_uart0 { status = "okay"; pinctrl-names = "default";
Add DTS node for LP87334E PMIC and two TPS6287x high current buck converters. LP87334E is responsible for supplying power to the MCU and MAIN domains as well as to LPDDR4. The two TPS6287x supply power to the MAIN domain for AVS and other core supplies. Signed-off-by: Neha Malcom Francis <n-francis@ti.com> Link: https://www.ti.com/lit/pdf/slda060 --- .../boot/dts/ti/k3-am68-sk-base-board.dts | 78 +++++++++++++++++++ 1 file changed, 78 insertions(+)