[PATCH v2] dt-bindings: watchdog: migrate mt7621 text bindings to YAML

[PATCH v2] dt-bindings: watchdog: migrate mt7621 text bindings to YAML

[Sergio Paracuellos]

Soc Mt7621 Watchdog bindings used text format, so migrate them to YAML.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
Changes in v2:
- Rebase onto last kernel version.
- Add Krzysztof Reviewed-by tag.

 .../watchdog/mediatek,mt7621-wdt.yaml         | 33 +++++++++++++++++++
 .../bindings/watchdog/mt7621-wdt.txt          | 12 -------
 2 files changed, 33 insertions(+), 12 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
 delete mode 100644 Documentation/devicetree/bindings/watchdog/mt7621-wdt.txt

diff --git a/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml b/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
new file mode 100644
index 000000000000..b2b17fdf4e39
--- /dev/null
+++ b/Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
@@ -0,0 +1,33 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/watchdog/mediatek,mt7621-wdt.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Ralink Watchdog Timers
+
+maintainers:
+  - Sergio Paracuellos <sergio.paracuellos@gmail.com>
+
+allOf:
+  - $ref: watchdog.yaml#
+
+properties:
+  compatible:
+    const: mediatek,mt7621-wdt
+
+  reg:
+    maxItems: 1
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    watchdog@100 {
+      compatible = "mediatek,mt7621-wdt";
+      reg = <0x100 0x100>;
+    };
diff --git a/Documentation/devicetree/bindings/watchdog/mt7621-wdt.txt b/Documentation/devicetree/bindings/watchdog/mt7621-wdt.txt
deleted file mode 100644
index c15ef0ef609f..000000000000
--- a/Documentation/devicetree/bindings/watchdog/mt7621-wdt.txt
+++ /dev/null
@@ -1,12 +0,0 @@
-Ralink Watchdog Timers
-
-Required properties:
-- compatible: must be "mediatek,mt7621-wdt"
-- reg: physical base address of the controller and length of the register range
-
-Example:
-
-	watchdog@100 {
-		compatible = "mediatek,mt7621-wdt";
-		reg = <0x100 0x10>;
-	};
-- 
2.25.1

Re: [PATCH v2] dt-bindings: watchdog: migrate mt7621 text bindings to YAML

[Rob Herring]

On Tue, Sep 20, 2022 at 07:16:17AM +0200, Sergio Paracuellos wrote:
> Soc Mt7621 Watchdog bindings used text format, so migrate them to YAML.
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> ---
> Changes in v2:
> - Rebase onto last kernel version.
> - Add Krzysztof Reviewed-by tag.
> 
>  .../watchdog/mediatek,mt7621-wdt.yaml         | 33 +++++++++++++++++++
>  .../bindings/watchdog/mt7621-wdt.txt          | 12 -------
>  2 files changed, 33 insertions(+), 12 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
>  delete mode 100644 Documentation/devicetree/bindings/watchdog/mt7621-wdt.txt

You will want resend to the watchdog maintainers if you want this 
applied. IOW, use get_maintainers.pl.

Rob

Re: [PATCH v2] dt-bindings: watchdog: migrate mt7621 text bindings to YAML

[Sergio Paracuellos]

Hi Rob,

On Mon, Sep 26, 2022 at 6:16 PM Rob Herring <robh@kernel.org> wrote:
>
> On Tue, Sep 20, 2022 at 07:16:17AM +0200, Sergio Paracuellos wrote:
> > Soc Mt7621 Watchdog bindings used text format, so migrate them to YAML.
> >
> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > ---
> > Changes in v2:
> > - Rebase onto last kernel version.
> > - Add Krzysztof Reviewed-by tag.
> >
> >  .../watchdog/mediatek,mt7621-wdt.yaml         | 33 +++++++++++++++++++
> >  .../bindings/watchdog/mt7621-wdt.txt          | 12 -------
> >  2 files changed, 33 insertions(+), 12 deletions(-)
> >  create mode 100644 Documentation/devicetree/bindings/watchdog/mediatek,mt7621-wdt.yaml
> >  delete mode 100644 Documentation/devicetree/bindings/watchdog/mt7621-wdt.txt
>
> You will want resend to the watchdog maintainers if you want this
> applied. IOW, use get_maintainers.pl.

Sure, I have just resent it also cc'ing watchdog maintainers.

Thanks,
    Sergio Paracuellos
>
> Rob