* [PATCH RESEND] arm64: dts: mediatek: mt8516: remove 2 invalid i2c clocks
@ 2021-11-10 19:35 Fabien Parent
2021-11-17 17:28 ` Matthias Brugger
0 siblings, 1 reply; 2+ messages in thread
From: Fabien Parent @ 2021-11-10 19:35 UTC (permalink / raw)
To: Rob Herring, Matthias Brugger
Cc: Fabien Parent, devicetree, linux-arm-kernel, linux-mediatek,
linux-kernel
The two clocks "main-source" and "main-sel" are not present in the
driver and not defined in the binding documentation. Remove them
as they are not used and not described in the documentation.
Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
arch/arm64/boot/dts/mediatek/mt8516.dtsi | 27 ++++++------------------
1 file changed, 6 insertions(+), 21 deletions(-)
diff --git a/arch/arm64/boot/dts/mediatek/mt8516.dtsi b/arch/arm64/boot/dts/mediatek/mt8516.dtsi
index bbe5a1419eff..d1b67c82d761 100644
--- a/arch/arm64/boot/dts/mediatek/mt8516.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8516.dtsi
@@ -345,14 +345,9 @@ i2c0: i2c@11009000 {
reg = <0 0x11009000 0 0x90>,
<0 0x11000180 0 0x80>;
interrupts = <GIC_SPI 80 IRQ_TYPE_LEVEL_LOW>;
- clocks = <&topckgen CLK_TOP_AHB_INFRA_D2>,
- <&infracfg CLK_IFR_I2C0_SEL>,
- <&topckgen CLK_TOP_I2C0>,
+ clocks = <&topckgen CLK_TOP_I2C0>,
<&topckgen CLK_TOP_APDMA>;
- clock-names = "main-source",
- "main-sel",
- "main",
- "dma";
+ clock-names = "main", "dma";
#address-cells = <1>;
#size-cells = <0>;
status = "disabled";
@@ -364,14 +359,9 @@ i2c1: i2c@1100a000 {
reg = <0 0x1100a000 0 0x90>,
<0 0x11000200 0 0x80>;
interrupts = <GIC_SPI 81 IRQ_TYPE_LEVEL_LOW>;
- clocks = <&topckgen CLK_TOP_AHB_INFRA_D2>,
- <&infracfg CLK_IFR_I2C1_SEL>,
- <&topckgen CLK_TOP_I2C1>,
+ clocks = <&topckgen CLK_TOP_I2C1>,
<&topckgen CLK_TOP_APDMA>;
- clock-names = "main-source",
- "main-sel",
- "main",
- "dma";
+ clock-names = "main", "dma";
#address-cells = <1>;
#size-cells = <0>;
status = "disabled";
@@ -383,14 +373,9 @@ i2c2: i2c@1100b000 {
reg = <0 0x1100b000 0 0x90>,
<0 0x11000280 0 0x80>;
interrupts = <GIC_SPI 82 IRQ_TYPE_LEVEL_LOW>;
- clocks = <&topckgen CLK_TOP_AHB_INFRA_D2>,
- <&infracfg CLK_IFR_I2C2_SEL>,
- <&topckgen CLK_TOP_I2C2>,
+ clocks = <&topckgen CLK_TOP_I2C2>,
<&topckgen CLK_TOP_APDMA>;
- clock-names = "main-source",
- "main-sel",
- "main",
- "dma";
+ clock-names = "main", "dma";
#address-cells = <1>;
#size-cells = <0>;
status = "disabled";
--
2.33.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH RESEND] arm64: dts: mediatek: mt8516: remove 2 invalid i2c clocks
2021-11-10 19:35 [PATCH RESEND] arm64: dts: mediatek: mt8516: remove 2 invalid i2c clocks Fabien Parent
@ 2021-11-17 17:28 ` Matthias Brugger
0 siblings, 0 replies; 2+ messages in thread
From: Matthias Brugger @ 2021-11-17 17:28 UTC (permalink / raw)
To: Fabien Parent, Rob Herring
Cc: devicetree, linux-arm-kernel, linux-mediatek, linux-kernel
On 10/11/2021 20:35, Fabien Parent wrote:
> The two clocks "main-source" and "main-sel" are not present in the
> driver and not defined in the binding documentation. Remove them
> as they are not used and not described in the documentation.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
Applied to v5.16-next/dts64
thanks
> ---
> arch/arm64/boot/dts/mediatek/mt8516.dtsi | 27 ++++++------------------
> 1 file changed, 6 insertions(+), 21 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8516.dtsi b/arch/arm64/boot/dts/mediatek/mt8516.dtsi
> index bbe5a1419eff..d1b67c82d761 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8516.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8516.dtsi
> @@ -345,14 +345,9 @@ i2c0: i2c@11009000 {
> reg = <0 0x11009000 0 0x90>,
> <0 0x11000180 0 0x80>;
> interrupts = <GIC_SPI 80 IRQ_TYPE_LEVEL_LOW>;
> - clocks = <&topckgen CLK_TOP_AHB_INFRA_D2>,
> - <&infracfg CLK_IFR_I2C0_SEL>,
> - <&topckgen CLK_TOP_I2C0>,
> + clocks = <&topckgen CLK_TOP_I2C0>,
> <&topckgen CLK_TOP_APDMA>;
> - clock-names = "main-source",
> - "main-sel",
> - "main",
> - "dma";
> + clock-names = "main", "dma";
> #address-cells = <1>;
> #size-cells = <0>;
> status = "disabled";
> @@ -364,14 +359,9 @@ i2c1: i2c@1100a000 {
> reg = <0 0x1100a000 0 0x90>,
> <0 0x11000200 0 0x80>;
> interrupts = <GIC_SPI 81 IRQ_TYPE_LEVEL_LOW>;
> - clocks = <&topckgen CLK_TOP_AHB_INFRA_D2>,
> - <&infracfg CLK_IFR_I2C1_SEL>,
> - <&topckgen CLK_TOP_I2C1>,
> + clocks = <&topckgen CLK_TOP_I2C1>,
> <&topckgen CLK_TOP_APDMA>;
> - clock-names = "main-source",
> - "main-sel",
> - "main",
> - "dma";
> + clock-names = "main", "dma";
> #address-cells = <1>;
> #size-cells = <0>;
> status = "disabled";
> @@ -383,14 +373,9 @@ i2c2: i2c@1100b000 {
> reg = <0 0x1100b000 0 0x90>,
> <0 0x11000280 0 0x80>;
> interrupts = <GIC_SPI 82 IRQ_TYPE_LEVEL_LOW>;
> - clocks = <&topckgen CLK_TOP_AHB_INFRA_D2>,
> - <&infracfg CLK_IFR_I2C2_SEL>,
> - <&topckgen CLK_TOP_I2C2>,
> + clocks = <&topckgen CLK_TOP_I2C2>,
> <&topckgen CLK_TOP_APDMA>;
> - clock-names = "main-source",
> - "main-sel",
> - "main",
> - "dma";
> + clock-names = "main", "dma";
> #address-cells = <1>;
> #size-cells = <0>;
> status = "disabled";
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-11-17 17:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-10 19:35 [PATCH RESEND] arm64: dts: mediatek: mt8516: remove 2 invalid i2c clocks Fabien Parent
2021-11-17 17:28 ` Matthias Brugger
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).