* [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck
@ 2022-10-24 15:16 Matt Ranostay
2022-10-28 5:37 ` Gole, Dhruva
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Matt Ranostay @ 2022-10-24 15:16 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh+dt, krzysztof.kozlowski+dt,
miquel.raynal
Cc: linux-arm-kernel, devicetree, Matt Ranostay
Avoid the following warnings from dt-schema by just renaming the
clock-names string from adc_tsc_fck to fck so it matches the values in
ti,am3359-tscadc.yaml
tscadc@40200000: clock-names:0: 'fck' was expected
Signed-off-by: Matt Ranostay <mranostay@ti.com>
---
arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 4 ++--
arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 4 ++--
4 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
index d6aa23681bbe..b61c15427bf7 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
@@ -612,7 +612,7 @@ tscadc0: tscadc@28001000 {
assigned-clocks = <&k3_clks 0 0>;
assigned-clock-parents = <&k3_clks 0 3>;
assigned-clock-rates = <60000000>;
- clock-names = "adc_tsc_fck";
+ clock-names = "fck";
adc {
#io-channel-cells = <1>;
diff --git a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
index 8d592bf41d6f..1807ec4730cb 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
@@ -85,7 +85,7 @@ tscadc0: tscadc@40200000 {
clocks = <&k3_clks 0 2>;
assigned-clocks = <&k3_clks 0 2>;
assigned-clock-rates = <60000000>;
- clock-names = "adc_tsc_fck";
+ clock-names = "fck";
dmas = <&mcu_udmap 0x7100>,
<&mcu_udmap 0x7101 >;
dma-names = "fifo0", "fifo1";
@@ -103,7 +103,7 @@ tscadc1: tscadc@40210000 {
clocks = <&k3_clks 1 2>;
assigned-clocks = <&k3_clks 1 2>;
assigned-clock-rates = <60000000>;
- clock-names = "adc_tsc_fck";
+ clock-names = "fck";
dmas = <&mcu_udmap 0x7102>,
<&mcu_udmap 0x7103>;
dma-names = "fifo0", "fifo1";
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
index 2025aab4b66f..3d9190a869c5 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
@@ -325,7 +325,7 @@ tscadc0: tscadc@40200000 {
clocks = <&k3_clks 0 1>;
assigned-clocks = <&k3_clks 0 3>;
assigned-clock-rates = <60000000>;
- clock-names = "adc_tsc_fck";
+ clock-names = "fck";
dmas = <&main_udmap 0x7400>,
<&main_udmap 0x7401>;
dma-names = "fifo0", "fifo1";
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 5d9ec221fa34..7799db6a167c 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
@@ -216,7 +216,7 @@ tscadc0: tscadc@40200000 {
clocks = <&k3_clks 0 1>;
assigned-clocks = <&k3_clks 0 3>;
assigned-clock-rates = <60000000>;
- clock-names = "adc_tsc_fck";
+ clock-names = "fck";
dmas = <&main_udmap 0x7400>,
<&main_udmap 0x7401>;
dma-names = "fifo0", "fifo1";
@@ -235,7 +235,7 @@ tscadc1: tscadc@40210000 {
clocks = <&k3_clks 1 1>;
assigned-clocks = <&k3_clks 1 3>;
assigned-clock-rates = <60000000>;
- clock-names = "adc_tsc_fck";
+ clock-names = "fck";
dmas = <&main_udmap 0x7402>,
<&main_udmap 0x7403>;
dma-names = "fifo0", "fifo1";
--
2.38.GIT
_______________________________________________
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] 5+ messages in thread* Re: [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck
2022-10-24 15:16 [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck Matt Ranostay
@ 2022-10-28 5:37 ` Gole, Dhruva
2022-11-02 20:14 ` Judith Mendez
2022-11-03 7:23 ` Nishanth Menon
2 siblings, 0 replies; 5+ messages in thread
From: Gole, Dhruva @ 2022-10-28 5:37 UTC (permalink / raw)
To: Matt Ranostay, nm, vigneshr, kristo, robh+dt,
krzysztof.kozlowski+dt, miquel.raynal
Cc: linux-arm-kernel, devicetree
On 10/24/2022 8:46 PM, Matt Ranostay wrote:
> Avoid the following warnings from dt-schema by just renaming the
> clock-names string from adc_tsc_fck to fck so it matches the values in
> ti,am3359-tscadc.yaml
>
> tscadc@40200000: clock-names:0: 'fck' was expected
>
> Signed-off-by: Matt Ranostay <mranostay@ti.com>
> ---
This patch LGTM.
Reviewed-by: Dhruva Gole <d-gole@ti.com>
> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 4 ++--
> arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 4 ++--
> 4 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> index d6aa23681bbe..b61c15427bf7 100644
> --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> @@ -612,7 +612,7 @@ tscadc0: tscadc@28001000 {
> assigned-clocks = <&k3_clks 0 0>;
> assigned-clock-parents = <&k3_clks 0 3>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
>
> adc {
> #io-channel-cells = <1>;
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> index 8d592bf41d6f..1807ec4730cb 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> @@ -85,7 +85,7 @@ tscadc0: tscadc@40200000 {
> clocks = <&k3_clks 0 2>;
> assigned-clocks = <&k3_clks 0 2>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&mcu_udmap 0x7100>,
> <&mcu_udmap 0x7101 >;
> dma-names = "fifo0", "fifo1";
> @@ -103,7 +103,7 @@ tscadc1: tscadc@40210000 {
> clocks = <&k3_clks 1 2>;
> assigned-clocks = <&k3_clks 1 2>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&mcu_udmap 0x7102>,
> <&mcu_udmap 0x7103>;
> dma-names = "fifo0", "fifo1";
> diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> index 2025aab4b66f..3d9190a869c5 100644
> --- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> @@ -325,7 +325,7 @@ tscadc0: tscadc@40200000 {
> clocks = <&k3_clks 0 1>;
> assigned-clocks = <&k3_clks 0 3>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&main_udmap 0x7400>,
> <&main_udmap 0x7401>;
> dma-names = "fifo0", "fifo1";
> 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 5d9ec221fa34..7799db6a167c 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> @@ -216,7 +216,7 @@ tscadc0: tscadc@40200000 {
> clocks = <&k3_clks 0 1>;
> assigned-clocks = <&k3_clks 0 3>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&main_udmap 0x7400>,
> <&main_udmap 0x7401>;
> dma-names = "fifo0", "fifo1";
> @@ -235,7 +235,7 @@ tscadc1: tscadc@40210000 {
> clocks = <&k3_clks 1 1>;
> assigned-clocks = <&k3_clks 1 3>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&main_udmap 0x7402>,
> <&main_udmap 0x7403>;
> dma-names = "fifo0", "fifo1";
_______________________________________________
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] 5+ messages in thread* Re: [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck
2022-10-24 15:16 [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck Matt Ranostay
2022-10-28 5:37 ` Gole, Dhruva
@ 2022-11-02 20:14 ` Judith Mendez
2022-11-02 21:32 ` Nishanth Menon
2022-11-03 7:23 ` Nishanth Menon
2 siblings, 1 reply; 5+ messages in thread
From: Judith Mendez @ 2022-11-02 20:14 UTC (permalink / raw)
To: Matt Ranostay, nm, vigneshr, kristo, robh+dt,
krzysztof.kozlowski+dt, miquel.raynal
Cc: linux-arm-kernel, devicetree
Hello,
On 10/24/22 10:16, Matt Ranostay wrote:
> Avoid the following warnings from dt-schema by just renaming the
> clock-names string from adc_tsc_fck to fck so it matches the values in
> ti,am3359-tscadc.yaml
>
> tscadc@40200000: clock-names:0: 'fck' was expected
>
Tested on AM64x GP EVM.
Verified-by: Judith Mendez <jm@ti.com>
> Signed-off-by: Matt Ranostay <mranostay@ti.com>
> ---
> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 4 ++--
> arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +-
> arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 4 ++--
> 4 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> index d6aa23681bbe..b61c15427bf7 100644
> --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> @@ -612,7 +612,7 @@ tscadc0: tscadc@28001000 {
> assigned-clocks = <&k3_clks 0 0>;
> assigned-clock-parents = <&k3_clks 0 3>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
>
> adc {
> #io-channel-cells = <1>;
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> index 8d592bf41d6f..1807ec4730cb 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
> @@ -85,7 +85,7 @@ tscadc0: tscadc@40200000 {
> clocks = <&k3_clks 0 2>;
> assigned-clocks = <&k3_clks 0 2>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&mcu_udmap 0x7100>,
> <&mcu_udmap 0x7101 >;
> dma-names = "fifo0", "fifo1";
> @@ -103,7 +103,7 @@ tscadc1: tscadc@40210000 {
> clocks = <&k3_clks 1 2>;
> assigned-clocks = <&k3_clks 1 2>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&mcu_udmap 0x7102>,
> <&mcu_udmap 0x7103>;
> dma-names = "fifo0", "fifo1";
> diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> index 2025aab4b66f..3d9190a869c5 100644
> --- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> @@ -325,7 +325,7 @@ tscadc0: tscadc@40200000 {
> clocks = <&k3_clks 0 1>;
> assigned-clocks = <&k3_clks 0 3>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&main_udmap 0x7400>,
> <&main_udmap 0x7401>;
> dma-names = "fifo0", "fifo1";
> 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 5d9ec221fa34..7799db6a167c 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi
> @@ -216,7 +216,7 @@ tscadc0: tscadc@40200000 {
> clocks = <&k3_clks 0 1>;
> assigned-clocks = <&k3_clks 0 3>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&main_udmap 0x7400>,
> <&main_udmap 0x7401>;
> dma-names = "fifo0", "fifo1";
> @@ -235,7 +235,7 @@ tscadc1: tscadc@40210000 {
> clocks = <&k3_clks 1 1>;
> assigned-clocks = <&k3_clks 1 3>;
> assigned-clock-rates = <60000000>;
> - clock-names = "adc_tsc_fck";
> + clock-names = "fck";
> dmas = <&main_udmap 0x7402>,
> <&main_udmap 0x7403>;
> dma-names = "fifo0", "fifo1";
_______________________________________________
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] 5+ messages in thread* Re: [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck
2022-11-02 20:14 ` Judith Mendez
@ 2022-11-02 21:32 ` Nishanth Menon
0 siblings, 0 replies; 5+ messages in thread
From: Nishanth Menon @ 2022-11-02 21:32 UTC (permalink / raw)
To: Judith Mendez
Cc: Matt Ranostay, vigneshr, kristo, robh+dt, krzysztof.kozlowski+dt,
miquel.raynal, linux-arm-kernel, devicetree
On 15:14-20221102, Judith Mendez wrote:
> Hello,
>
> On 10/24/22 10:16, Matt Ranostay wrote:
> > Avoid the following warnings from dt-schema by just renaming the
> > clock-names string from adc_tsc_fck to fck so it matches the values in
> > ti,am3359-tscadc.yaml
> >
> > tscadc@40200000: clock-names:0: 'fck' was expected
> >
>
> Tested on AM64x GP EVM.
>
> Verified-by: Judith Mendez <jm@ti.com>
Documentation/process/submitting-patches.rst Could you use the standard
conventions defined there? I think you mean Tested-by? if so, please use
that tag.
Quote from documentation:
"A Tested-by: tag indicates that the patch has been successfully tested (in
some environment) by the person named. This tag informs maintainers that
some testing has been performed, provides a means to locate testers for
future patches, and ensures credit for the testers.
"
--
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] 5+ messages in thread
* Re: [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck
2022-10-24 15:16 [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck Matt Ranostay
2022-10-28 5:37 ` Gole, Dhruva
2022-11-02 20:14 ` Judith Mendez
@ 2022-11-03 7:23 ` Nishanth Menon
2 siblings, 0 replies; 5+ messages in thread
From: Nishanth Menon @ 2022-11-03 7:23 UTC (permalink / raw)
To: krzysztof.kozlowski+dt, kristo, vigneshr, mranostay,
miquel.raynal, robh+dt
Cc: Nishanth Menon, linux-arm-kernel, devicetree
Hi Matt Ranostay,
On Mon, 24 Oct 2022 08:16:48 -0700, Matt Ranostay wrote:
> Avoid the following warnings from dt-schema by just renaming the
> clock-names string from adc_tsc_fck to fck so it matches the values in
> ti,am3359-tscadc.yaml
>
> tscadc@40200000: clock-names:0: 'fck' was expected
>
>
> [...]
I have applied the following to branch ti-k3-dts-next on [1].
Thank you!
[1/1] arm64: dts: ti: rename clock-names adc_tsc_fck to fck
commit: e5bad300e46c5bd0c5cf0a89c74116fbbf608a8c
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
_______________________________________________
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] 5+ messages in thread
end of thread, other threads:[~2022-11-03 7:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-24 15:16 [PATCH] arm64: dts: ti: rename clock-names adc_tsc_fck to fck Matt Ranostay
2022-10-28 5:37 ` Gole, Dhruva
2022-11-02 20:14 ` Judith Mendez
2022-11-02 21:32 ` Nishanth Menon
2022-11-03 7:23 ` Nishanth Menon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox