* [PATCH v3 1/5] power: pmic: tps65941: Add compatible for LP876441
2021-07-14 20:52 [PATCH v3 0/5] Add PMIC node for J7200 Gowtham Tammana
@ 2021-07-14 20:52 ` Gowtham Tammana
2021-07-14 20:52 ` [PATCH v3 2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node Gowtham Tammana
` (4 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Gowtham Tammana @ 2021-07-14 20:52 UTC (permalink / raw)
To: Lokesh Vutla, jh80.chung, trini; +Cc: praneeth, u-boot, Gowtham Tammana
TI J7200 EVM has lp876441 pmic that is similar to tps65941. Add support
for same with existing driver with new compatible.
Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
---
v3:
- Added Acked by
https://lore.kernel.org/u-boot/e366d12e-8302-4d4e-9fda-d22451b08885@samsung.com/
drivers/power/pmic/tps65941.c | 1 +
include/power/tps65941.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/drivers/power/pmic/tps65941.c b/drivers/power/pmic/tps65941.c
index 3dfc1918d5..114ef4d238 100644
--- a/drivers/power/pmic/tps65941.c
+++ b/drivers/power/pmic/tps65941.c
@@ -72,6 +72,7 @@ static struct dm_pmic_ops tps65941_ops = {
static const struct udevice_id tps65941_ids[] = {
{ .compatible = "ti,tps659411", .data = TPS659411 },
{ .compatible = "ti,tps659413", .data = TPS659413 },
+ { .compatible = "ti,lp876441", .data = LP876441 },
{ }
};
diff --git a/include/power/tps65941.h b/include/power/tps65941.h
index 2d48b31ae3..a2bc6814ba 100644
--- a/include/power/tps65941.h
+++ b/include/power/tps65941.h
@@ -2,6 +2,7 @@
#define TPS659412 0x1
#define TPS659413 0x2
#define TPS659414 0x3
+#define LP876441 0x4
/* I2C device address for pmic tps65941 */
#define TPS65941_I2C_ADDR (0x12 >> 1)
--
2.32.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v3 2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node
2021-07-14 20:52 [PATCH v3 0/5] Add PMIC node for J7200 Gowtham Tammana
2021-07-14 20:52 ` [PATCH v3 1/5] power: pmic: tps65941: Add compatible for LP876441 Gowtham Tammana
@ 2021-07-14 20:52 ` Gowtham Tammana
2021-07-15 16:18 ` Grygorii Strashko
2021-07-14 20:52 ` [PATCH v3 3/5] arm/dts: k3-j7200-r5-common: Add VTM node Gowtham Tammana
` (3 subsequent siblings)
5 siblings, 1 reply; 8+ messages in thread
From: Gowtham Tammana @ 2021-07-14 20:52 UTC (permalink / raw)
To: Lokesh Vutla, jh80.chung, trini; +Cc: praneeth, u-boot, Gowtham Tammana
Add pmic lp876411 node needed for CPU AVS support.
Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
---
.../arm/dts/k3-j7200-r5-common-proc-board.dts | 26 +++++++++++++++++++
1 file changed, 26 insertions(+)
diff --git a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
index 8dc1809979..6ef92145ef 100644
--- a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
+++ b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
@@ -239,6 +239,32 @@
ti,driver-strength-ohm = <50>;
};
+&wkup_i2c0 {
+ u-boot,dm-spl;
+ lp876441: lp876441@4c {
+ compatible = "ti,lp876441";
+ reg = <0x4c>;
+ u-boot,dm-spl;
+ pinctrl-names = "default";
+ pinctrl-0 = <&wkup_i2c0_pins_default>;
+ clock-frequency = <400000>;
+
+ regulators: regulators {
+ u-boot,dm-spl;
+ buck1_reg: buck1 {
+ /*VDD_CPU_AVS_REG*/
+ regulator-name = "buck1";
+ regulator-min-microvolt = <800000>;
+ regulator-max-microvolt = <1250000>;
+ regulator-always-on;
+ regulator-boot-on;
+ u-boot,dm-spl;
+ };
+ };
+ };
+
+};
+
&main_i2c0 {
pinctrl-names = "default";
pinctrl-0 = <&main_i2c0_pins_default>;
--
2.32.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH v3 2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node
2021-07-14 20:52 ` [PATCH v3 2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node Gowtham Tammana
@ 2021-07-15 16:18 ` Grygorii Strashko
0 siblings, 0 replies; 8+ messages in thread
From: Grygorii Strashko @ 2021-07-15 16:18 UTC (permalink / raw)
To: Gowtham Tammana, Lokesh Vutla, jh80.chung, trini; +Cc: praneeth, u-boot
On 14/07/2021 23:52, Gowtham Tammana wrote:
> Add pmic lp876411 node needed for CPU AVS support.
>
> Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
> ---
> .../arm/dts/k3-j7200-r5-common-proc-board.dts | 26 +++++++++++++++++++
> 1 file changed, 26 insertions(+)
>
> diff --git a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
> index 8dc1809979..6ef92145ef 100644
> --- a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
> +++ b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
> @@ -239,6 +239,32 @@
> ti,driver-strength-ohm = <50>;
> };
>
> +&wkup_i2c0 {
> + u-boot,dm-spl;
> + lp876441: lp876441@4c {
> + compatible = "ti,lp876441";
> + reg = <0x4c>;
> + u-boot,dm-spl;
> + pinctrl-names = "default";
> + pinctrl-0 = <&wkup_i2c0_pins_default>;
> + clock-frequency = <400000>;
pls, send follow up fix to place pimux and freq props correctly in i2c node
> +
> + regulators: regulators {
> + u-boot,dm-spl;
> + buck1_reg: buck1 {
> + /*VDD_CPU_AVS_REG*/
> + regulator-name = "buck1";
> + regulator-min-microvolt = <800000>;
> + regulator-max-microvolt = <1250000>;
> + regulator-always-on;
> + regulator-boot-on;
> + u-boot,dm-spl;
> + };
> + };
> + };
> +
> +};
> +
> &main_i2c0 {
> pinctrl-names = "default";
> pinctrl-0 = <&main_i2c0_pins_default>;
>
--
Best regards,
grygorii
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v3 3/5] arm/dts: k3-j7200-r5-common: Add VTM node
2021-07-14 20:52 [PATCH v3 0/5] Add PMIC node for J7200 Gowtham Tammana
2021-07-14 20:52 ` [PATCH v3 1/5] power: pmic: tps65941: Add compatible for LP876441 Gowtham Tammana
2021-07-14 20:52 ` [PATCH v3 2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node Gowtham Tammana
@ 2021-07-14 20:52 ` Gowtham Tammana
2021-07-14 20:52 ` [PATCH v3 4/5] arm/dts: k3-j7200-r5-common: Hook buck1_reg to vtm supply Gowtham Tammana
` (2 subsequent siblings)
5 siblings, 0 replies; 8+ messages in thread
From: Gowtham Tammana @ 2021-07-14 20:52 UTC (permalink / raw)
To: Lokesh Vutla, jh80.chung, trini; +Cc: praneeth, u-boot, Gowtham Tammana
Add voltage and thermal management (VTM) node. The efuse values for the
OPPs are stored under the VTM, and is needed for AVS class 0 support.
Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
---
v3: resolved merge conflicts
arch/arm/dts/k3-j7200-r5-common-proc-board.dts | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
index 6ef92145ef..891e0b6931 100644
--- a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
+++ b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
@@ -89,6 +89,13 @@
<&mcu_secproxy 23>;
u-boot,dm-spl;
};
+
+ wkup_vtm0: vtm@42040000 {
+ compatible = "ti,am654-vtm", "ti,j721e-avs";
+ reg = <0x0 0x42040000 0x0 0x330>;
+ power-domains = <&k3_pds 154 TI_SCI_PD_EXCLUSIVE>;
+ #thermal-sensor-cells = <1>;
+ };
};
&dmsc {
--
2.32.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v3 4/5] arm/dts: k3-j7200-r5-common: Hook buck1_reg to vtm supply
2021-07-14 20:52 [PATCH v3 0/5] Add PMIC node for J7200 Gowtham Tammana
` (2 preceding siblings ...)
2021-07-14 20:52 ` [PATCH v3 3/5] arm/dts: k3-j7200-r5-common: Add VTM node Gowtham Tammana
@ 2021-07-14 20:52 ` Gowtham Tammana
2021-07-14 20:53 ` [PATCH v3 5/5] configs: j7200_evm_r5_defconfig: Enable AVS, PMIC and dependent configs Gowtham Tammana
2021-07-15 11:51 ` [PATCH v3 0/5] Add PMIC node for J7200 Lokesh Vutla
5 siblings, 0 replies; 8+ messages in thread
From: Gowtham Tammana @ 2021-07-14 20:52 UTC (permalink / raw)
To: Lokesh Vutla, jh80.chung, trini; +Cc: praneeth, u-boot, Gowtham Tammana
Hook buck1_reg to vtm avs supply.
Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
---
arch/arm/dts/k3-j7200-r5-common-proc-board.dts | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
index 891e0b6931..b1f9e714d9 100644
--- a/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
+++ b/arch/arm/dts/k3-j7200-r5-common-proc-board.dts
@@ -272,6 +272,11 @@
};
+&wkup_vtm0 {
+ vdd-supply-2 = <&buck1_reg>;
+ u-boot,dm-spl;
+};
+
&main_i2c0 {
pinctrl-names = "default";
pinctrl-0 = <&main_i2c0_pins_default>;
--
2.32.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH v3 5/5] configs: j7200_evm_r5_defconfig: Enable AVS, PMIC and dependent configs
2021-07-14 20:52 [PATCH v3 0/5] Add PMIC node for J7200 Gowtham Tammana
` (3 preceding siblings ...)
2021-07-14 20:52 ` [PATCH v3 4/5] arm/dts: k3-j7200-r5-common: Hook buck1_reg to vtm supply Gowtham Tammana
@ 2021-07-14 20:53 ` Gowtham Tammana
2021-07-15 11:51 ` [PATCH v3 0/5] Add PMIC node for J7200 Lokesh Vutla
5 siblings, 0 replies; 8+ messages in thread
From: Gowtham Tammana @ 2021-07-14 20:53 UTC (permalink / raw)
To: Lokesh Vutla, jh80.chung, trini; +Cc: praneeth, u-boot, Gowtham Tammana
Enable AVS, PMIC and dependent configs.
Signed-off-by: Gowtham Tammana <g-tammana@ti.com>
---
v3: resolved merge conflicts
configs/j7200_evm_r5_defconfig | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/configs/j7200_evm_r5_defconfig b/configs/j7200_evm_r5_defconfig
index b449bdb3b8..862061e5b2 100644
--- a/configs/j7200_evm_r5_defconfig
+++ b/configs/j7200_evm_r5_defconfig
@@ -114,6 +114,13 @@ CONFIG_SPL_PINCTRL=y
CONFIG_PINCTRL_SINGLE=y
CONFIG_POWER_DOMAIN=y
CONFIG_TI_POWER_DOMAIN=y
+CONFIG_TI_SCI_POWER_DOMAIN=y
+CONFIG_DM_PMIC=y
+CONFIG_PMIC_TPS65941=y
+CONFIG_DM_REGULATOR=y
+CONFIG_SPL_DM_REGULATOR=y
+CONFIG_DM_REGULATOR_TPS65941=y
+CONFIG_K3_AVS0=y
CONFIG_K3_SYSTEM_CONTROLLER=y
CONFIG_REMOTEPROC_TI_K3_ARM64=y
CONFIG_DM_RESET=y
--
2.32.0
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH v3 0/5] Add PMIC node for J7200
2021-07-14 20:52 [PATCH v3 0/5] Add PMIC node for J7200 Gowtham Tammana
` (4 preceding siblings ...)
2021-07-14 20:53 ` [PATCH v3 5/5] configs: j7200_evm_r5_defconfig: Enable AVS, PMIC and dependent configs Gowtham Tammana
@ 2021-07-15 11:51 ` Lokesh Vutla
5 siblings, 0 replies; 8+ messages in thread
From: Lokesh Vutla @ 2021-07-15 11:51 UTC (permalink / raw)
To: trini, jh80.chung, Gowtham Tammana; +Cc: Lokesh Vutla, praneeth, u-boot
On Wed, 14 Jul 2021 15:52:55 -0500, Gowtham Tammana wrote:
> The J7200 EVM has PMIC LP876441 for supporting CPU AVS. This patchset
> adds dt nodes, compatible string, and configs to enable the
> corresponding driver.
>
> v3:
> - rebased to resolve minor conflicts against master
>
> [...]
Applied to https://source.denx.de/u-boot/custodians/u-boot-ti.git for-rc, thanks!
[1/5] power: pmic: tps65941: Add compatible for LP876441
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/d4a344c393
[2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/cbd49ed9d6
[3/5] arm/dts: k3-j7200-r5-common: Add VTM node
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/993fa93f2d
[4/5] arm/dts: k3-j7200-r5-common: Hook buck1_reg to vtm supply
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/9925c76752
[5/5] configs: j7200_evm_r5_defconfig: Enable AVS, PMIC and dependent configs
https://source.denx.de/u-boot/custodians/u-boot-ti/-/commit/672758829a
--
Thanks and Regards,
Lokesh
^ permalink raw reply [flat|nested] 8+ messages in thread