| Message ID | 20221024151648.394623-1-mranostay@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: ti: rename clock-names adc_tsc_fck to fck | expand |
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";
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";
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. "
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
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";
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(-)