Re: [PATCH] dt-bindings: leds: Add missing (unevaluated|additional)Properties on child nodes

From: Rob Herring
Date: Wed Aug 24 2022 - 13:37:20 EST


On Tue, Aug 23, 2022 at 1:28 PM Pavel Machek <pavel@xxxxxx> wrote:
>
> Hi!
>
> > In order to ensure only documented properties are present, node schemas
> > must have unevaluatedProperties or additionalProperties set to false
> > (typically).
> >
> > Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
>
> Patch does way more than that:

It only 'ensure only documented properties are present', but yeah, it
is sometimes more than just adding unevaluatedProperties.

> Can we get some explanation why that's correct?

...adds missing $ref if needed, and fixes resulting errors about
unevaluatedProperties.

BTW, these are just the simple ones. The led bindings with multi-led
nodes are more of a mess. I haven't decided exactly how to fix those.

Rob