[PATCH 09/10] DONOTMERGE: arm: dts: j721s2: Fix Power domain for VTM node.

Kumar, Udit u-kumar1 at ti.com
Tue Feb 6 15:07:29 CET 2024


On 2/6/2024 5:51 PM, Manorit Chawdhry wrote:
> Patch is sent to upstream linux [0].
>
> [0]: https://lore.kernel.org/all/20240201-b4-upstream-j721s2-fix-vtm-devid-v2-0-85fd568b77e3@ti.com/
>
> Signed-off-by: Manorit Chawdhry <m-chawdhry at ti.com>
> ---
>   arch/arm/dts/k3-j721s2-mcu-wakeup.dtsi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/dts/k3-j721s2-mcu-wakeup.dtsi b/arch/arm/dts/k3-j721s2-mcu-wakeup.dtsi
> index 80aa33c58a45..a47cb557dd95 100644
> --- a/arch/arm/dts/k3-j721s2-mcu-wakeup.dtsi
> +++ b/arch/arm/dts/k3-j721s2-mcu-wakeup.dtsi
> @@ -663,7 +663,7 @@
>   		compatible = "ti,j7200-vtm";
>   		reg = <0x00 0x42040000 0x0 0x350>,
>   		      <0x00 0x42050000 0x0 0x350>;
> -		power-domains = <&k3_pds 154 TI_SCI_PD_SHARED>;
> +		power-domains = <&k3_pds 180 TI_SCI_PD_SHARED>;

I kind of disagree with this patch as Do not Merge, I understand you 
want to do this change as part of next sync.

But we got already node in u-boot for VTM with incorrect power-domain.

I am not sure, with incorrect power-domain, how AVS will be behave.

Here are my 2 cents

Either mark this patch part of series or Hold AVS till next sync


>   		#thermal-sensor-cells = <1>;
>   	};
>   
>


More information about the U-Boot mailing list