Re: [PATCH] dt-bindings: Add missing array size constraints

[Paul Cercueil <paul@crapouillou.net>]

Hi,

Le mar. 5 janv. 2021 à 16:27, Mathieu Poirier 
<mathieu.poirier@linaro.org> a écrit :
> Adding Suman and Paul - guys please have a look.
> 
> On Mon, Jan 04, 2021 at 04:02:53PM -0700, Rob Herring wrote:
>>  DT properties which can have multiple entries need to specify what 
>> the
>>  entries are and define how many entries there can be. In the case of
>>  only a single entry, just 'maxItems: 1' is sufficient.
>> 
>>  Add the missing entry constraints. These were found with a modified
>>  meta-schema. Unfortunately, there are a few cases where the size
>>  constraints are not defined such as common bindings, so the 
>> meta-schema
>>  can't be part of the normal checks.
>> 
>>  Cc: Jens Axboe <axboe@kernel.dk>
>>  Cc: Stephen Boyd <sboyd@kernel.org>
>>  Cc: Thierry Reding <thierry.reding@gmail.com>
>>  Cc: MyungJoo Ham <myungjoo.ham@samsung.com>
>>  Cc: Chanwoo Choi <cw00.choi@samsung.com>
>>  Cc: Linus Walleij <linus.walleij@linaro.org>
>>  Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>  Cc: Jonathan Cameron <jic23@kernel.org>
>>  Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
>>  Cc: Thomas Gleixner <tglx@linutronix.de>
>>  Cc: Marc Zyngier <maz@kernel.org>
>>  Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
>>  Cc: Chen-Yu Tsai <wens@csie.org>
>>  Cc: Ulf Hansson <ulf.hansson@linaro.org>
>>  Cc: "David S. Miller" <davem@davemloft.net>
>>  Cc: Jakub Kicinski <kuba@kernel.org>
>>  Cc: Sebastian Reichel <sre@kernel.org>
>>  Cc: Ohad Ben-Cohen <ohad@wizery.com>
>>  Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
>>  Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>>  Signed-off-by: Rob Herring <robh@kernel.org>
>>  ---
>>  Cc: linux-arm-kernel@lists.infradead.org
>>  Cc: linux-ide@vger.kernel.org
>>  Cc: linux-clk@vger.kernel.org
>>  Cc: dri-devel@lists.freedesktop.org
>>  Cc: linux-gpio@vger.kernel.org
>>  Cc: linux-iio@vger.kernel.org
>>  Cc: linux-input@vger.kernel.org
>>  Cc: linux-media@vger.kernel.org
>>  Cc: linux-mmc@vger.kernel.org
>>  Cc: netdev@vger.kernel.org
>>  Cc: linux-pm@vger.kernel.org
>>  Cc: linux-remoteproc@vger.kernel.org
>>  Cc: linux-riscv@lists.infradead.org
>>  Cc: linux-serial@vger.kernel.org
>>  Cc: alsa-devel@alsa-project.org
>>  Cc: linux-spi@vger.kernel.org
>>  Cc: linux-usb@vger.kernel.org
> 
>> 
>>  ---
>>   .../socionext,uniphier-system-cache.yaml      |  4 ++--
>>   .../bindings/ata/sata_highbank.yaml           |  1 +
>>   .../bindings/clock/canaan,k210-clk.yaml       |  1 +
>>   .../bindings/display/brcm,bcm2711-hdmi.yaml   |  1 +
>>   .../bindings/display/brcm,bcm2835-hdmi.yaml   |  1 +
>>   .../display/panel/jdi,lt070me05000.yaml       |  1 +
>>   .../display/panel/mantix,mlaf057we51-x.yaml   |  3 ++-
>>   .../display/panel/novatek,nt36672a.yaml       |  1 +
>>   .../devicetree/bindings/dsp/fsl,dsp.yaml      |  2 +-
>>   .../devicetree/bindings/eeprom/at25.yaml      |  3 +--
>>   .../bindings/extcon/extcon-ptn5150.yaml       |  2 ++
>>   .../bindings/gpio/gpio-pca95xx.yaml           |  1 +
>>   .../bindings/iio/adc/adi,ad7768-1.yaml        |  2 ++
>>   .../bindings/iio/adc/aspeed,ast2400-adc.yaml  |  1 +
>>   .../bindings/iio/adc/lltc,ltc2496.yaml        |  2 +-
>>   .../bindings/iio/adc/qcom,spmi-vadc.yaml      |  1 +
>>   .../bindings/iio/adc/st,stm32-adc.yaml        |  2 ++
>>   .../iio/magnetometer/asahi-kasei,ak8975.yaml  |  1 +
>>   .../iio/potentiometer/adi,ad5272.yaml         |  1 +
>>   .../input/touchscreen/elan,elants_i2c.yaml    |  1 +
>>   .../interrupt-controller/fsl,intmux.yaml      |  2 +-
>>   .../interrupt-controller/st,stm32-exti.yaml   |  2 ++
>>   .../allwinner,sun4i-a10-video-engine.yaml     |  1 +
>>   .../devicetree/bindings/media/i2c/imx219.yaml |  1 +
>>   .../memory-controllers/exynos-srom.yaml       |  2 ++
>>   .../bindings/misc/fsl,dpaa2-console.yaml      |  1 +
>>   .../bindings/mmc/mmc-controller.yaml          |  2 ++
>>   .../bindings/net/ti,k3-am654-cpsw-nuss.yaml   |  1 +
>>   .../bindings/net/ti,k3-am654-cpts.yaml        |  1 +
>>   .../phy/allwinner,sun4i-a10-usb-phy.yaml      |  2 ++
>>   .../phy/allwinner,sun50i-a64-usb-phy.yaml     |  2 ++
>>   .../phy/allwinner,sun50i-h6-usb-phy.yaml      |  2 ++
>>   .../phy/allwinner,sun5i-a13-usb-phy.yaml      |  2 ++
>>   .../phy/allwinner,sun6i-a31-usb-phy.yaml      |  2 ++
>>   .../phy/allwinner,sun8i-a23-usb-phy.yaml      |  2 ++
>>   .../phy/allwinner,sun8i-a83t-usb-phy.yaml     |  2 ++
>>   .../phy/allwinner,sun8i-h3-usb-phy.yaml       |  2 ++
>>   .../phy/allwinner,sun8i-r40-usb-phy.yaml      |  2 ++
>>   .../phy/allwinner,sun8i-v3s-usb-phy.yaml      |  2 ++
>>   .../phy/allwinner,sun9i-a80-usb-phy.yaml      | 19 
>> ++++++++-----------
>>   .../phy/socionext,uniphier-ahci-phy.yaml      |  2 +-
>>   .../phy/socionext,uniphier-pcie-phy.yaml      |  2 +-
>>   .../phy/socionext,uniphier-usb3hs-phy.yaml    |  2 +-
>>   .../phy/socionext,uniphier-usb3ss-phy.yaml    |  2 +-
>>   .../bindings/phy/ti,phy-gmii-sel.yaml         |  2 +-
>>   .../pinctrl/aspeed,ast2400-pinctrl.yaml       |  3 +--
>>   .../pinctrl/aspeed,ast2500-pinctrl.yaml       |  4 ++--
>>   .../bindings/power/supply/bq25980.yaml        |  1 +
>>   .../bindings/remoteproc/ingenic,vpu.yaml      |  2 +-
>>   .../remoteproc/ti,omap-remoteproc.yaml        |  3 +++
>>   .../bindings/riscv/sifive-l2-cache.yaml       |  1 +
>>   .../bindings/serial/renesas,hscif.yaml        |  2 ++
>>   .../bindings/serial/renesas,scif.yaml         |  2 ++
>>   .../bindings/serial/renesas,scifa.yaml        |  2 ++
>>   .../bindings/serial/renesas,scifb.yaml        |  2 ++
>>   .../sound/allwinner,sun4i-a10-codec.yaml      |  1 +
>>   .../bindings/sound/google,sc7180-trogdor.yaml |  1 +
>>   .../bindings/sound/samsung,aries-wm8994.yaml  |  3 +++
>>   .../bindings/sound/samsung,midas-audio.yaml   |  2 ++
>>   .../devicetree/bindings/sound/tas2562.yaml    |  2 ++
>>   .../devicetree/bindings/sound/tas2770.yaml    |  2 ++
>>   .../bindings/sound/tlv320adcx140.yaml         |  1 +
>>   .../devicetree/bindings/spi/renesas,rspi.yaml |  2 ++
>>   .../devicetree/bindings/sram/sram.yaml        |  2 ++
>>   .../timer/allwinner,sun4i-a10-timer.yaml      |  2 ++
>>   .../bindings/timer/intel,ixp4xx-timer.yaml    |  2 +-
>>   .../usb/allwinner,sun4i-a10-musb.yaml         |  2 +-
>>   .../bindings/usb/brcm,usb-pinmap.yaml         |  3 +++
>>   .../devicetree/bindings/usb/generic-ehci.yaml |  1 +
>>   .../devicetree/bindings/usb/generic-ohci.yaml |  1 +
>>   .../devicetree/bindings/usb/ingenic,musb.yaml |  2 +-
>>   .../bindings/usb/renesas,usbhs.yaml           |  1 +
>>   .../devicetree/bindings/usb/ti,j721e-usb.yaml |  3 ++-
>>   .../bindings/usb/ti,keystone-dwc3.yaml        |  2 ++
>>   74 files changed, 118 insertions(+), 33 deletions(-)

For bindings/remoteproc/ingenic,vpu.yaml and 
devicetree/bindings/usb/ingenic,musb.yaml:

Acked-by: Paul Cercueil <paul@crapouillou.net>

Cheers,
-Paul