On Tue, May 26, 2020 at 06:41:17PM +0300, Serge Semin wrote: > Modern device tree bindings are supposed to be created as YAML-files > in accordance with dt-schema. This commit replaces the DW Watchdog > legacy bare text bindings with YAML file. As before the binding states > that the corresponding dts node is supposed to have a registers > range, a watchdog timer references clock source, optional reset line and > pre-timeout interrupt. > > Signed-off-by: Serge Semin <[email protected]> > Reviewed-by: Rob Herring <[email protected]> > Cc: Alexey Malahov <[email protected]> > Cc: Thomas Bogendoerfer <[email protected]> > Cc: Arnd Bergmann <[email protected]> > Cc: [email protected]
Reviewed-by: Guenter Roeck <[email protected]> > --- > > Changelog v2: > - Rearrange SoBs. > - Discard BE copyright header. > - Replace "additionalProperties: false" with "unevaluatedProperties: false" > property. > - Discard interrupts property from the required properties list. > - Remove a label definition from the binding example. > - Move the asynchronous APB3 clock support into a dedicated patch. > --- > .../devicetree/bindings/watchdog/dw_wdt.txt | 24 --------- > .../bindings/watchdog/snps,dw-wdt.yaml | 50 +++++++++++++++++++ > 2 files changed, 50 insertions(+), 24 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/watchdog/dw_wdt.txt > create mode 100644 > Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml > > diff --git a/Documentation/devicetree/bindings/watchdog/dw_wdt.txt > b/Documentation/devicetree/bindings/watchdog/dw_wdt.txt > deleted file mode 100644 > index eb0914420c7c..000000000000 > --- a/Documentation/devicetree/bindings/watchdog/dw_wdt.txt > +++ /dev/null > @@ -1,24 +0,0 @@ > -Synopsys Designware Watchdog Timer > - > -Required Properties: > - > -- compatible : Should contain "snps,dw-wdt" > -- reg : Base address and size of the watchdog timer registers. > -- clocks : phandle + clock-specifier for the clock that drives the > - watchdog timer. > - > -Optional Properties: > - > -- interrupts : The interrupt used for the watchdog timeout warning. > -- resets : phandle pointing to the system reset controller with > - line index for the watchdog. > - > -Example: > - > - watchdog0: wd@ffd02000 { > - compatible = "snps,dw-wdt"; > - reg = <0xffd02000 0x1000>; > - interrupts = <0 171 4>; > - clocks = <&per_base_clk>; > - resets = <&rst WDT0_RESET>; > - }; > diff --git a/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml > b/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml > new file mode 100644 > index 000000000000..4f6944756ab4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/snps,dw-wdt.yaml > @@ -0,0 +1,50 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/watchdog/snps,dw-wdt.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Synopsys Designware Watchdog Timer > + > +allOf: > + - $ref: "watchdog.yaml#" > + > +maintainers: > + - Jamie Iles <[email protected]> > + > +properties: > + compatible: > + const: snps,dw-wdt > + > + reg: > + maxItems: 1 > + > + interrupts: > + description: DW Watchdog pre-timeout interrupt > + maxItems: 1 > + > + clocks: > + items: > + - description: Watchdog timer reference clock > + > + resets: > + description: Phandle to the DW Watchdog reset lane > + maxItems: 1 > + > +unevaluatedProperties: false > + > +required: > + - compatible > + - reg > + - clocks > + > +examples: > + - | > + watchdog@ffd02000 { > + compatible = "snps,dw-wdt"; > + reg = <0xffd02000 0x1000>; > + interrupts = <0 171 4>; > + clocks = <&per_base_clk>; > + resets = <&wdt_rst>; > + }; > +...
