lkml.org 
[lkml]   [2020]   [May]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2 1/7] dt-bindings: watchdog: Convert DW WDT binding to DT schema
    Date
    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 <Sergey.Semin@baikalelectronics.ru>
    Cc: Alexey Malahov <Alexey.Malahov@baikalelectronics.ru>
    Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
    Cc: Paul Burton <paulburton@kernel.org>
    Cc: Ralf Baechle <ralf@linux-mips.org>
    Cc: Arnd Bergmann <arnd@arndb.de>
    Cc: Philipp Zabel <p.zabel@pengutronix.de>
    Cc: linux-mips@vger.kernel.org

    ---

    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 <jamie@jamieiles.com>
    +
    +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>;
    + };
    +...
    --
    2.25.1
    \
     
     \ /
      Last update: 2020-05-10 12:59    [W:4.137 / U:0.224 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site