Thread (37 messages) 37 messages, 17 authors, 2020-10-07

Re: [PATCH 0/4] dt-bindings: additional/unevaluatedProperties clean-ups

From: Srinivas Kandagatla <hidden>
Date: 2020-10-06 08:25:18
Also in: dmaengine, dri-devel, linux-can, linux-clk, linux-devicetree, linux-gpio, linux-hwmon, linux-i2c, linux-ide, linux-iio, linux-input, linux-leds, linux-media, linux-mips, linux-mmc, linux-pci, linux-pm, linux-pwm, linux-riscv, linux-serial, linux-spi, linux-usb, linux-watchdog, lkml


On 05/10/2020 19:38, Rob Herring wrote:
The default behavior for json-schema is any unknown property is allowed.
T

hat is generally not the behavior we want for DT. In order to disallow
extra properties, schemas need to define 'additionalProperties: false'
typically. Ideally, we'd just add that automatically with the tools, but
there are some exceptions so only making things explicit everywhere
really works. Missing 'additionalProperties' or 'unevaluatedProperties'
has been a constant source of review comments, so a meta-schema check is
really needed here.

  Documentation/devicetree/bindings/nvmem/nvmem.yaml     |  2 ++
  .../devicetree/bindings/nvmem/qcom,qfprom.yaml         |  2 ++
for nvmem parts,

Acked-by: Srinivas Kandagatla <redacted>


thanks,
--srini
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help