Re: [PATCH] dt-bindings: Drop redundant minItems/maxItems
From: Jonathan Cameron <jic23@kernel.org>
Date: 2021-06-16 12:27:52
Also in:
alsa-devel, dri-devel, linux-can, linux-clk, linux-crypto, linux-devicetree, linux-gpio, linux-i2c, linux-ide, linux-iio, linux-iommu, linux-media, linux-mmc, linux-pci, linux-phy, linux-pm, linux-pwm, linux-remoteproc, linux-riscv, linux-rtc, linux-serial, linux-spi, linux-usb, linux-watchdog, lkml, netdev
On Tue, 15 Jun 2021 13:15:43 -0600 Rob Herring [off-list ref] wrote:
If a property has an 'items' list, then a 'minItems' or 'maxItems' with the same size as the list is redundant and can be dropped. Note that is DT schema specific behavior and not standard json-schema behavior. The tooling will fixup the final schema adding any unspecified minItems/maxItems. This condition is partially checked with the meta-schema already, but only if both 'minItems' and 'maxItems' are equal to the 'items' length. An improved meta-schema is pending.
...
.../devicetree/bindings/iio/adc/amlogic,meson-saradc.yaml | 1 -
For this one, the fact it overrides maxItems elsewhere makes this a little bit odd. I guess we can get used to it being implicit.
.../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml | 2 --
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>