[PATCH 6/6] dt-bindings: watchdog: realtek,otto-wdt: simplify requiring interrupt-names

Sander Vanheule sander at svanheule.net
Sun Apr 16 12:00:17 PDT 2023


Hi Krzysztof,

On Sat, 2023-04-15 at 11:51 +0200, Krzysztof Kozlowski wrote:
> Required properties should be listed in "required:" block.  Since
> interrupts are already there, the dependency of interrupt-names on the
> interrupts can be simplified.

Maybe I'm not reading this right, but isn't the dependency stated in the binding
"interrupts requires interrupt-names to be present"? resource-names.txt
describes the reverse dependency ("interrupt-names is only meaningful with an
associated interrupts").

> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> ---

In any case, I'm OK with a flattened requirements list:

Acked-by: Sander Vanheule <sander at svanheule.net>


Best,
Sander





More information about the linux-amlogic mailing list