[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20250818170904.GA1477625-robh@kernel.org>
Date: Mon, 18 Aug 2025 12:09:04 -0500
From: Rob Herring <robh@...nel.org>
To: Aleksander Jan Bajkowski <olek2@...pl>
Cc: wim@...ux-watchdog.org, linux@...ck-us.net, krzk+dt@...nel.org,
conor+dt@...nel.org, hauke@...ke-m.de,
linux-watchdog@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: watchdog: lantiq,wdt: convert bindings to
dtschema
On Mon, Aug 11, 2025 at 03:10:54PM +0200, Aleksander Jan Bajkowski wrote:
> Convert the Lantiq WDT Watchdog bindings to yaml format.
>
> Signed-off-by: Aleksander Jan Bajkowski <olek2@...pl>
> ---
> .../bindings/watchdog/lantiq,wdt.yaml | 50 +++++++++++++++++++
> .../bindings/watchdog/lantiq-wdt.txt | 24 ---------
> 2 files changed, 50 insertions(+), 24 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/watchdog/lantiq,wdt.yaml
> delete mode 100644 Documentation/devicetree/bindings/watchdog/lantiq-wdt.txt
>
> diff --git a/Documentation/devicetree/bindings/watchdog/lantiq,wdt.yaml b/Documentation/devicetree/bindings/watchdog/lantiq,wdt.yaml
> new file mode 100644
> index 000000000000..f1102fff2d92
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/watchdog/lantiq,wdt.yaml
> @@ -0,0 +1,50 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/watchdog/lantiq,wdt.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Lantiq WTD watchdog
> +
> +maintainers:
> + - Hauke Mehrtens <hauke@...ke-m.de>
> +
> +allOf:
> + - $ref: watchdog.yaml#
> +
> +properties:
> + compatible:
> + oneOf:
> + - enum:
> + - lantiq,falcon-wdt
> + - lantiq,wdt
> + - lantiq,xrx100-wdt
> + - items:
> + - enum:
> + - lantiq,xrx200-wdt
> + - lantiq,xrx300-wdt
> + - const: lantiq,xrx100-wdt
> +
> + reg:
> + maxItems: 1
> +
> + lantiq,rcu:
> + $ref: /schemas/types.yaml#/definitions/phandle
> + description:
> + Phandle to the RCU syscon node (required for
> + "lantiq,falcon-wdt" and "lantiq,xrx100-wdt")
Express the 'required' as a schema, not freeform text.
> +
> +required:
> + - compatible
> + - reg
> +
> +unevaluatedProperties: false
> +
> +examples:
> + - |
> + watchdog@...f0 {
> + compatible = "lantiq,xrx200-wdt", "lantiq,xrx100-wdt";
> + reg = <0x803f0 0x10>;
> +
> + lantiq,rcu = <&rcu0>;
> + };
Powered by blists - more mailing lists