Messages in this thread | | | Subject | Re: [PATCH 1/3] dt-bindings: watchdog: sama5d4-wdt: convert to yaml | From | Nicolas Ferre <> | Date | Sun, 6 Jun 2021 18:52:58 +0200 |
| |
On 27/05/2021 at 12:01, Eugen Hristev wrote: > Convert the old txt binding to yaml format. > > Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
Series queued on at91-dt for 5.14. Will be sent to arm-soc soon. Thanks, best regards, Nicolas
> --- > .../bindings/watchdog/atmel,sama5d4-wdt.yaml | 73 +++++++++++++++++++ > .../bindings/watchdog/atmel-sama5d4-wdt.txt | 34 --------- > 2 files changed, 73 insertions(+), 34 deletions(-) > create mode 100644 Documentation/devicetree/bindings/watchdog/atmel,sama5d4-wdt.yaml > delete mode 100644 Documentation/devicetree/bindings/watchdog/atmel-sama5d4-wdt.txt > > diff --git a/Documentation/devicetree/bindings/watchdog/atmel,sama5d4-wdt.yaml b/Documentation/devicetree/bindings/watchdog/atmel,sama5d4-wdt.yaml > new file mode 100644 > index 000000000000..0d0ab81da040 > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/atmel,sama5d4-wdt.yaml > @@ -0,0 +1,73 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/watchdog/atmel,sama5d4-wdt.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Atmel SAMA5D4 Watchdog Timer (WDT) Controller > + > +maintainers: > + - Eugen Hristev <eugen.hristev@microchip.com> > + > +allOf: > + - $ref: "watchdog.yaml#" > + > +properties: > + compatible: > + enum: > + - atmel,sama5d4-wdt > + - microchip,sam9x60-wdt > + > + reg: > + maxItems: 1 > + > + atmel,watchdog-type: > + $ref: /schemas/types.yaml#/definitions/string > + description: should be hardware or software. > + oneOf: > + - description: > + Enable watchdog fault reset. A watchdog fault triggers > + watchdog reset. > + const: hardware > + - description: > + Enable watchdog fault interrupt. A watchdog fault asserts > + watchdog interrupt. > + const: software > + default: hardware > + > + atmel,idle-halt: > + $ref: /schemas/types.yaml#/definitions/flag > + description: | > + present if you want to stop the watchdog when the CPU is in idle state. > + CAUTION: This property should be used with care, it actually makes the > + watchdog not counting when the CPU is in idle state, therefore the > + watchdog reset time depends on mean CPU usage and will not reset at all > + if the CPU stop working while it is in idle state, which is probably > + not what you want. > + > + atmel,dbg-halt: > + $ref: /schemas/types.yaml#/definitions/flag > + description: | > + present if you want to stop the watchdog when the CPU is in debug state. > + > +required: > + - compatible > + - reg > + > +unevaluatedProperties: false > + > +examples: > + - | > + #include <dt-bindings/interrupt-controller/irq.h> > + > + watchdog@fc068640 { > + compatible = "atmel,sama5d4-wdt"; > + reg = <0xfc068640 0x10>; > + interrupts = <4 IRQ_TYPE_LEVEL_HIGH 5>; > + timeout-sec = <10>; > + atmel,watchdog-type = "hardware"; > + atmel,dbg-halt; > + atmel,idle-halt; > + }; > + > +... > diff --git a/Documentation/devicetree/bindings/watchdog/atmel-sama5d4-wdt.txt b/Documentation/devicetree/bindings/watchdog/atmel-sama5d4-wdt.txt > deleted file mode 100644 > index 44727fcc2729..000000000000 > --- a/Documentation/devicetree/bindings/watchdog/atmel-sama5d4-wdt.txt > +++ /dev/null > @@ -1,34 +0,0 @@ > -* Atmel SAMA5D4 Watchdog Timer (WDT) Controller > - > -Required properties: > -- compatible: "atmel,sama5d4-wdt" or "microchip,sam9x60-wdt" > -- reg: base physical address and length of memory mapped region. > - > -Optional properties: > -- timeout-sec: watchdog timeout value (in seconds). > -- interrupts: interrupt number to the CPU. > -- atmel,watchdog-type: should be "hardware" or "software". > - "hardware": enable watchdog fault reset. A watchdog fault triggers > - watchdog reset. > - "software": enable watchdog fault interrupt. A watchdog fault asserts > - watchdog interrupt. > -- atmel,idle-halt: present if you want to stop the watchdog when the CPU is > - in idle state. > - CAUTION: This property should be used with care, it actually makes the > - watchdog not counting when the CPU is in idle state, therefore the > - watchdog reset time depends on mean CPU usage and will not reset at all > - if the CPU stop working while it is in idle state, which is probably > - not what you want. > -- atmel,dbg-halt: present if you want to stop the watchdog when the CPU is > - in debug state. > - > -Example: > - watchdog@fc068640 { > - compatible = "atmel,sama5d4-wdt"; > - reg = <0xfc068640 0x10>; > - interrupts = <4 IRQ_TYPE_LEVEL_HIGH 5>; > - timeout-sec = <10>; > - atmel,watchdog-type = "hardware"; > - atmel,dbg-halt; > - atmel,idle-halt; > - }; >
-- Nicolas Ferre
| |