Re: [PATCH v3] dt-bindings: watchdog: gpio: Convert bindings to YAML

[Marek Vasut <marex@denx.de>]

On 12/15/22 23:45, Marek Vasut wrote:
> On 12/15/22 21:37, Rob Herring wrote:
>>
>> On Mon, 07 Nov 2022 17:03:38 +0100, Marek Vasut wrote:
>>> Convert the gpio-wdt bindings from text to YAML ones, to permit DT 
>>> validation.
>>>
>>> Reviewed-by: Rob Herring <robh@kernel.org>
>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>> ---
>>> Cc: Guenter Roeck <linux@roeck-us.net>
>>> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
>>> Cc: Rob Herring <robh+dt@kernel.org>
>>> Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
>>> Cc: linux-watchdog@vger.kernel.org
>>> To: devicetree@vger.kernel.org
>>> ---
>>> V2: - Add missing required: properties
>>>      - Drop quotes around ref: referenced schema file name
>>> V3: - Add Guenter as maintainer of the bindings
>>>      - Add RB from Rob
>>> ---
>>>   .../devicetree/bindings/watchdog/gpio-wdt.txt | 28 -----------
>>>   .../bindings/watchdog/linux,wdt-gpio.yaml     | 47 +++++++++++++++++++
>>>   2 files changed, 47 insertions(+), 28 deletions(-)
>>>   delete mode 100644 
>>> Documentation/devicetree/bindings/watchdog/gpio-wdt.txt
>>>   create mode 100644 
>>> Documentation/devicetree/bindings/watchdog/linux,wdt-gpio.yaml
>>>
>>
>> Looks like this was missed. Applied, thanks! You missed
>> 'always-running', so I added it from the other conversion that was
>> done[1].
>>
>> Rob
>>
>> [1] 
>> https://lore.kernel.org/all/20221125112904.48652-1-robert.marko@sartura.hr/
> 
> There is no 'always-enabled' DT property used anywhere in the Linux 
> kernel, at least 'git grep' in latest next-20221215 indicates so. Shall 
> we really add it in the bindings ?

Please ignore what I wrote, always-running was indeed missed.