2023-07-13 19:14:00

by Nishanth Menon

[permalink] [raw]
Subject: [PATCH] arm64: dts: ti: Fix compatible of ti,*-ehrpwm-tbclk

TI EHRPWM compatible is just ti,*-ehrpwm-tbclk without needing a
syscon compatibility.

Fixes the following dtbs_check warnings:
compatible: [''ti,am654-ehrpwm-tbclk, 'syscon'] is too long
compatible: ['ti,am64-epwm-tbclk', 'syscon'] is too long
compatible: ['ti,am62-epwm-tbclk', 'syscon'] is too long

Signed-off-by: Nishanth Menon <[email protected]>
---
Based on next-20230713
Full list of warnings squashed is a bit long.. but
https://gist.github.com/nmenon/5182ac2165297d5b07e2483d5f020e0c is the
delta.

arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +-
4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
index 4a1dab9e40a0..5cea46be40f5 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
@@ -56,7 +56,7 @@ phy_gmii_sel: phy@4044 {
};

epwm_tbclk: clock@4130 {
- compatible = "ti,am62-epwm-tbclk", "syscon";
+ compatible = "ti,am62-epwm-tbclk";
reg = <0x4130 0x4>;
#clock-cells = <1>;
};
diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
index a57ea7130ca3..d3dd8c426dad 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
@@ -222,7 +222,7 @@ phy_gmii_sel: phy@4044 {
};

epwm_tbclk: clock@4140 {
- compatible = "ti,am64-epwm-tbclk", "syscon";
+ compatible = "ti,am64-epwm-tbclk";
reg = <0x4130 0x4>;
#clock-cells = <1>;
};
diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
index 791c6b38993d..350beddc3d3c 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
@@ -503,7 +503,7 @@ dss_oldi_io_ctrl: dss-oldi-io-ctrl@41e0 {
};

ehrpwm_tbclk: clock@4140 {
- compatible = "ti,am654-ehrpwm-tbclk", "syscon";
+ compatible = "ti,am654-ehrpwm-tbclk";
reg = <0x4140 0x18>;
#clock-cells = <1>;
};
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
index 575435c1a984..6c9fe28b6de7 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
@@ -76,7 +76,7 @@ usb_serdes_mux: mux-controller@4000 {
};

ehrpwm_tbclk: clock-controller@4140 {
- compatible = "ti,am654-ehrpwm-tbclk", "syscon";
+ compatible = "ti,am654-ehrpwm-tbclk";
reg = <0x4140 0x18>;
#clock-cells = <1>;
};
--
2.40.0



2023-07-14 04:40:19

by Vignesh Raghavendra

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ti: Fix compatible of ti,*-ehrpwm-tbclk



On 14/07/23 00:17, Nishanth Menon wrote:
> TI EHRPWM compatible is just ti,*-ehrpwm-tbclk without needing a
> syscon compatibility.
>
> Fixes the following dtbs_check warnings:
> compatible: [''ti,am654-ehrpwm-tbclk, 'syscon'] is too long
> compatible: ['ti,am64-epwm-tbclk', 'syscon'] is too long
> compatible: ['ti,am62-epwm-tbclk', 'syscon'] is too long
>
> Signed-off-by: Nishanth Menon <[email protected]>
> ---

Reviewed-by: Vignesh Raghavendra <[email protected]>


--
Regards
Vignesh

2023-07-14 23:31:38

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ti: Fix compatible of ti,*-ehrpwm-tbclk

Hi Nishanth Menon,

On Thu, 13 Jul 2023 13:47:59 -0500, Nishanth Menon wrote:
> TI EHRPWM compatible is just ti,*-ehrpwm-tbclk without needing a
> syscon compatibility.
>
> Fixes the following dtbs_check warnings:
> compatible: [''ti,am654-ehrpwm-tbclk, 'syscon'] is too long
> compatible: ['ti,am64-epwm-tbclk', 'syscon'] is too long
> compatible: ['ti,am62-epwm-tbclk', 'syscon'] is too long
>
> [...]

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/1] arm64: dts: ti: Fix compatible of ti,*-ehrpwm-tbclk
commit: 2a7cc7bedb58d4eedc177e642efedc25d713445a

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] git://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D