Re: [PATCH] arm64: dts: ti: k3-am625-beagleplay: Add boot phase tags for pmic

Hello Anshul,

On 8/13/2026 1:58 PM, Anshul Dalal wrote:
> The PMIC regulators for beagleplay are required for booting, therefore
> this patch moves the corresponding 'bootph-all' tag from the parent
> tps65219 node to all the regulator sub-nodes.
> 

Was there any issue with this platform that is triggering this change 
now ? I think we need a fixes tag here below.

> Signed-off-by: Anshul Dalal <[email protected]>
> ---
>   arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts | 8 +++++++-
>   1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
> index c468b9c5fc09a..5f7265c426bbe 100644
> --- a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
> @@ -681,7 +681,6 @@ rtc: rtc@68 {
>   	};
>   
>   	tps65219: pmic@30 {
> -		bootph-all;

Wouldn't removing this from here prevent the driver from binding at all 
in that phase ? I am not sure if the child tag below would also imply to 
the parent node here, pls check once.

https://github.com/devicetree-org/dt-schema/blob/main/dtschema/schemas/bootph.yaml

Moreover, k3-am62-pocketbeagle2.dts seems to use the same PMIC. Isn't 
the same change required for this platform ?

Regards,
Hari

>   		compatible = "ti,tps65219";
>   		reg = <0x30>;
>   		buck1-supply = <&vsys_5v0>;
> @@ -709,6 +708,7 @@ buck1_reg: buck1 {
>   				regulator-max-microvolt = <850000>;
>   				regulator-boot-on;
>   				regulator-always-on;
> +				bootph-all;
>   			};
>   
>   			buck2_reg: buck2 {
> @@ -717,6 +717,7 @@ buck2_reg: buck2 {
>   				regulator-max-microvolt = <1800000>;
>   				regulator-boot-on;
>   				regulator-always-on;
> +				bootph-all;
>   			};
>   
>   			buck3_reg: buck3 {
> @@ -725,6 +726,7 @@ buck3_reg: buck3 {
>   				regulator-max-microvolt = <1200000>;
>   				regulator-boot-on;
>   				regulator-always-on;
> +				bootph-all;
>   			};
>   
>   			ldo1_reg: ldo1 {
> @@ -739,6 +741,7 @@ ldo1_reg: ldo1 {
>   				regulator-allow-bypass;
>   				regulator-boot-on;
>   				regulator-always-on;
> +				bootph-all;
>   			};
>   
>   			ldo2_reg: ldo2 {
> @@ -747,6 +750,7 @@ ldo2_reg: ldo2 {
>   				regulator-max-microvolt = <850000>;
>   				regulator-boot-on;
>   				regulator-always-on;
> +				bootph-all;
>   			};
>   
>   			ldo3_reg: ldo3 {
> @@ -755,6 +759,7 @@ ldo3_reg: ldo3 {
>   				regulator-max-microvolt = <1800000>;
>   				regulator-boot-on;
>   				regulator-always-on;
> +				bootph-all;
>   			};
>   
>   			ldo4_reg: ldo4 {
> @@ -763,6 +768,7 @@ ldo4_reg: ldo4 {
>   				regulator-max-microvolt = <2500000>;
>   				regulator-boot-on;
>   				regulator-always-on;
> +				bootph-all;
>   			};
>   		};
>   	};
> 
> ---
> base-commit: 848acc8ffe1b7cd5f1bf427b93069becfebc2c9d
> change-id: 20260813-beagleplay_pmic_boot_phase_fix-c04b7f55de63
> 
> Best regards,
> --
> Anshul Dalal <[email protected]>
> 
>