[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