Re: [PATCH] dt-bindings: Fix erroneous 'additionalProperties'

From: Lee Jones
Date: Mon Apr 27 2020 - 02:52:06 EST


On Fri, 24 Apr 2020, Rob Herring wrote:

> There's several cases of json-schema 'additionalProperties' at the wrong
> indentation level which has the effect of making them DT properties. This
> is harmless, but let's fix them so a meta-schema check for this can be
> added.
>
> In all the cases, either the 'additionalProperties' was extra or doesn't
> work because there's a $ref to more properties. In the latter case, we
> can use 'unevaluatedProperties' instead.
>
> Reported-by: Iskren Chernev <iskren.chernev@xxxxxxxxx>
> Cc: Lee Jones <lee.jones@xxxxxxxxxx>
> Cc: Saravanan Sekar <sravanhome@xxxxxxxxx>
> Cc: Liam Girdwood <lgirdwood@xxxxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxx>
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/mfd/st,stpmic1.yaml | 2 --

Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>

> Documentation/devicetree/bindings/regulator/mps,mp5416.yaml | 1 -
> .../devicetree/bindings/regulator/mps,mpq7920.yaml | 3 ++-
> .../bindings/regulator/rohm,bd71828-regulator.yaml | 6 +++++-
> .../bindings/regulator/rohm,bd71837-regulator.yaml | 6 +++++-
> .../bindings/regulator/rohm,bd71847-regulator.yaml | 6 +++++-
> 6 files changed, 17 insertions(+), 7 deletions(-)

--
Lee Jones [æçæ]
Linaro Services Technical Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog