On 16/09/2023 17:48, Nik Bune wrote: > Convert txt file to yaml. > Add reg to the list of required properties. > Add mainteiners from ./scripts/get_maintainer.pl output. > > Signed-off-by: Nik Bune <n3q5u8@xxxxxxxxx> > --- > .../watchdog/atmel,at91rm9200-wdt.yaml | 31 +++++++++++++++++++ > .../watchdog/atmel-at91rm9200-wdt.txt | 9 ------ > 2 files changed, 31 insertions(+), 9 deletions(-) > create mode 100644 Documentation/devicetree/bindings/watchdog/atmel,at91rm9200-wdt.yaml > delete mode 100644 Documentation/devicetree/bindings/watchdog/atmel-at91rm9200-wdt.txt > > diff --git a/Documentation/devicetree/bindings/watchdog/atmel,at91rm9200-wdt.yaml b/Documentation/devicetree/bindings/watchdog/atmel,at91rm9200-wdt.yaml > new file mode 100644 > index 000000000000..e9706b9c1e6b > --- /dev/null > +++ b/Documentation/devicetree/bindings/watchdog/atmel,at91rm9200-wdt.yaml > @@ -0,0 +1,31 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +# Copyright (C) 2022 Microchip Technology, Inc. and its subsidiaries How this copyright appeared here? Why are you adding some 2022 copyrights of someone else? > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/watchdog/atmel,at91rm9200-wdt.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Atmel AT91RM9200 System Timer Watchdog > + > +maintainers: > + - Wim Van Sebroeck <wim@xxxxxxxxxxxxxxxxxx> > + - Guenter Roeck <linux@xxxxxxxxxxxx> > + Missing ref to watchdog.yaml. Please open existing files and take a look how it is done. > +properties: > + compatible: > + const: atmel,at91sam9260-wdt Missing blank line > + reg: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + > +unevaluatedProperties: false This would not make sense without $ref... Anyway, you need to also test the DTS. Best regards, Krzysztof
