Message ID | 20250416-wmt-updates-v1-6-f9af689cdfc2@gmail.com |
---|---|
State | New |
Headers | show |
Series | ARM: vt8500: DT bindings and dts updates | expand |
On Thu, Apr 17, 2025 at 12:16 AM Rob Herring <robh@kernel.org> wrote: > > On Wed, Apr 16, 2025 at 12:21:31PM +0400, Alexey Charkov wrote: > > Rewrite the textual description for the VIA/WonderMedia timer > > as YAML schema. > > > > The IP can generate up to four interrupts from four respective match > > registers, so reflect that in the schema. > > > > Signed-off-by: Alexey Charkov <alchark@gmail.com> > > --- > > .../devicetree/bindings/timer/via,vt8500-timer.txt | 15 --------- > > .../bindings/timer/via,vt8500-timer.yaml | 36 ++++++++++++++++++++++ > > MAINTAINERS | 1 + > > 3 files changed, 37 insertions(+), 15 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/timer/via,vt8500-timer.txt b/Documentation/devicetree/bindings/timer/via,vt8500-timer.txt > > deleted file mode 100644 > > index 901c73f0d8ef05fb54d517b807d04f80eef2e736..0000000000000000000000000000000000000000 > > --- a/Documentation/devicetree/bindings/timer/via,vt8500-timer.txt > > +++ /dev/null > > @@ -1,15 +0,0 @@ > > -VIA/Wondermedia VT8500 Timer > > ------------------------------------------------------ > > - > > -Required properties: > > -- compatible : "via,vt8500-timer" > > -- reg : Should contain 1 register ranges(address and length) > > -- interrupts : interrupt for the timer > > - > > -Example: > > - > > - timer@d8130100 { > > - compatible = "via,vt8500-timer"; > > - reg = <0xd8130100 0x28>; > > - interrupts = <36>; > > - }; > > diff --git a/Documentation/devicetree/bindings/timer/via,vt8500-timer.yaml b/Documentation/devicetree/bindings/timer/via,vt8500-timer.yaml > > new file mode 100644 > > index 0000000000000000000000000000000000000000..7d9dc93b708823de9594d20ef6b7e0367c5a36f1 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/timer/via,vt8500-timer.yaml > > @@ -0,0 +1,36 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/timer/via,vt8500-timer.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: VIA/Wondermedia VT8500 Timer > > + > > +maintainers: > > + - Alexey Charkov <alchark@gmail.com> > > + > > +properties: > > + compatible: > > + const: via,vt8500-timer > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + minItems: 1 > > + maxItems: 4 > > Need to define what each entry is. Will do, thanks. These are OS Timer Channel 0~3 Match, respectively. The hardware has a single 32-bit counter running at 3 MHz and four match registers, each of which is associated with a dedicated match interrupt, and the first of which can also serve as the system watchdog (if watchdog function is enabled, it will reset the system upon match instead of triggering its respective interrupt). Best regards, Alexey
diff --git a/Documentation/devicetree/bindings/timer/via,vt8500-timer.txt b/Documentation/devicetree/bindings/timer/via,vt8500-timer.txt deleted file mode 100644 index 901c73f0d8ef05fb54d517b807d04f80eef2e736..0000000000000000000000000000000000000000 --- a/Documentation/devicetree/bindings/timer/via,vt8500-timer.txt +++ /dev/null @@ -1,15 +0,0 @@ -VIA/Wondermedia VT8500 Timer ------------------------------------------------------ - -Required properties: -- compatible : "via,vt8500-timer" -- reg : Should contain 1 register ranges(address and length) -- interrupts : interrupt for the timer - -Example: - - timer@d8130100 { - compatible = "via,vt8500-timer"; - reg = <0xd8130100 0x28>; - interrupts = <36>; - }; diff --git a/Documentation/devicetree/bindings/timer/via,vt8500-timer.yaml b/Documentation/devicetree/bindings/timer/via,vt8500-timer.yaml new file mode 100644 index 0000000000000000000000000000000000000000..7d9dc93b708823de9594d20ef6b7e0367c5a36f1 --- /dev/null +++ b/Documentation/devicetree/bindings/timer/via,vt8500-timer.yaml @@ -0,0 +1,36 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/timer/via,vt8500-timer.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: VIA/Wondermedia VT8500 Timer + +maintainers: + - Alexey Charkov <alchark@gmail.com> + +properties: + compatible: + const: via,vt8500-timer + + reg: + maxItems: 1 + + interrupts: + minItems: 1 + maxItems: 4 + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + timer@d8130100 { + compatible = "via,vt8500-timer"; + reg = <0xd8130100 0x28>; + interrupts = <36>; + }; diff --git a/MAINTAINERS b/MAINTAINERS index d131004add981446b08b1b3d572a3daa8377fd61..46339ed45ad18ab2b9470c749f604d9aaf72015b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3432,6 +3432,7 @@ F: Documentation/devicetree/bindings/interrupt-controller/via,vt8500-intc.yaml F: Documentation/devicetree/bindings/mmc/wm,wm8505-sdhc.yaml F: Documentation/devicetree/bindings/net/via,vt8500-rhine.yaml F: Documentation/devicetree/bindings/pwm/via,vt8500-pwm.yaml +F: Documentation/devicetree/bindings/timer/via,vt8500-timer.yaml F: arch/arm/boot/dts/vt8500/ F: arch/arm/mach-vt8500/ F: drivers/clocksource/timer-vt8500.c
Rewrite the textual description for the VIA/WonderMedia timer as YAML schema. The IP can generate up to four interrupts from four respective match registers, so reflect that in the schema. Signed-off-by: Alexey Charkov <alchark@gmail.com> --- .../devicetree/bindings/timer/via,vt8500-timer.txt | 15 --------- .../bindings/timer/via,vt8500-timer.yaml | 36 ++++++++++++++++++++++ MAINTAINERS | 1 + 3 files changed, 37 insertions(+), 15 deletions(-)