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

Manorit Chawdhry m-chawdhry at ti.com
Wed Feb 7 11:08:20 CET 2024


Hi Udit,

On 19:37-20240206, Kumar, Udit wrote:
> 
> 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
> 

Can mark the series for AVS as RFC till then ig, will see what to be
done for DT Sync.

Regards,
Manorit

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


More information about the U-Boot mailing list