Subject: [PATCH 0/2] Convert mtk-timer to YAML

This is a conversion of the mediatek,mtk-timer.txt binding to schema.

While checking the schema validity, I've seen that mt7629.dtsi was
declaring two timer interrupts while effectively there's only one:
removing it was also necessary to be compliant with the new schema.

AngeloGioacchino Del Regno (2):
arm: dts: mt7629: Remove extra interrupt from timer node
dt-bindings: timer: mediatek: Convert to json-schema

.../bindings/timer/mediatek,mtk-timer.txt | 47 ----------
.../bindings/timer/mediatek,timer.yaml | 86 +++++++++++++++++++
arch/arm/boot/dts/mt7629.dtsi | 3 +-
3 files changed, 87 insertions(+), 49 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
create mode 100644 Documentation/devicetree/bindings/timer/mediatek,timer.yaml

--
2.37.2


Subject: [PATCH 1/2] arm: dts: mt7629: Remove extra interrupt from timer node

There's only one system timer event interrupt.

Signed-off-by: AngeloGioacchino Del Regno <[email protected]>
---
arch/arm/boot/dts/mt7629.dtsi | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/mt7629.dtsi b/arch/arm/boot/dts/mt7629.dtsi
index 46fc236e1b89..acab0883a3bb 100644
--- a/arch/arm/boot/dts/mt7629.dtsi
+++ b/arch/arm/boot/dts/mt7629.dtsi
@@ -106,8 +106,7 @@ timer: timer@10009000 {
compatible = "mediatek,mt7629-timer",
"mediatek,mt6765-timer";
reg = <0x10009000 0x60>;
- interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>,
- <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&clk20m>;
clock-names = "clk20m";
};
--
2.37.2

Subject: Re: [PATCH 0/2] Convert mtk-timer to YAML

Il 29/09/22 14:28, AngeloGioacchino Del Regno ha scritto:
> This is a conversion of the mediatek,mtk-timer.txt binding to schema.
>
> While checking the schema validity, I've seen that mt7629.dtsi was
> declaring two timer interrupts while effectively there's only one:
> removing it was also necessary to be compliant with the new schema.
>
> AngeloGioacchino Del Regno (2):
> arm: dts: mt7629: Remove extra interrupt from timer node
> dt-bindings: timer: mediatek: Convert to json-schema
>
> .../bindings/timer/mediatek,mtk-timer.txt | 47 ----------
> .../bindings/timer/mediatek,timer.yaml | 86 +++++++++++++++++++
> arch/arm/boot/dts/mt7629.dtsi | 3 +-
> 3 files changed, 87 insertions(+), 49 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/timer/mediatek,mtk-timer.txt
> create mode 100644 Documentation/devicetree/bindings/timer/mediatek,timer.yaml
>

Gentle ping about this one...

Thanks,
Angelo

2022-11-22 17:56:34

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH 1/2] arm: dts: mt7629: Remove extra interrupt from timer node



On 29/09/2022 14:29, AngeloGioacchino Del Regno wrote:
> There's only one system timer event interrupt.
>
> Signed-off-by: AngeloGioacchino Del Regno <[email protected]>

Queued now for the next merge window.

Matthias

> ---
> arch/arm/boot/dts/mt7629.dtsi | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/mt7629.dtsi b/arch/arm/boot/dts/mt7629.dtsi
> index 46fc236e1b89..acab0883a3bb 100644
> --- a/arch/arm/boot/dts/mt7629.dtsi
> +++ b/arch/arm/boot/dts/mt7629.dtsi
> @@ -106,8 +106,7 @@ timer: timer@10009000 {
> compatible = "mediatek,mt7629-timer",
> "mediatek,mt6765-timer";
> reg = <0x10009000 0x60>;
> - interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>,
> - <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>;
> + interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&clk20m>;
> clock-names = "clk20m";
> };