| Message ID | 20230424154841.353631-2-nm@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: ti: k3-j721e: dtbs_check warnings fixups | expand |
Hi On 4/24/2023 9:18 PM, Nishanth Menon wrote: > Switch to "ti,j721e-system-controller" compatible to be more specific. > > Signed-off-by: Nishanth Menon <nm@ti.com> > --- > arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > index 24e8125db8c4..71b8b04a7789 100644 > --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > @@ -35,7 +35,7 @@ k3_reset: reset-controller { > }; > > mcu_conf: syscon@40f00000 { > - compatible = "syscon", "simple-mfd"; > + compatible = "ti,j721e-system-controller", "syscon", "simple-mfd"; > reg = <0x0 0x40f00000 0x0 0x20000>; > #address-cells = <1>; > #size-cells = <1>; As Andrew suggested at [0], lets work on getting rid of unneeded "syscon" here and move to simple-bus instead [0] https://lore.kernel.org/all/76da0b98-3274-b047-db11-ecabc117ae11@ti.com/ Regards Vignesh
On 20:43-20230605, Vignesh Raghavendra wrote: > Hi > > On 4/24/2023 9:18 PM, Nishanth Menon wrote: > > Switch to "ti,j721e-system-controller" compatible to be more specific. > > > > Signed-off-by: Nishanth Menon <nm@ti.com> > > --- > > arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > > index 24e8125db8c4..71b8b04a7789 100644 > > --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > > +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi > > @@ -35,7 +35,7 @@ k3_reset: reset-controller { > > }; > > > > mcu_conf: syscon@40f00000 { > > - compatible = "syscon", "simple-mfd"; > > + compatible = "ti,j721e-system-controller", "syscon", "simple-mfd"; > > reg = <0x0 0x40f00000 0x0 0x20000>; > > #address-cells = <1>; > > #size-cells = <1>; > > As Andrew suggested at [0], lets work on getting rid of unneeded > "syscon" here and move to simple-bus instead > > [0] https://lore.kernel.org/all/76da0b98-3274-b047-db11-ecabc117ae11@ti.com/ OK. Let me relook at the following series as well: https://lore.kernel.org/all/20230424144949.244135-1-nm@ti.com/ -> AM65x dtbs_check warning fixups?? https://lore.kernel.org/all/20230424154841.353631-1-nm@ti.com/ -> J721e dtbs_check warning fixups?? https://lore.kernel.org/linux-arm-kernel/20230424173623.477577-1-nm@ti.com/ -> J7200 dtbs_check warning fixups?? I will try and respond/ respin the relevant patchsets.
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi index 24e8125db8c4..71b8b04a7789 100644 --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi @@ -35,7 +35,7 @@ k3_reset: reset-controller { }; mcu_conf: syscon@40f00000 { - compatible = "syscon", "simple-mfd"; + compatible = "ti,j721e-system-controller", "syscon", "simple-mfd"; reg = <0x0 0x40f00000 0x0 0x20000>; #address-cells = <1>; #size-cells = <1>;
Switch to "ti,j721e-system-controller" compatible to be more specific. Signed-off-by: Nishanth Menon <nm@ti.com> --- arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)