[PATCH v3 0/5] Add PMIC node for J7200

[PATCH v3 0/5] Add PMIC node for J7200

[Gowtham Tammana]

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

v2:
 - rebased the changes are reordered patches 3/4
 - https://lore.kernel.org/u-boot/20210623211454.31393-1-g-tammana@ti.com/

v1:
 - https://lore.kernel.org/u-boot/20200915113633.25449-1-g-tammana@ti.com/


Gowtham Tammana (5):
  power: pmic: tps65941: Add compatible for LP876441
  arm/dts: k3-j7200-r5-common: Add pmic lp876441 node
  arm/dts: k3-j7200-r5-common: Add VTM node
  arm/dts: k3-j7200-r5-common: Hook buck1_reg to vtm supply
  configs: j7200_evm_r5_defconfig: Enable AVS, PMIC and dependent
    configs

 .../arm/dts/k3-j7200-r5-common-proc-board.dts | 38 +++++++++++++++++++
 configs/j7200_evm_r5_defconfig                |  7 ++++
 drivers/power/pmic/tps65941.c                 |  1 +
 include/power/tps65941.h                      |  1 +
 4 files changed, 47 insertions(+)

-- 
2.32.0

[PATCH v3 1/5] power: pmic: tps65941: Add compatible for LP876441

[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

[PATCH v3 2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node

[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

[PATCH v3 3/5] arm/dts: k3-j7200-r5-common: Add VTM node

[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

[PATCH v3 4/5] arm/dts: k3-j7200-r5-common: Hook buck1_reg to vtm supply

[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

[PATCH v3 5/5] configs: j7200_evm_r5_defconfig: Enable AVS, PMIC and dependent configs

[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

Re: [PATCH v3 0/5] Add PMIC node for J7200

[Lokesh Vutla]

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

Re: [PATCH v3 2/5] arm/dts: k3-j7200-r5-common: Add pmic lp876441 node

[Grygorii Strashko]

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