2023-04-24 15:57:02

by Nishanth Menon

[permalink] [raw]
Subject: [PATCH 0/2] arm64: dts: ti: k3-j721e: dtbs_check warnings fixups

Hi,

Minor series of two patches to fixup the pending dtbs_check warnings in
j721e.

Nishanth Menon (2):
arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to
"ti,j721e-system-controller" compatible
arm64: dts: ti: k3-j721e-mcu-wakeup: Fix fss node name

arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

--
2.40.0


2023-04-24 16:01:49

by Nishanth Menon

[permalink] [raw]
Subject: [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible

Switch to "ti,j721e-system-controller" compatible to be more specific.

Signed-off-by: Nishanth Menon <[email protected]>
---
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>;
--
2.40.0

2023-06-05 15:30:43

by Vignesh Raghavendra

[permalink] [raw]
Subject: Re: [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible

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 <[email protected]>
> ---
> 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/[email protected]/

Regards
Vignesh

2023-06-05 21:13:12

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible

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 <[email protected]>
> > ---
> > 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/[email protected]/

OK. Let me relook at the following series as well:
https://lore.kernel.org/all/[email protected]/
-> AM65x dtbs_check warning fixups??

https://lore.kernel.org/all/[email protected]/
-> J721e dtbs_check warning fixups??

https://lore.kernel.org/linux-arm-kernel/[email protected]/
-> J7200 dtbs_check warning fixups??

I will try and respond/ respin the relevant patchsets.

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D

2023-06-06 16:31:30

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH 0/2] arm64: dts: ti: k3-j721e: dtbs_check warnings fixups

On 10:48-20230424, Nishanth Menon wrote:
> Hi,
>
> Minor series of two patches to fixup the pending dtbs_check warnings in
> j721e.
>
> Nishanth Menon (2):
> arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to
> "ti,j721e-system-controller" compatible
> arm64: dts: ti: k3-j721e-mcu-wakeup: Fix fss node name
>
> arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> --
> 2.40.0
>
As discussed in https://lore.kernel.org/all/20230605205220.rjmcsi5tjn4auqa7@arose/

Lets drop this series, once the dependent patches are merged, we can
switch to bus in the next kernel rev.

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D