* [PATCH 0/2] arm64: dts: ti: k3-j721e: dtbs_check warnings fixups
@ 2023-04-24 15:48 ` Nishanth Menon
0 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-04-24 15:48 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo, Nishanth Menon
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 0/2] arm64: dts: ti: k3-j721e: dtbs_check warnings fixups
@ 2023-04-24 15:48 ` Nishanth Menon
0 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-04-24 15:48 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo, Nishanth Menon
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible
2023-04-24 15:48 ` Nishanth Menon
@ 2023-04-24 15:48 ` Nishanth Menon
-1 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-04-24 15:48 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo, Nishanth Menon
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>;
--
2.40.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible
@ 2023-04-24 15:48 ` Nishanth Menon
0 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-04-24 15:48 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo, Nishanth Menon
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>;
--
2.40.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Fix fss node name
2023-04-24 15:48 ` Nishanth Menon
@ 2023-04-24 15:48 ` Nishanth Menon
-1 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-04-24 15:48 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo, Nishanth Menon
Switch the fss node name to bus@ to indicate this is a simple-bus.
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 71b8b04a7789..6237e1f3a477 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
@@ -174,7 +174,7 @@ wkup_i2c0: i2c@42120000 {
status = "disabled";
};
- fss: fss@47000000 {
+ fss: bus@47000000 {
compatible = "simple-bus";
reg = <0x0 0x47000000 0x0 0x100>;
#address-cells = <2>;
--
2.40.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Fix fss node name
@ 2023-04-24 15:48 ` Nishanth Menon
0 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-04-24 15:48 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo, Nishanth Menon
Switch the fss node name to bus@ to indicate this is a simple-bus.
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 71b8b04a7789..6237e1f3a477 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
@@ -174,7 +174,7 @@ wkup_i2c0: i2c@42120000 {
status = "disabled";
};
- fss: fss@47000000 {
+ fss: bus@47000000 {
compatible = "simple-bus";
reg = <0x0 0x47000000 0x0 0x100>;
#address-cells = <2>;
--
2.40.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible
2023-04-24 15:48 ` Nishanth Menon
@ 2023-06-05 15:13 ` Raghavendra, Vignesh
-1 siblings, 0 replies; 12+ messages in thread
From: Raghavendra, Vignesh @ 2023-06-05 15:13 UTC (permalink / raw)
To: Nishanth Menon, Krzysztof Kozlowski, Rob Herring
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible
@ 2023-06-05 15:13 ` Raghavendra, Vignesh
0 siblings, 0 replies; 12+ messages in thread
From: Raghavendra, Vignesh @ 2023-06-05 15:13 UTC (permalink / raw)
To: Nishanth Menon, Krzysztof Kozlowski, Rob Herring
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible
2023-06-05 15:13 ` Raghavendra, Vignesh
@ 2023-06-05 20:52 ` Nishanth Menon
-1 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-06-05 20:52 UTC (permalink / raw)
To: Raghavendra, Vignesh
Cc: Krzysztof Kozlowski, Rob Herring, linux-kernel, devicetree,
linux-arm-kernel, Tero Kristo
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.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible
@ 2023-06-05 20:52 ` Nishanth Menon
0 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-06-05 20:52 UTC (permalink / raw)
To: Raghavendra, Vignesh
Cc: Krzysztof Kozlowski, Rob Herring, linux-kernel, devicetree,
linux-arm-kernel, Tero Kristo
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.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/2] arm64: dts: ti: k3-j721e: dtbs_check warnings fixups
2023-04-24 15:48 ` Nishanth Menon
@ 2023-06-06 16:14 ` Nishanth Menon
-1 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-06-06 16:14 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/2] arm64: dts: ti: k3-j721e: dtbs_check warnings fixups
@ 2023-06-06 16:14 ` Nishanth Menon
0 siblings, 0 replies; 12+ messages in thread
From: Nishanth Menon @ 2023-06-06 16:14 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2023-06-06 16:15 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-24 15:48 [PATCH 0/2] arm64: dts: ti: k3-j721e: dtbs_check warnings fixups Nishanth Menon
2023-04-24 15:48 ` Nishanth Menon
2023-04-24 15:48 ` [PATCH 1/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Switch to "ti,j721e-system-controller" compatible Nishanth Menon
2023-04-24 15:48 ` Nishanth Menon
2023-06-05 15:13 ` Raghavendra, Vignesh
2023-06-05 15:13 ` Raghavendra, Vignesh
2023-06-05 20:52 ` Nishanth Menon
2023-06-05 20:52 ` Nishanth Menon
2023-04-24 15:48 ` [PATCH 2/2] arm64: dts: ti: k3-j721e-mcu-wakeup: Fix fss node name Nishanth Menon
2023-04-24 15:48 ` Nishanth Menon
2023-06-06 16:14 ` [PATCH 0/2] arm64: dts: ti: k3-j721e: dtbs_check warnings fixups Nishanth Menon
2023-06-06 16:14 ` Nishanth Menon
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.