dt-bindings: Drop redundant minItems/maxItems
Commit Message
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.
Cc: Jens Axboe <axboe@kernel.dk>
Cc: Stephen Boyd <sboyd@kernel.org>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Cc: Kamal Dasu <kdasu.kdev@gmail.com>
Cc: Jonathan Cameron <jic23@kernel.org>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Marc Zyngier <maz@kernel.org>
Cc: Joerg Roedel <joro@8bytes.org>
Cc: Jassi Brar <jassisinghbrar@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: Wolfgang Grandegger <wg@grandegger.com>
Cc: Marc Kleine-Budde <mkl@pengutronix.de>
Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Vivien Didelot <vivien.didelot@gmail.com>
Cc: Vladimir Oltean <olteanv@gmail.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>
Cc: Kishon Vijay Abraham I <kishon@ti.com>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Ohad Ben-Cohen <ohad@wizery.com>
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Philipp Zabel <p.zabel@pengutronix.de>
Cc: Paul Walmsley <paul.walmsley@sifive.com>
Cc: Palmer Dabbelt <palmer@dabbelt.com>
Cc: Albert Ou <aou@eecs.berkeley.edu>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Zhang Rui <rui.zhang@intel.com>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
Cc: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Rob Herring <robh@kernel.org>
---
.../devicetree/bindings/ata/nvidia,tegra-ahci.yaml | 1 -
.../devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml | 2 --
.../devicetree/bindings/clock/qcom,gcc-apq8064.yaml | 1 -
Documentation/devicetree/bindings/clock/qcom,gcc-sdx55.yaml | 2 --
.../devicetree/bindings/clock/qcom,gcc-sm8350.yaml | 2 --
.../devicetree/bindings/clock/sprd,sc9863a-clk.yaml | 1 -
.../devicetree/bindings/crypto/allwinner,sun8i-ce.yaml | 2 --
Documentation/devicetree/bindings/crypto/fsl-dcp.yaml | 1 -
.../display/allwinner,sun4i-a10-display-backend.yaml | 6 ------
.../bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml | 1 -
.../bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml | 4 ----
.../bindings/display/allwinner,sun8i-a83t-hdmi-phy.yaml | 2 --
.../bindings/display/allwinner,sun8i-r40-tcon-top.yaml | 2 --
.../devicetree/bindings/display/bridge/cdns,mhdp8546.yaml | 2 --
.../bindings/display/rockchip/rockchip,dw-hdmi.yaml | 2 --
Documentation/devicetree/bindings/display/st,stm32-dsi.yaml | 2 --
.../devicetree/bindings/display/st,stm32-ltdc.yaml | 1 -
.../devicetree/bindings/display/xlnx/xlnx,zynqmp-dpsub.yaml | 4 ----
.../devicetree/bindings/dma/renesas,rcar-dmac.yaml | 1 -
.../devicetree/bindings/edac/amazon,al-mc-edac.yaml | 2 --
Documentation/devicetree/bindings/eeprom/at24.yaml | 1 -
Documentation/devicetree/bindings/example-schema.yaml | 2 --
Documentation/devicetree/bindings/gpu/brcm,bcm-v3d.yaml | 1 -
Documentation/devicetree/bindings/gpu/vivante,gc.yaml | 1 -
Documentation/devicetree/bindings/i2c/brcm,brcmstb-i2c.yaml | 1 -
.../devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml | 2 --
.../devicetree/bindings/i2c/mellanox,i2c-mlxbf.yaml | 1 -
.../devicetree/bindings/iio/adc/amlogic,meson-saradc.yaml | 1 -
.../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml | 2 --
.../bindings/interrupt-controller/fsl,irqsteer.yaml | 1 -
.../bindings/interrupt-controller/loongson,liointc.yaml | 1 -
Documentation/devicetree/bindings/iommu/arm,smmu-v3.yaml | 1 -
.../devicetree/bindings/iommu/renesas,ipmmu-vmsa.yaml | 1 -
.../devicetree/bindings/mailbox/st,stm32-ipcc.yaml | 2 --
.../devicetree/bindings/media/amlogic,gx-vdec.yaml | 1 -
Documentation/devicetree/bindings/media/i2c/adv7604.yaml | 1 -
.../devicetree/bindings/media/marvell,mmp2-ccic.yaml | 1 -
.../devicetree/bindings/media/qcom,sc7180-venus.yaml | 1 -
.../devicetree/bindings/media/qcom,sdm845-venus-v2.yaml | 1 -
.../devicetree/bindings/media/qcom,sm8250-venus.yaml | 1 -
Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 -
.../bindings/memory-controllers/mediatek,smi-common.yaml | 6 ++----
.../bindings/memory-controllers/mediatek,smi-larb.yaml | 1 -
.../devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 2 --
Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml | 1 -
Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 2 --
Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml | 2 --
Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 1 -
Documentation/devicetree/bindings/mmc/sdhci-pxa.yaml | 1 -
.../devicetree/bindings/net/amlogic,meson-dwmac.yaml | 2 --
.../devicetree/bindings/net/brcm,bcm4908-enet.yaml | 2 --
Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 2 --
Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml | 2 --
Documentation/devicetree/bindings/net/snps,dwmac.yaml | 2 --
Documentation/devicetree/bindings/net/stm32-dwmac.yaml | 1 -
Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml | 2 --
Documentation/devicetree/bindings/pci/loongson.yaml | 1 -
.../devicetree/bindings/pci/mediatek-pcie-gen3.yaml | 1 -
.../devicetree/bindings/pci/microchip,pcie-host.yaml | 2 --
Documentation/devicetree/bindings/perf/arm,cmn.yaml | 1 -
.../devicetree/bindings/phy/brcm,bcm63xx-usbh-phy.yaml | 1 -
.../devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml | 3 ---
Documentation/devicetree/bindings/phy/brcm,sata-phy.yaml | 1 -
Documentation/devicetree/bindings/phy/mediatek,tphy.yaml | 2 --
.../devicetree/bindings/phy/phy-cadence-sierra.yaml | 2 --
.../devicetree/bindings/phy/phy-cadence-torrent.yaml | 4 ----
.../devicetree/bindings/phy/qcom,ipq806x-usb-phy-hs.yaml | 1 -
.../devicetree/bindings/phy/qcom,ipq806x-usb-phy-ss.yaml | 1 -
Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 1 -
Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml | 2 --
Documentation/devicetree/bindings/phy/renesas,usb2-phy.yaml | 2 --
Documentation/devicetree/bindings/phy/renesas,usb3-phy.yaml | 1 -
.../devicetree/bindings/pinctrl/actions,s500-pinctrl.yaml | 1 -
.../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml | 1 -
.../devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml | 1 -
.../devicetree/bindings/remoteproc/st,stm32-rproc.yaml | 2 --
.../devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml | 1 -
.../devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml | 1 -
Documentation/devicetree/bindings/reset/fsl,imx-src.yaml | 1 -
.../devicetree/bindings/riscv/sifive-l2-cache.yaml | 1 -
.../devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml | 1 -
Documentation/devicetree/bindings/rtc/imxdi-rtc.yaml | 1 -
Documentation/devicetree/bindings/serial/fsl-lpuart.yaml | 2 --
Documentation/devicetree/bindings/serial/samsung_uart.yaml | 1 -
.../devicetree/bindings/soc/qcom/qcom,geni-se.yaml | 1 -
Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 2 --
.../bindings/sound/nvidia,tegra-audio-graph-card.yaml | 1 -
.../devicetree/bindings/sound/nvidia,tegra210-i2s.yaml | 2 --
Documentation/devicetree/bindings/sound/st,stm32-sai.yaml | 3 ---
.../devicetree/bindings/spi/amlogic,meson-gx-spicc.yaml | 1 -
.../devicetree/bindings/spi/brcm,spi-bcm-qspi.yaml | 2 --
.../bindings/thermal/allwinner,sun8i-a83t-ths.yaml | 2 --
Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 -
.../bindings/timer/allwinner,sun5i-a13-hstimer.yaml | 1 -
Documentation/devicetree/bindings/timer/arm,arch_timer.yaml | 1 -
.../devicetree/bindings/timer/arm,arch_timer_mmio.yaml | 2 --
.../devicetree/bindings/timer/intel,ixp4xx-timer.yaml | 1 -
.../devicetree/bindings/usb/maxim,max3420-udc.yaml | 2 --
.../devicetree/bindings/usb/nvidia,tegra-xudc.yaml | 4 ----
Documentation/devicetree/bindings/usb/renesas,usbhs.yaml | 3 ---
.../devicetree/bindings/watchdog/st,stm32-iwdg.yaml | 1 -
101 files changed, 2 insertions(+), 163 deletions(-)
Comments
On Tue, Jun 15, 2021 at 01:15:43PM -0600, Rob Herring 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.
>
> Cc: Jens Axboe <axboe@kernel.dk>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: David Airlie <airlied@linux.ie>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Cc: Kamal Dasu <kdasu.kdev@gmail.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Marc Zyngier <maz@kernel.org>
> Cc: Joerg Roedel <joro@8bytes.org>
> Cc: Jassi Brar <jassisinghbrar@gmail.com>
> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Wolfgang Grandegger <wg@grandegger.com>
> Cc: Marc Kleine-Budde <mkl@pengutronix.de>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Vivien Didelot <vivien.didelot@gmail.com>
> Cc: Vladimir Oltean <olteanv@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Ohad Ben-Cohen <ohad@wizery.com>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Philipp Zabel <p.zabel@pengutronix.de>
> Cc: Paul Walmsley <paul.walmsley@sifive.com>
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: Albert Ou <aou@eecs.berkeley.edu>
> Cc: Alessandro Zummo <a.zummo@towertech.it>
> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
On 15-06-21, 13:15, Rob Herring 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/dma/renesas,rcar-dmac.yaml | 1 -
> Documentation/devicetree/bindings/phy/brcm,sata-phy.yaml | 1 -
> Documentation/devicetree/bindings/phy/mediatek,tphy.yaml | 2 --
> .../devicetree/bindings/phy/phy-cadence-sierra.yaml | 2 --
> .../devicetree/bindings/phy/phy-cadence-torrent.yaml | 4 ----
> .../devicetree/bindings/phy/qcom,ipq806x-usb-phy-hs.yaml | 1 -
> .../devicetree/bindings/phy/qcom,ipq806x-usb-phy-ss.yaml | 1 -
> Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 1 -
> Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml | 2 --
> Documentation/devicetree/bindings/phy/renesas,usb2-phy.yaml | 2 --
> Documentation/devicetree/bindings/phy/renesas,usb3-phy.yaml | 1 -
Acked-By: Vinod Koul <vkoul@kernel.org>
On 15.06.2021 13:15:43, Rob Herring 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.
[...]
> Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 2 --
Acked-by: Marc Kleine-Budde <mkl@pengutronix.de>
regards,
Marc
On Tue, 2021-06-15 at 13:15 -0600, Rob Herring 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.
[...]
> Documentation/devicetree/bindings/reset/fsl,imx-src.yaml | 1 -
[...]
> diff --git a/Documentation/devicetree/bindings/reset/fsl,imx-src.yaml b/Documentation/devicetree/bindings/reset/fsl,imx-src.yaml
> index 27c5e34a3ac6..b11ac533f914 100644
> --- a/Documentation/devicetree/bindings/reset/fsl,imx-src.yaml
> +++ b/Documentation/devicetree/bindings/reset/fsl,imx-src.yaml
> @@ -59,7 +59,6 @@ properties:
> - description: SRC interrupt
> - description: CPU WDOG interrupts out of SRC
> minItems: 1
> - maxItems: 2
>
> '#reset-cells':
> const: 1
Acked-by: Philipp Zabel <p.zabel@pengutronix.de>
regards
Philipp
On Tue, Jun 15, 2021 at 01:15:43PM -0600, Rob Herring 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.
Acked-by: Mark Brown <broonie@kernel.org>
On Tue, 15 Jun 2021 13:15:43 -0600
Rob Herring <robh@kernel.org> 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>
On Tue, Jun 15, 2021 at 01:15:43PM -0600, Rob Herring 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.
>
> Cc: Jens Axboe <axboe@kernel.dk>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: David Airlie <airlied@linux.ie>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Cc: Kamal Dasu <kdasu.kdev@gmail.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Marc Zyngier <maz@kernel.org>
> Cc: Joerg Roedel <joro@8bytes.org>
> Cc: Jassi Brar <jassisinghbrar@gmail.com>
> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Wolfgang Grandegger <wg@grandegger.com>
> Cc: Marc Kleine-Budde <mkl@pengutronix.de>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Vivien Didelot <vivien.didelot@gmail.com>
> Cc: Vladimir Oltean <olteanv@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Ohad Ben-Cohen <ohad@wizery.com>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Philipp Zabel <p.zabel@pengutronix.de>
> Cc: Paul Walmsley <paul.walmsley@sifive.com>
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: Albert Ou <aou@eecs.berkeley.edu>
> Cc: Alessandro Zummo <a.zummo@towertech.it>
> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Wolfram Sang <wsa@kernel.org> # for I2C
On Tue, 15 Jun 2021 at 21:15, Rob Herring <robh@kernel.org> 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.
>
> Cc: Jens Axboe <axboe@kernel.dk>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: David Airlie <airlied@linux.ie>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Cc: Kamal Dasu <kdasu.kdev@gmail.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Marc Zyngier <maz@kernel.org>
> Cc: Joerg Roedel <joro@8bytes.org>
> Cc: Jassi Brar <jassisinghbrar@gmail.com>
> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Wolfgang Grandegger <wg@grandegger.com>
> Cc: Marc Kleine-Budde <mkl@pengutronix.de>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Vivien Didelot <vivien.didelot@gmail.com>
> Cc: Vladimir Oltean <olteanv@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Ohad Ben-Cohen <ohad@wizery.com>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Philipp Zabel <p.zabel@pengutronix.de>
> Cc: Paul Walmsley <paul.walmsley@sifive.com>
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: Albert Ou <aou@eecs.berkeley.edu>
> Cc: Alessandro Zummo <a.zummo@towertech.it>
> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Ulf Hansson <ulf.hansson@linaro.org> # for MMC
[...]
Kind regards
Uffe
On Tue, Jun 15, 2021 at 2:15 PM Rob Herring <robh@kernel.org> 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.
>
> Cc: Jens Axboe <axboe@kernel.dk>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: David Airlie <airlied@linux.ie>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Cc: Kamal Dasu <kdasu.kdev@gmail.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Marc Zyngier <maz@kernel.org>
> Cc: Joerg Roedel <joro@8bytes.org>
> Cc: Jassi Brar <jassisinghbrar@gmail.com>
> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Wolfgang Grandegger <wg@grandegger.com>
> Cc: Marc Kleine-Budde <mkl@pengutronix.de>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Vivien Didelot <vivien.didelot@gmail.com>
> Cc: Vladimir Oltean <olteanv@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Ohad Ben-Cohen <ohad@wizery.com>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Philipp Zabel <p.zabel@pengutronix.de>
> Cc: Paul Walmsley <paul.walmsley@sifive.com>
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: Albert Ou <aou@eecs.berkeley.edu>
> Cc: Alessandro Zummo <a.zummo@towertech.it>
> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> .../devicetree/bindings/ata/nvidia,tegra-ahci.yaml | 1 -
> .../devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml | 2 --
> .../devicetree/bindings/clock/qcom,gcc-apq8064.yaml | 1 -
> Documentation/devicetree/bindings/clock/qcom,gcc-sdx55.yaml | 2 --
> .../devicetree/bindings/clock/qcom,gcc-sm8350.yaml | 2 --
> .../devicetree/bindings/clock/sprd,sc9863a-clk.yaml | 1 -
> .../devicetree/bindings/crypto/allwinner,sun8i-ce.yaml | 2 --
> Documentation/devicetree/bindings/crypto/fsl-dcp.yaml | 1 -
> .../display/allwinner,sun4i-a10-display-backend.yaml | 6 ------
> .../bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml | 1 -
> .../bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml | 4 ----
> .../bindings/display/allwinner,sun8i-a83t-hdmi-phy.yaml | 2 --
> .../bindings/display/allwinner,sun8i-r40-tcon-top.yaml | 2 --
> .../devicetree/bindings/display/bridge/cdns,mhdp8546.yaml | 2 --
> .../bindings/display/rockchip/rockchip,dw-hdmi.yaml | 2 --
> Documentation/devicetree/bindings/display/st,stm32-dsi.yaml | 2 --
> .../devicetree/bindings/display/st,stm32-ltdc.yaml | 1 -
> .../devicetree/bindings/display/xlnx/xlnx,zynqmp-dpsub.yaml | 4 ----
> .../devicetree/bindings/dma/renesas,rcar-dmac.yaml | 1 -
> .../devicetree/bindings/edac/amazon,al-mc-edac.yaml | 2 --
> Documentation/devicetree/bindings/eeprom/at24.yaml | 1 -
> Documentation/devicetree/bindings/example-schema.yaml | 2 --
> Documentation/devicetree/bindings/gpu/brcm,bcm-v3d.yaml | 1 -
> Documentation/devicetree/bindings/gpu/vivante,gc.yaml | 1 -
> Documentation/devicetree/bindings/i2c/brcm,brcmstb-i2c.yaml | 1 -
> .../devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml | 2 --
> .../devicetree/bindings/i2c/mellanox,i2c-mlxbf.yaml | 1 -
> .../devicetree/bindings/iio/adc/amlogic,meson-saradc.yaml | 1 -
> .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml | 2 --
> .../bindings/interrupt-controller/fsl,irqsteer.yaml | 1 -
> .../bindings/interrupt-controller/loongson,liointc.yaml | 1 -
> Documentation/devicetree/bindings/iommu/arm,smmu-v3.yaml | 1 -
> .../devicetree/bindings/iommu/renesas,ipmmu-vmsa.yaml | 1 -
> .../devicetree/bindings/mailbox/st,stm32-ipcc.yaml | 2 --
> .../devicetree/bindings/media/amlogic,gx-vdec.yaml | 1 -
> Documentation/devicetree/bindings/media/i2c/adv7604.yaml | 1 -
> .../devicetree/bindings/media/marvell,mmp2-ccic.yaml | 1 -
> .../devicetree/bindings/media/qcom,sc7180-venus.yaml | 1 -
> .../devicetree/bindings/media/qcom,sdm845-venus-v2.yaml | 1 -
> .../devicetree/bindings/media/qcom,sm8250-venus.yaml | 1 -
> Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 -
> .../bindings/memory-controllers/mediatek,smi-common.yaml | 6 ++----
> .../bindings/memory-controllers/mediatek,smi-larb.yaml | 1 -
> .../devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 2 --
> Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml | 1 -
> Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 2 --
> Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml | 2 --
> Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 1 -
> Documentation/devicetree/bindings/mmc/sdhci-pxa.yaml | 1 -
> .../devicetree/bindings/net/amlogic,meson-dwmac.yaml | 2 --
> .../devicetree/bindings/net/brcm,bcm4908-enet.yaml | 2 --
> Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 2 --
> Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml | 2 --
> Documentation/devicetree/bindings/net/snps,dwmac.yaml | 2 --
> Documentation/devicetree/bindings/net/stm32-dwmac.yaml | 1 -
> Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml | 2 --
> Documentation/devicetree/bindings/pci/loongson.yaml | 1 -
> .../devicetree/bindings/pci/mediatek-pcie-gen3.yaml | 1 -
> .../devicetree/bindings/pci/microchip,pcie-host.yaml | 2 --
> Documentation/devicetree/bindings/perf/arm,cmn.yaml | 1 -
> .../devicetree/bindings/phy/brcm,bcm63xx-usbh-phy.yaml | 1 -
> .../devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml | 3 ---
> Documentation/devicetree/bindings/phy/brcm,sata-phy.yaml | 1 -
> Documentation/devicetree/bindings/phy/mediatek,tphy.yaml | 2 --
> .../devicetree/bindings/phy/phy-cadence-sierra.yaml | 2 --
> .../devicetree/bindings/phy/phy-cadence-torrent.yaml | 4 ----
> .../devicetree/bindings/phy/qcom,ipq806x-usb-phy-hs.yaml | 1 -
> .../devicetree/bindings/phy/qcom,ipq806x-usb-phy-ss.yaml | 1 -
> Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 1 -
> Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml | 2 --
> Documentation/devicetree/bindings/phy/renesas,usb2-phy.yaml | 2 --
> Documentation/devicetree/bindings/phy/renesas,usb3-phy.yaml | 1 -
> .../devicetree/bindings/pinctrl/actions,s500-pinctrl.yaml | 1 -
> .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml | 1 -
> .../devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml | 1 -
> .../devicetree/bindings/remoteproc/st,stm32-rproc.yaml | 2 --
> .../devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml | 1 -
> .../devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml | 1 -
> Documentation/devicetree/bindings/reset/fsl,imx-src.yaml | 1 -
> .../devicetree/bindings/riscv/sifive-l2-cache.yaml | 1 -
> .../devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml | 1 -
> Documentation/devicetree/bindings/rtc/imxdi-rtc.yaml | 1 -
> Documentation/devicetree/bindings/serial/fsl-lpuart.yaml | 2 --
> Documentation/devicetree/bindings/serial/samsung_uart.yaml | 1 -
> .../devicetree/bindings/soc/qcom/qcom,geni-se.yaml | 1 -
> Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 2 --
> .../bindings/sound/nvidia,tegra-audio-graph-card.yaml | 1 -
> .../devicetree/bindings/sound/nvidia,tegra210-i2s.yaml | 2 --
> Documentation/devicetree/bindings/sound/st,stm32-sai.yaml | 3 ---
> .../devicetree/bindings/spi/amlogic,meson-gx-spicc.yaml | 1 -
> .../devicetree/bindings/spi/brcm,spi-bcm-qspi.yaml | 2 --
> .../bindings/thermal/allwinner,sun8i-a83t-ths.yaml | 2 --
> Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 -
> .../bindings/timer/allwinner,sun5i-a13-hstimer.yaml | 1 -
> Documentation/devicetree/bindings/timer/arm,arch_timer.yaml | 1 -
> .../devicetree/bindings/timer/arm,arch_timer_mmio.yaml | 2 --
> .../devicetree/bindings/timer/intel,ixp4xx-timer.yaml | 1 -
> .../devicetree/bindings/usb/maxim,max3420-udc.yaml | 2 --
> .../devicetree/bindings/usb/nvidia,tegra-xudc.yaml | 4 ----
> Documentation/devicetree/bindings/usb/renesas,usbhs.yaml | 3 ---
> .../devicetree/bindings/watchdog/st,stm32-iwdg.yaml | 1 -
> 101 files changed, 2 insertions(+), 163 deletions(-)
>
mailbox stuff
Acked-by: Jassi Brar <jassisinghbrar@gmail.com>
thanks.
Hi Rob,
On 6/15/21 2:15 PM, Rob Herring 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.
>
> Cc: Jens Axboe <axboe@kernel.dk>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: David Airlie <airlied@linux.ie>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> Cc: Kamal Dasu <kdasu.kdev@gmail.com>
> Cc: Jonathan Cameron <jic23@kernel.org>
> Cc: Lars-Peter Clausen <lars@metafoo.de>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Marc Zyngier <maz@kernel.org>
> Cc: Joerg Roedel <joro@8bytes.org>
> Cc: Jassi Brar <jassisinghbrar@gmail.com>
> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Wolfgang Grandegger <wg@grandegger.com>
> Cc: Marc Kleine-Budde <mkl@pengutronix.de>
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Vivien Didelot <vivien.didelot@gmail.com>
> Cc: Vladimir Oltean <olteanv@gmail.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Linus Walleij <linus.walleij@linaro.org>
> Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Ohad Ben-Cohen <ohad@wizery.com>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Philipp Zabel <p.zabel@pengutronix.de>
> Cc: Paul Walmsley <paul.walmsley@sifive.com>
> Cc: Palmer Dabbelt <palmer@dabbelt.com>
> Cc: Albert Ou <aou@eecs.berkeley.edu>
> Cc: Alessandro Zummo <a.zummo@towertech.it>
> Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> .../devicetree/bindings/ata/nvidia,tegra-ahci.yaml | 1 -
> .../devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml | 2 --
> .../devicetree/bindings/clock/qcom,gcc-apq8064.yaml | 1 -
> Documentation/devicetree/bindings/clock/qcom,gcc-sdx55.yaml | 2 --
> .../devicetree/bindings/clock/qcom,gcc-sm8350.yaml | 2 --
> .../devicetree/bindings/clock/sprd,sc9863a-clk.yaml | 1 -
> .../devicetree/bindings/crypto/allwinner,sun8i-ce.yaml | 2 --
> Documentation/devicetree/bindings/crypto/fsl-dcp.yaml | 1 -
> .../display/allwinner,sun4i-a10-display-backend.yaml | 6 ------
> .../bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml | 1 -
> .../bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml | 4 ----
> .../bindings/display/allwinner,sun8i-a83t-hdmi-phy.yaml | 2 --
> .../bindings/display/allwinner,sun8i-r40-tcon-top.yaml | 2 --
> .../devicetree/bindings/display/bridge/cdns,mhdp8546.yaml | 2 --
> .../bindings/display/rockchip/rockchip,dw-hdmi.yaml | 2 --
> Documentation/devicetree/bindings/display/st,stm32-dsi.yaml | 2 --
> .../devicetree/bindings/display/st,stm32-ltdc.yaml | 1 -
> .../devicetree/bindings/display/xlnx/xlnx,zynqmp-dpsub.yaml | 4 ----
> .../devicetree/bindings/dma/renesas,rcar-dmac.yaml | 1 -
> .../devicetree/bindings/edac/amazon,al-mc-edac.yaml | 2 --
> Documentation/devicetree/bindings/eeprom/at24.yaml | 1 -
> Documentation/devicetree/bindings/example-schema.yaml | 2 --
> Documentation/devicetree/bindings/gpu/brcm,bcm-v3d.yaml | 1 -
> Documentation/devicetree/bindings/gpu/vivante,gc.yaml | 1 -
> Documentation/devicetree/bindings/i2c/brcm,brcmstb-i2c.yaml | 1 -
> .../devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml | 2 --
> .../devicetree/bindings/i2c/mellanox,i2c-mlxbf.yaml | 1 -
> .../devicetree/bindings/iio/adc/amlogic,meson-saradc.yaml | 1 -
> .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml | 2 --
> .../bindings/interrupt-controller/fsl,irqsteer.yaml | 1 -
> .../bindings/interrupt-controller/loongson,liointc.yaml | 1 -
> Documentation/devicetree/bindings/iommu/arm,smmu-v3.yaml | 1 -
> .../devicetree/bindings/iommu/renesas,ipmmu-vmsa.yaml | 1 -
> .../devicetree/bindings/mailbox/st,stm32-ipcc.yaml | 2 --
> .../devicetree/bindings/media/amlogic,gx-vdec.yaml | 1 -
> Documentation/devicetree/bindings/media/i2c/adv7604.yaml | 1 -
> .../devicetree/bindings/media/marvell,mmp2-ccic.yaml | 1 -
> .../devicetree/bindings/media/qcom,sc7180-venus.yaml | 1 -
> .../devicetree/bindings/media/qcom,sdm845-venus-v2.yaml | 1 -
> .../devicetree/bindings/media/qcom,sm8250-venus.yaml | 1 -
> Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 -
> .../bindings/memory-controllers/mediatek,smi-common.yaml | 6 ++----
> .../bindings/memory-controllers/mediatek,smi-larb.yaml | 1 -
> .../devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 2 --
> Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml | 1 -
> Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 2 --
> Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml | 2 --
> Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 1 -
> Documentation/devicetree/bindings/mmc/sdhci-pxa.yaml | 1 -
> .../devicetree/bindings/net/amlogic,meson-dwmac.yaml | 2 --
> .../devicetree/bindings/net/brcm,bcm4908-enet.yaml | 2 --
> Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 2 --
> Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml | 2 --
> Documentation/devicetree/bindings/net/snps,dwmac.yaml | 2 --
> Documentation/devicetree/bindings/net/stm32-dwmac.yaml | 1 -
> Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml | 2 --
> Documentation/devicetree/bindings/pci/loongson.yaml | 1 -
> .../devicetree/bindings/pci/mediatek-pcie-gen3.yaml | 1 -
> .../devicetree/bindings/pci/microchip,pcie-host.yaml | 2 --
> Documentation/devicetree/bindings/perf/arm,cmn.yaml | 1 -
> .../devicetree/bindings/phy/brcm,bcm63xx-usbh-phy.yaml | 1 -
> .../devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml | 3 ---
> Documentation/devicetree/bindings/phy/brcm,sata-phy.yaml | 1 -
> Documentation/devicetree/bindings/phy/mediatek,tphy.yaml | 2 --
> .../devicetree/bindings/phy/phy-cadence-sierra.yaml | 2 --
> .../devicetree/bindings/phy/phy-cadence-torrent.yaml | 4 ----
> .../devicetree/bindings/phy/qcom,ipq806x-usb-phy-hs.yaml | 1 -
> .../devicetree/bindings/phy/qcom,ipq806x-usb-phy-ss.yaml | 1 -
> Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 1 -
> Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml | 2 --
> Documentation/devicetree/bindings/phy/renesas,usb2-phy.yaml | 2 --
> Documentation/devicetree/bindings/phy/renesas,usb3-phy.yaml | 1 -
> .../devicetree/bindings/pinctrl/actions,s500-pinctrl.yaml | 1 -
> .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml | 1 -
> .../devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml | 1 -
> .../devicetree/bindings/remoteproc/st,stm32-rproc.yaml | 2 --
> .../devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml | 1 -
> .../devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml | 1 -
> Documentation/devicetree/bindings/reset/fsl,imx-src.yaml | 1 -
> .../devicetree/bindings/riscv/sifive-l2-cache.yaml | 1 -
> .../devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml | 1 -
> Documentation/devicetree/bindings/rtc/imxdi-rtc.yaml | 1 -
> Documentation/devicetree/bindings/serial/fsl-lpuart.yaml | 2 --
> Documentation/devicetree/bindings/serial/samsung_uart.yaml | 1 -
> .../devicetree/bindings/soc/qcom/qcom,geni-se.yaml | 1 -
> Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 2 --
> .../bindings/sound/nvidia,tegra-audio-graph-card.yaml | 1 -
> .../devicetree/bindings/sound/nvidia,tegra210-i2s.yaml | 2 --
> Documentation/devicetree/bindings/sound/st,stm32-sai.yaml | 3 ---
> .../devicetree/bindings/spi/amlogic,meson-gx-spicc.yaml | 1 -
> .../devicetree/bindings/spi/brcm,spi-bcm-qspi.yaml | 2 --
> .../bindings/thermal/allwinner,sun8i-a83t-ths.yaml | 2 --
> Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 -
> .../bindings/timer/allwinner,sun5i-a13-hstimer.yaml | 1 -
> Documentation/devicetree/bindings/timer/arm,arch_timer.yaml | 1 -
> .../devicetree/bindings/timer/arm,arch_timer_mmio.yaml | 2 --
> .../devicetree/bindings/timer/intel,ixp4xx-timer.yaml | 1 -
> .../devicetree/bindings/usb/maxim,max3420-udc.yaml | 2 --
> .../devicetree/bindings/usb/nvidia,tegra-xudc.yaml | 4 ----
> Documentation/devicetree/bindings/usb/renesas,usbhs.yaml | 3 ---
> .../devicetree/bindings/watchdog/st,stm32-iwdg.yaml | 1 -
> 101 files changed, 2 insertions(+), 163 deletions(-)
>
[snip]
> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> index 6070456a7b67..f399743b631b 100644
> --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> @@ -57,7 +57,6 @@ properties:
>
> memory-region:
> minItems: 2
> - maxItems: 8
> description: |
> phandle to the reserved memory nodes to be associated with the remoteproc
> device. There should be at least two reserved memory nodes defined. The
Does this enforce the maxItems to be 2 only now? Or should this be dropping the
minItems here which matches the length of items instead of maxItems?
I have originally listed the individual item list only for the mandatory items
and rest are scalable. I provided this through "additionalItems: true" under
this property.
Also, have the exact same usage in
Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml as well which
is not included in this patch.
> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml
> index 73400bc6e91d..75161f191ac3 100644
> --- a/Documentation/devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml
> @@ -116,7 +116,6 @@ properties:
> list, in the specified order, each representing the corresponding
> internal RAM memory region.
> minItems: 1
> - maxItems: 3
> items:
> - const: l2ram
> - const: l1pram
[snip]
> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
> index dbc62821c60b..9790617af1bc 100644
> --- a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
> +++ b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
> @@ -100,7 +100,6 @@ patternProperties:
> properties:
> reg:
> minItems: 2 # On AM437x one of two PRUSS units don't contain Shared RAM.
> - maxItems: 3
> items:
> - description: Address and size of the Data RAM0.
> - description: Address and size of the Data RAM1.
> @@ -111,7 +110,6 @@ patternProperties:
>
> reg-names:
> minItems: 2
> - maxItems: 3
> items:
> - const: dram0
> - const: dram1
regards
Suman
On Thu, Jun 17, 2021 at 10:06 AM Suman Anna <s-anna@ti.com> wrote:
>
> Hi Rob,
>
> On 6/15/21 2:15 PM, Rob Herring 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.
> >
> > Cc: Jens Axboe <axboe@kernel.dk>
> > Cc: Stephen Boyd <sboyd@kernel.org>
> > Cc: Herbert Xu <herbert@gondor.apana.org.au>
> > Cc: "David S. Miller" <davem@davemloft.net>
> > Cc: David Airlie <airlied@linux.ie>
> > Cc: Daniel Vetter <daniel@ffwll.ch>
> > Cc: Vinod Koul <vkoul@kernel.org>
> > Cc: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> > Cc: Kamal Dasu <kdasu.kdev@gmail.com>
> > Cc: Jonathan Cameron <jic23@kernel.org>
> > Cc: Lars-Peter Clausen <lars@metafoo.de>
> > Cc: Thomas Gleixner <tglx@linutronix.de>
> > Cc: Marc Zyngier <maz@kernel.org>
> > Cc: Joerg Roedel <joro@8bytes.org>
> > Cc: Jassi Brar <jassisinghbrar@gmail.com>
> > Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> > Cc: Ulf Hansson <ulf.hansson@linaro.org>
> > Cc: Jakub Kicinski <kuba@kernel.org>
> > Cc: Wolfgang Grandegger <wg@grandegger.com>
> > Cc: Marc Kleine-Budde <mkl@pengutronix.de>
> > Cc: Andrew Lunn <andrew@lunn.ch>
> > Cc: Vivien Didelot <vivien.didelot@gmail.com>
> > Cc: Vladimir Oltean <olteanv@gmail.com>
> > Cc: Bjorn Helgaas <bhelgaas@google.com>
> > Cc: Kishon Vijay Abraham I <kishon@ti.com>
> > Cc: Linus Walleij <linus.walleij@linaro.org>
> > Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: Ohad Ben-Cohen <ohad@wizery.com>
> > Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> > Cc: Philipp Zabel <p.zabel@pengutronix.de>
> > Cc: Paul Walmsley <paul.walmsley@sifive.com>
> > Cc: Palmer Dabbelt <palmer@dabbelt.com>
> > Cc: Albert Ou <aou@eecs.berkeley.edu>
> > Cc: Alessandro Zummo <a.zummo@towertech.it>
> > Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>
> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > Cc: Mark Brown <broonie@kernel.org>
> > Cc: Zhang Rui <rui.zhang@intel.com>
> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> > Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
> > Cc: Guenter Roeck <linux@roeck-us.net>
> > Signed-off-by: Rob Herring <robh@kernel.org>
> > ---
> > .../devicetree/bindings/ata/nvidia,tegra-ahci.yaml | 1 -
> > .../devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml | 2 --
> > .../devicetree/bindings/clock/qcom,gcc-apq8064.yaml | 1 -
> > Documentation/devicetree/bindings/clock/qcom,gcc-sdx55.yaml | 2 --
> > .../devicetree/bindings/clock/qcom,gcc-sm8350.yaml | 2 --
> > .../devicetree/bindings/clock/sprd,sc9863a-clk.yaml | 1 -
> > .../devicetree/bindings/crypto/allwinner,sun8i-ce.yaml | 2 --
> > Documentation/devicetree/bindings/crypto/fsl-dcp.yaml | 1 -
> > .../display/allwinner,sun4i-a10-display-backend.yaml | 6 ------
> > .../bindings/display/allwinner,sun6i-a31-mipi-dsi.yaml | 1 -
> > .../bindings/display/allwinner,sun8i-a83t-dw-hdmi.yaml | 4 ----
> > .../bindings/display/allwinner,sun8i-a83t-hdmi-phy.yaml | 2 --
> > .../bindings/display/allwinner,sun8i-r40-tcon-top.yaml | 2 --
> > .../devicetree/bindings/display/bridge/cdns,mhdp8546.yaml | 2 --
> > .../bindings/display/rockchip/rockchip,dw-hdmi.yaml | 2 --
> > Documentation/devicetree/bindings/display/st,stm32-dsi.yaml | 2 --
> > .../devicetree/bindings/display/st,stm32-ltdc.yaml | 1 -
> > .../devicetree/bindings/display/xlnx/xlnx,zynqmp-dpsub.yaml | 4 ----
> > .../devicetree/bindings/dma/renesas,rcar-dmac.yaml | 1 -
> > .../devicetree/bindings/edac/amazon,al-mc-edac.yaml | 2 --
> > Documentation/devicetree/bindings/eeprom/at24.yaml | 1 -
> > Documentation/devicetree/bindings/example-schema.yaml | 2 --
> > Documentation/devicetree/bindings/gpu/brcm,bcm-v3d.yaml | 1 -
> > Documentation/devicetree/bindings/gpu/vivante,gc.yaml | 1 -
> > Documentation/devicetree/bindings/i2c/brcm,brcmstb-i2c.yaml | 1 -
> > .../devicetree/bindings/i2c/marvell,mv64xxx-i2c.yaml | 2 --
> > .../devicetree/bindings/i2c/mellanox,i2c-mlxbf.yaml | 1 -
> > .../devicetree/bindings/iio/adc/amlogic,meson-saradc.yaml | 1 -
> > .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml | 2 --
> > .../bindings/interrupt-controller/fsl,irqsteer.yaml | 1 -
> > .../bindings/interrupt-controller/loongson,liointc.yaml | 1 -
> > Documentation/devicetree/bindings/iommu/arm,smmu-v3.yaml | 1 -
> > .../devicetree/bindings/iommu/renesas,ipmmu-vmsa.yaml | 1 -
> > .../devicetree/bindings/mailbox/st,stm32-ipcc.yaml | 2 --
> > .../devicetree/bindings/media/amlogic,gx-vdec.yaml | 1 -
> > Documentation/devicetree/bindings/media/i2c/adv7604.yaml | 1 -
> > .../devicetree/bindings/media/marvell,mmp2-ccic.yaml | 1 -
> > .../devicetree/bindings/media/qcom,sc7180-venus.yaml | 1 -
> > .../devicetree/bindings/media/qcom,sdm845-venus-v2.yaml | 1 -
> > .../devicetree/bindings/media/qcom,sm8250-venus.yaml | 1 -
> > Documentation/devicetree/bindings/media/renesas,drif.yaml | 1 -
> > .../bindings/memory-controllers/mediatek,smi-common.yaml | 6 ++----
> > .../bindings/memory-controllers/mediatek,smi-larb.yaml | 1 -
> > .../devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 2 --
> > Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml | 1 -
> > Documentation/devicetree/bindings/mmc/mtk-sd.yaml | 2 --
> > Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml | 2 --
> > Documentation/devicetree/bindings/mmc/sdhci-am654.yaml | 1 -
> > Documentation/devicetree/bindings/mmc/sdhci-pxa.yaml | 1 -
> > .../devicetree/bindings/net/amlogic,meson-dwmac.yaml | 2 --
> > .../devicetree/bindings/net/brcm,bcm4908-enet.yaml | 2 --
> > Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 2 --
> > Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml | 2 --
> > Documentation/devicetree/bindings/net/snps,dwmac.yaml | 2 --
> > Documentation/devicetree/bindings/net/stm32-dwmac.yaml | 1 -
> > Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml | 2 --
> > Documentation/devicetree/bindings/pci/loongson.yaml | 1 -
> > .../devicetree/bindings/pci/mediatek-pcie-gen3.yaml | 1 -
> > .../devicetree/bindings/pci/microchip,pcie-host.yaml | 2 --
> > Documentation/devicetree/bindings/perf/arm,cmn.yaml | 1 -
> > .../devicetree/bindings/phy/brcm,bcm63xx-usbh-phy.yaml | 1 -
> > .../devicetree/bindings/phy/brcm,brcmstb-usb-phy.yaml | 3 ---
> > Documentation/devicetree/bindings/phy/brcm,sata-phy.yaml | 1 -
> > Documentation/devicetree/bindings/phy/mediatek,tphy.yaml | 2 --
> > .../devicetree/bindings/phy/phy-cadence-sierra.yaml | 2 --
> > .../devicetree/bindings/phy/phy-cadence-torrent.yaml | 4 ----
> > .../devicetree/bindings/phy/qcom,ipq806x-usb-phy-hs.yaml | 1 -
> > .../devicetree/bindings/phy/qcom,ipq806x-usb-phy-ss.yaml | 1 -
> > Documentation/devicetree/bindings/phy/qcom,qmp-phy.yaml | 1 -
> > Documentation/devicetree/bindings/phy/qcom,qusb2-phy.yaml | 2 --
> > Documentation/devicetree/bindings/phy/renesas,usb2-phy.yaml | 2 --
> > Documentation/devicetree/bindings/phy/renesas,usb3-phy.yaml | 1 -
> > .../devicetree/bindings/pinctrl/actions,s500-pinctrl.yaml | 1 -
> > .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml | 1 -
> > .../devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml | 1 -
> > .../devicetree/bindings/remoteproc/st,stm32-rproc.yaml | 2 --
> > .../devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml | 1 -
> > .../devicetree/bindings/remoteproc/ti,omap-remoteproc.yaml | 1 -
> > Documentation/devicetree/bindings/reset/fsl,imx-src.yaml | 1 -
> > .../devicetree/bindings/riscv/sifive-l2-cache.yaml | 1 -
> > .../devicetree/bindings/rtc/allwinner,sun6i-a31-rtc.yaml | 1 -
> > Documentation/devicetree/bindings/rtc/imxdi-rtc.yaml | 1 -
> > Documentation/devicetree/bindings/serial/fsl-lpuart.yaml | 2 --
> > Documentation/devicetree/bindings/serial/samsung_uart.yaml | 1 -
> > .../devicetree/bindings/soc/qcom/qcom,geni-se.yaml | 1 -
> > Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 2 --
> > .../bindings/sound/nvidia,tegra-audio-graph-card.yaml | 1 -
> > .../devicetree/bindings/sound/nvidia,tegra210-i2s.yaml | 2 --
> > Documentation/devicetree/bindings/sound/st,stm32-sai.yaml | 3 ---
> > .../devicetree/bindings/spi/amlogic,meson-gx-spicc.yaml | 1 -
> > .../devicetree/bindings/spi/brcm,spi-bcm-qspi.yaml | 2 --
> > .../bindings/thermal/allwinner,sun8i-a83t-ths.yaml | 2 --
> > Documentation/devicetree/bindings/thermal/qcom-tsens.yaml | 1 -
> > .../bindings/timer/allwinner,sun5i-a13-hstimer.yaml | 1 -
> > Documentation/devicetree/bindings/timer/arm,arch_timer.yaml | 1 -
> > .../devicetree/bindings/timer/arm,arch_timer_mmio.yaml | 2 --
> > .../devicetree/bindings/timer/intel,ixp4xx-timer.yaml | 1 -
> > .../devicetree/bindings/usb/maxim,max3420-udc.yaml | 2 --
> > .../devicetree/bindings/usb/nvidia,tegra-xudc.yaml | 4 ----
> > Documentation/devicetree/bindings/usb/renesas,usbhs.yaml | 3 ---
> > .../devicetree/bindings/watchdog/st,stm32-iwdg.yaml | 1 -
> > 101 files changed, 2 insertions(+), 163 deletions(-)
> >
>
> [snip]
>
> > diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> > index 6070456a7b67..f399743b631b 100644
> > --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> > +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> > @@ -57,7 +57,6 @@ properties:
> >
> > memory-region:
> > minItems: 2
> > - maxItems: 8
> > description: |
> > phandle to the reserved memory nodes to be associated with the remoteproc
> > device. There should be at least two reserved memory nodes defined. The
>
> Does this enforce the maxItems to be 2 only now? Or should this be dropping the
> minItems here which matches the length of items instead of maxItems?
>
> I have originally listed the individual item list only for the mandatory items
> and rest are scalable. I provided this through "additionalItems: true" under
> this property.
Good catch. This should be dropped. The meta-schema doesn't enforce
this if "additionalItems: true" which is rarely used.
> Also, have the exact same usage in
> Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml as well which
> is not included in this patch.
Yeah, I just missed this one. I've double checked and there aren't any more.
Rob
Hello ROb,
On 6/15/21 9:15 PM, Rob Herring 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/st,stm32-dfsdm-adc.yaml | 2 --
[...]
> .../devicetree/bindings/mailbox/st,stm32-ipcc.yaml | 2 --
[...]
> .../devicetree/bindings/remoteproc/st,stm32-rproc.yaml | 2 --
[...]
> Documentation/devicetree/bindings/sound/st,stm32-sai.yaml | 3 ---
Reviewed-by: Arnaud Pouliquen <arnaud.pouliquen@st.com>
Thanks,
Arnaud
Hi Rob,
On Tue, Jun 15, 2021 at 9:16 PM Rob Herring <robh@kernel.org> 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.
> Signed-off-by: Rob Herring <robh@kernel.org>
> --- a/Documentation/devicetree/bindings/net/stm32-dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/stm32-dwmac.yaml
> @@ -46,7 +46,6 @@ properties:
>
> clocks:
> minItems: 3
> - maxItems: 5
> items:
> - description: GMAC main clock
> - description: MAC TX clock
While resolving the conflict with commit fea99822914039c6
("dt-bindings: net: document ptp_ref clk in dwmac") in soc/for-next,
I noticed the following construct for clock-names:
clock-names:
minItems: 3
maxItems: 6
contains:
enum:
- stmmaceth
- mac-clk-tx
- mac-clk-rx
- ethstp
- eth-ck
- ptp_ref
Should this use items instead of enum, and drop maxItems, or is this
a valid construct to support specifying the clocks in random order?
If the latter, it does mean that the order of clock-names may not
match the order of the clock descriptions.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
On Tue, Jun 22, 2021 at 2:17 AM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> Hi Rob,
>
> On Tue, Jun 15, 2021 at 9:16 PM Rob Herring <robh@kernel.org> 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.
>
> > Signed-off-by: Rob Herring <robh@kernel.org>
>
> > --- a/Documentation/devicetree/bindings/net/stm32-dwmac.yaml
> > +++ b/Documentation/devicetree/bindings/net/stm32-dwmac.yaml
> > @@ -46,7 +46,6 @@ properties:
> >
> > clocks:
> > minItems: 3
> > - maxItems: 5
> > items:
> > - description: GMAC main clock
> > - description: MAC TX clock
>
> While resolving the conflict with commit fea99822914039c6
> ("dt-bindings: net: document ptp_ref clk in dwmac") in soc/for-next,
> I noticed the following construct for clock-names:
>
> clock-names:
> minItems: 3
> maxItems: 6
> contains:
> enum:
> - stmmaceth
> - mac-clk-tx
> - mac-clk-rx
> - ethstp
> - eth-ck
> - ptp_ref
>
> Should this use items instead of enum, and drop maxItems, or is this
> a valid construct to support specifying the clocks in random order?
> If the latter, it does mean that the order of clock-names may not
> match the order of the clock descriptions.
'contains' is true if one or more entries match the strings. So it is
really saying one of these is required. That's not really much of a
constraint. There's 'minContains' and 'maxContains' in newer
json-schema versions (not yet supported) that could add some
constraints if there has to be at least N entries from contains. An
'items' schema (as opposed to a list) would say all items have to
match one of the strings. I'm sure that's too strict.
TLDR: clocks for this binding are a mess and the above is probably all
we can do here.
Rob
@@ -20,7 +20,6 @@ properties:
reg:
minItems: 2
- maxItems: 3
items:
- description: AHCI registers
- description: SATA configuration and IPFS registers
@@ -51,7 +51,6 @@ properties:
clocks:
minItems: 2
- maxItems: 4
items:
- description: High Frequency Oscillator (usually at 24MHz)
- description: Low Frequency Oscillator (usually at 32kHz)
@@ -60,7 +59,6 @@ properties:
clock-names:
minItems: 2
- maxItems: 4
items:
- const: hosc
- const: losc
@@ -46,7 +46,6 @@ properties:
nvmem-cell-names:
minItems: 1
- maxItems: 2
items:
- const: calib
- const: calib_backup
@@ -27,7 +27,6 @@ properties:
- description: Sleep clock source
- description: PLL test clock source (Optional clock)
minItems: 2
- maxItems: 3
clock-names:
items:
@@ -35,7 +34,6 @@ properties:
- const: sleep_clk
- const: core_bi_pll_test_se # Optional clock
minItems: 2
- maxItems: 3
'#clock-cells':
const: 1
@@ -36,7 +36,6 @@ properties:
- description: USB3 phy wrapper pipe clock source (Optional clock)
- description: USB3 phy sec pipe clock source (Optional clock)
minItems: 2
- maxItems: 13
clock-names:
items:
@@ -54,7 +53,6 @@ properties:
- const: usb3_phy_wrapper_gcc_usb30_pipe_clk # Optional clock
- const: usb3_uni_phy_sec_gcc_usb30_pipe_clk # Optional clock
minItems: 2
- maxItems: 13
'#clock-cells':
const: 1
@@ -40,7 +40,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 4
items:
- const: ext-26m
- const: ext-32k
@@ -30,7 +30,6 @@ properties:
- description: Module clock
- description: MBus clock
minItems: 2
- maxItems: 3
clock-names:
items:
@@ -38,7 +37,6 @@ properties:
- const: mod
- const: ram
minItems: 2
- maxItems: 3
resets:
maxItems: 1
@@ -27,7 +27,6 @@ properties:
- description: MXS DCP DCP interrupt
- description: MXS DCP secure interrupt
minItems: 2
- maxItems: 3
clocks:
maxItems: 1
@@ -26,14 +26,12 @@ properties:
reg:
minItems: 1
- maxItems: 2
items:
- description: Display Backend registers
- description: SAT registers
reg-names:
minItems: 1
- maxItems: 2
items:
- const: be
- const: sat
@@ -43,7 +41,6 @@ properties:
clocks:
minItems: 3
- maxItems: 4
items:
- description: The backend interface clock
- description: The backend module clock
@@ -52,7 +49,6 @@ properties:
clock-names:
minItems: 3
- maxItems: 4
items:
- const: ahb
- const: mod
@@ -61,14 +57,12 @@ properties:
resets:
minItems: 1
- maxItems: 2
items:
- description: The Backend reset line
- description: The SAT reset line
reset-names:
minItems: 1
- maxItems: 2
items:
- const: be
- const: sat
@@ -24,7 +24,6 @@ properties:
clocks:
minItems: 1
- maxItems: 2
items:
- description: Bus Clock
- description: Module Clock
@@ -46,7 +46,6 @@ properties:
clocks:
minItems: 3
- maxItems: 6
items:
- description: Bus Clock
- description: Register Clock
@@ -57,7 +56,6 @@ properties:
clock-names:
minItems: 3
- maxItems: 6
items:
- const: iahb
- const: isfr
@@ -68,14 +66,12 @@ properties:
resets:
minItems: 1
- maxItems: 2
items:
- description: HDMI Controller Reset
- description: HDCP Reset
reset-names:
minItems: 1
- maxItems: 2
items:
- const: ctrl
- const: hdcp
@@ -27,7 +27,6 @@ properties:
clocks:
minItems: 2
- maxItems: 4
items:
- description: Bus Clock
- description: Module Clock
@@ -36,7 +35,6 @@ properties:
clock-names:
minItems: 2
- maxItems: 4
items:
- const: bus
- const: mod
@@ -48,7 +48,6 @@ properties:
clocks:
minItems: 2
- maxItems: 6
items:
- description: The TCON TOP interface clock
- description: The TCON TOP TV0 clock
@@ -59,7 +58,6 @@ properties:
clock-names:
minItems: 2
- maxItems: 6
items:
- const: bus
- const: tcon-tv0
@@ -18,7 +18,6 @@ properties:
reg:
minItems: 1
- maxItems: 2
items:
- description:
Register block of mhdptx apb registers up to PHY mapped area (AUX_CONFIG_P).
@@ -29,7 +28,6 @@ properties:
reg-names:
minItems: 1
- maxItems: 2
items:
- const: mhdptx
- const: j721e-intg
@@ -29,7 +29,6 @@ properties:
clocks:
minItems: 2
- maxItems: 5
items:
- {}
- {}
@@ -41,7 +40,6 @@ properties:
clock-names:
minItems: 2
- maxItems: 5
items:
- {}
- {}
@@ -29,7 +29,6 @@ properties:
- description: DSI bus clock
- description: Pixel clock
minItems: 2
- maxItems: 3
clock-names:
items:
@@ -37,7 +36,6 @@ properties:
- const: ref
- const: px_clk
minItems: 2
- maxItems: 3
resets:
maxItems: 1
@@ -22,7 +22,6 @@ properties:
- description: events interrupt line.
- description: errors interrupt line.
minItems: 1
- maxItems: 2
clocks:
maxItems: 1
@@ -65,7 +65,6 @@ properties:
The APB clock and at least one video clock are mandatory, the audio clock
is optional.
minItems: 2
- maxItems: 4
items:
- description: dp_apb_clk is the APB clock
- description: dp_aud_clk is the Audio clock
@@ -78,13 +77,11 @@ properties:
clock-names:
oneOf:
- minItems: 2
- maxItems: 3
items:
- const: dp_apb_clk
- enum: [ dp_vtc_pixel_clk_in, dp_live_video_in_clk ]
- enum: [ dp_vtc_pixel_clk_in, dp_live_video_in_clk ]
- minItems: 3
- maxItems: 4
items:
- const: dp_apb_clk
- const: dp_aud_clk
@@ -116,7 +113,6 @@ properties:
maxItems: 2
phy-names:
minItems: 1
- maxItems: 2
items:
- const: dp-phy0
- const: dp-phy1
@@ -52,7 +52,6 @@ properties:
interrupt-names:
minItems: 9
- maxItems: 17
items:
- const: error
- pattern: "^ch([0-9]|1[0-5])$"
@@ -30,14 +30,12 @@ properties:
interrupts:
minItems: 1
- maxItems: 2
items:
- description: uncorrectable error interrupt
- description: correctable error interrupt
interrupt-names:
minItems: 1
- maxItems: 2
items:
- const: ue
- const: ce
@@ -32,7 +32,6 @@ properties:
oneOf:
- allOf:
- minItems: 1
- maxItems: 2
items:
- pattern: "^(atmel|catalyst|microchip|nxp|ramtron|renesas|rohm|st),(24(c|cs|lc|mac)[0-9]+|spd)$"
- pattern: "^atmel,(24(c|cs|mac)[0-9]+|spd)$"
@@ -91,7 +91,6 @@ properties:
interrupts:
# Either 1 or 2 interrupts can be present
minItems: 1
- maxItems: 2
items:
- description: tx or combined interrupt
- description: rx interrupt
@@ -105,7 +104,6 @@ properties:
interrupt-names:
# minItems must be specified here because the default would be 2
minItems: 1
- maxItems: 2
items:
- const: tx irq
- const: rx irq
@@ -34,7 +34,6 @@ properties:
- enum: [ bridge, gca ]
- enum: [ bridge, gca ]
minItems: 2
- maxItems: 4
interrupts:
items:
@@ -36,7 +36,6 @@ properties:
- description: AHB/slave interface clock (only required if GPU can gate
slave interface independently)
minItems: 1
- maxItems: 4
clock-names:
items:
@@ -21,7 +21,6 @@ properties:
reg:
minItems: 1
- maxItems: 2
items:
- description: BSC register range
- description: Auto-I2C register range
@@ -43,14 +43,12 @@ properties:
clocks:
minItems: 1
- maxItems: 2
items:
- description: Reference clock for the I2C bus
- description: Bus clock (Only for Armada 7K/8K)
clock-names:
minItems: 1
- maxItems: 2
items:
- const: core
- const: reg
@@ -20,7 +20,6 @@ properties:
reg:
minItems: 3
- maxItems: 4
items:
- description: Smbus block registers
- description: Cause master registers
@@ -41,7 +41,6 @@ properties:
clock-names:
minItems: 2
- maxItems: 4
items:
- const: clkin
- const: core
@@ -38,14 +38,12 @@ properties:
dfsdm clock can also feed CLKOUT, when CLKOUT is used.
- description: audio clock can be used as an alternate to feed CLKOUT.
minItems: 1
- maxItems: 2
clock-names:
items:
- const: dfsdm
- const: audio
minItems: 1
- maxItems: 2
"#address-cells":
const: 1
@@ -35,7 +35,6 @@ properties:
- description: output interrupt 6
- description: output interrupt 7
minItems: 1
- maxItems: 8
clocks:
maxItems: 1
@@ -50,7 +50,6 @@ properties:
- const: int2
- const: int3
minItems: 1
- maxItems: 4
'#interrupt-cells':
const: 2
@@ -38,7 +38,6 @@ properties:
If provided, then the combined interrupt will be used in preference to
any others.
- minItems: 2
- maxItems: 4
items:
- const: eventq # Event Queue not empty
- const: gerror # Global Error activated
@@ -49,7 +49,6 @@ properties:
interrupts:
minItems: 1
- maxItems: 2
description:
Specifiers for the MMU fault interrupts. Not required for cache IPMMUs.
items:
@@ -32,7 +32,6 @@ properties:
- description: tx channel free
- description: wakeup source
minItems: 2
- maxItems: 3
interrupt-names:
items:
@@ -40,7 +39,6 @@ properties:
- const: tx
- const: wakeup
minItems: 2
- maxItems: 3
wakeup-source: true
@@ -67,7 +67,6 @@ properties:
clock-names:
minItems: 4
- maxItems: 5
items:
- const: dos_parser
- const: dos
@@ -30,7 +30,6 @@ properties:
reg-names:
minItems: 1
- maxItems: 13
items:
- const: main
- enum: [ avlink, cec, infoframe, esdp, dpp, afe, rep, edid, hdmi, test, cp, vdp ]
@@ -43,7 +43,6 @@ properties:
clocks:
minItems: 1
- maxItems: 3
items:
- description: AXI bus interface clock
- description: Peripheral clock
@@ -30,7 +30,6 @@ properties:
power-domain-names:
minItems: 2
- maxItems: 3
items:
- const: venus
- const: vcodec0
@@ -30,7 +30,6 @@ properties:
power-domain-names:
minItems: 3
- maxItems: 4
items:
- const: venus
- const: vcodec0
@@ -30,7 +30,6 @@ properties:
power-domain-names:
minItems: 2
- maxItems: 3
items:
- const: venus
- const: vcodec0
@@ -78,7 +78,6 @@ properties:
dma-names:
minItems: 1
- maxItems: 2
items:
- const: rx
- const: rx
@@ -53,14 +53,12 @@ properties:
apb and smi are mandatory. the async is only for generation 1 smi HW.
gals(global async local sync) also is optional, see below.
minItems: 2
- maxItems: 4
items:
- description: apb is Advanced Peripheral Bus clock, It's the clock for
setting the register.
- description: smi is the clock for transfer data and command.
- - description: async is asynchronous clock, it help transform the smi
- clock into the emi clock domain.
- - description: gals0 is the path0 clock of gals.
+ - description: Either asynchronous clock to help transform the smi clock
+ into the emi clock domain on Gen1 h/w, or the path0 clock of gals.
- description: gals1 is the path1 clock of gals.
clock-names:
@@ -37,7 +37,6 @@ properties:
description: |
apb and smi are mandatory. gals(global async local sync) is optional.
minItems: 2
- maxItems: 3
items:
- description: apb is Advanced Peripheral Bus clock, It's the clock for
setting the register.
@@ -64,7 +64,6 @@ properties:
clocks:
minItems: 2
- maxItems: 4
items:
- description: Bus Clock
- description: Module Clock
@@ -73,7 +72,6 @@ properties:
clock-names:
minItems: 2
- maxItems: 4
items:
- const: ahb
- const: mmc
@@ -116,7 +116,6 @@ properties:
pinctrl-names:
minItems: 1
- maxItems: 4
items:
- const: default
- const: state_100mhz
@@ -38,7 +38,6 @@ properties:
description:
Should contain phandle for the clock feeding the MMC controller.
minItems: 2
- maxItems: 8
items:
- description: source clock (required).
- description: HCLK which used for host (required).
@@ -51,7 +50,6 @@ properties:
clock-names:
minItems: 2
- maxItems: 8
items:
- const: source
- const: hclk
@@ -75,7 +75,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: core
- const: cd
@@ -107,7 +106,6 @@ properties:
pinctrl-names:
minItems: 1
- maxItems: 2
items:
- const: default
- const: state_uhs
@@ -45,7 +45,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: clk_ahb
- const: clk_xin
@@ -57,7 +57,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: io
- const: core
@@ -43,7 +43,6 @@ allOf:
properties:
clocks:
minItems: 3
- maxItems: 4
items:
- description: GMAC main clock
- description: First parent clock of the internal mux
@@ -52,7 +51,6 @@ allOf:
clock-names:
minItems: 3
- maxItems: 4
items:
- const: stmmaceth
- const: clkin0
@@ -23,14 +23,12 @@ properties:
interrupts:
minItems: 1
- maxItems: 2
items:
- description: RX interrupt
- description: TX interrupt
interrupt-names:
minItems: 1
- maxItems: 2
items:
- const: rx
- const: tx
@@ -30,14 +30,12 @@ properties:
- description: interrupt line0
- description: interrupt line1
minItems: 1
- maxItems: 2
interrupt-names:
items:
- const: int0
- const: int1
minItems: 1
- maxItems: 2
clocks:
items:
@@ -48,14 +48,12 @@ properties:
clocks:
minItems: 1
- maxItems: 2
items:
- description: switch's main clock
- description: dividing of the switch core clock
clock-names:
minItems: 1
- maxItems: 2
items:
- const: sw_switch
- const: sw_switch_mdiv
@@ -82,7 +82,6 @@ properties:
interrupts:
minItems: 1
- maxItems: 3
items:
- description: Combined signal for various interrupt events
- description: The interrupt to manage the remote wake-up packet detection
@@ -90,7 +89,6 @@ properties:
interrupt-names:
minItems: 1
- maxItems: 3
items:
- const: macirq
- const: eth_wake_irq
@@ -46,7 +46,6 @@ properties:
clocks:
minItems: 3
- maxItems: 5
items:
- description: GMAC main clock
- description: MAC TX clock
@@ -25,14 +25,12 @@ properties:
interrupts:
minItems: 1
- maxItems: 2
items:
- description: PCIe host controller
- description: builtin MSI controller
interrupt-names:
minItems: 1
- maxItems: 2
items:
- const: pcie
- const: msi
@@ -24,7 +24,6 @@ properties:
reg:
minItems: 1
- maxItems: 2
items:
- description: CFG0 standard config space register
- description: CFG1 extended config space register
@@ -70,7 +70,6 @@ properties:
reset-names:
minItems: 1
- maxItems: 2
items:
- const: phy
- const: mac
@@ -26,14 +26,12 @@ properties:
interrupts:
minItems: 1
- maxItems: 2
items:
- description: PCIe host controller
- description: builtin MSI controller
interrupt-names:
minItems: 1
- maxItems: 2
items:
- const: pcie
- const: msi
@@ -21,7 +21,6 @@ properties:
interrupts:
minItems: 1
- maxItems: 4
items:
- description: Overflow interrupt for DTC0
- description: Overflow interrupt for DTC1
@@ -28,7 +28,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: usbh
- const: usb_ref
@@ -22,7 +22,6 @@ properties:
reg:
minItems: 1
- maxItems: 6
items:
- description: the base CTRL register
- description: XHCI EC register
@@ -33,7 +32,6 @@ properties:
reg-names:
minItems: 1
- maxItems: 6
items:
- const: ctrl
- const: xhci_ec
@@ -51,7 +49,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: sw_usb
- const: sw_usb3
@@ -35,7 +35,6 @@ properties:
reg-names:
minItems: 1
- maxItems: 2
items:
- const: phy
- const: phy-ctrl
@@ -131,7 +131,6 @@ patternProperties:
clocks:
minItems: 1
- maxItems: 2
items:
- description: Reference clock, (HS is 48Mhz, SS/P is 24~27Mhz)
- description: Reference clock of analog phy
@@ -141,7 +140,6 @@ patternProperties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: ref
- const: da_ref
@@ -31,14 +31,12 @@ properties:
resets:
minItems: 1
- maxItems: 2
items:
- description: Sierra PHY reset.
- description: Sierra APB reset. This is optional.
reset-names:
minItems: 1
- maxItems: 2
items:
- const: sierra_reset
- const: sierra_apb
@@ -52,28 +52,24 @@ properties:
reg:
minItems: 1
- maxItems: 2
items:
- description: Offset of the Torrent PHY configuration registers.
- description: Offset of the DPTX PHY configuration registers.
reg-names:
minItems: 1
- maxItems: 2
items:
- const: torrent_phy
- const: dptx_phy
resets:
minItems: 1
- maxItems: 2
items:
- description: Torrent PHY reset.
- description: Torrent APB reset. This is optional.
reset-names:
minItems: 1
- maxItems: 2
items:
- const: torrent_reset
- const: torrent_apb
@@ -30,7 +30,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: ref
- const: xo
@@ -30,7 +30,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: ref
- const: xo
@@ -49,7 +49,6 @@ properties:
reg:
minItems: 1
- maxItems: 2
items:
- description: Address and length of PHY's common serdes block.
- description: Address and length of PHY's DP_COM control block.
@@ -36,7 +36,6 @@ properties:
clocks:
minItems: 2
- maxItems: 3
items:
- description: phy config clock
- description: 19.2 MHz ref clk
@@ -44,7 +43,6 @@ properties:
clock-names:
minItems: 2
- maxItems: 3
items:
- const: cfg_ahb
- const: ref
@@ -39,7 +39,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: fck
- const: usb_x1
@@ -61,7 +60,6 @@ properties:
resets:
minItems: 1
- maxItems: 2
items:
- description: reset of USB 2.0 host side
- description: reset of USB 2.0 peripheral side
@@ -33,7 +33,6 @@ properties:
# If you want to use the ssc, the clock-frequency of usb_extal
# must not be 0.
minItems: 2
- maxItems: 3
items:
- const: usb3-if # The funcional clock
- const: usb3s_clk # The usb3's external clock
@@ -26,7 +26,6 @@ properties:
- description: PAD Pull Control + PAD Schmitt Trigger Enable + PAD Control
- description: PAD Drive Capacity Select
minItems: 1
- maxItems: 4
clocks:
maxItems: 1
@@ -37,7 +37,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: vpu
- const: vapb
@@ -37,7 +37,6 @@ properties:
clocks:
minItems: 1
- maxItems: 2
items:
- description: Module Clock
- description: Bus Clock
@@ -72,7 +72,6 @@ properties:
- from local to remote, where ACK from the remote means that communnication
as been stopped on the remote side.
minItems: 1
- maxItems: 4
mbox-names:
items:
@@ -81,7 +80,6 @@ properties:
- const: shutdown
- const: detach
minItems: 1
- maxItems: 4
memory-region:
description:
@@ -57,7 +57,6 @@ properties:
memory-region:
minItems: 2
- maxItems: 8
description: |
phandle to the reserved memory nodes to be associated with the remoteproc
device. There should be at least two reserved memory nodes defined. The
@@ -116,7 +116,6 @@ properties:
list, in the specified order, each representing the corresponding
internal RAM memory region.
minItems: 1
- maxItems: 3
items:
- const: l2ram
- const: l1pram
@@ -59,7 +59,6 @@ properties:
- description: SRC interrupt
- description: CPU WDOG interrupts out of SRC
minItems: 1
- maxItems: 2
'#reset-cells':
const: 1
@@ -56,7 +56,6 @@ properties:
interrupts:
minItems: 3
- maxItems: 4
items:
- description: DirError interrupt
- description: DataError interrupt
@@ -32,7 +32,6 @@ properties:
interrupts:
minItems: 1
- maxItems: 2
items:
- description: RTC Alarm 0
- description: RTC Alarm 1
@@ -21,7 +21,6 @@ properties:
- description: rtc alarm interrupt
- description: dryice security violation interrupt
minItems: 1
- maxItems: 2
clocks:
maxItems: 1
@@ -36,14 +36,12 @@ properties:
- description: ipg clock
- description: baud clock
minItems: 1
- maxItems: 2
clock-names:
items:
- const: ipg
- const: baud
minItems: 1
- maxItems: 2
dmas:
items:
@@ -44,7 +44,6 @@ properties:
clock-names:
description: N = 0 is allowed for SoCs without internal baud clock mux.
minItems: 2
- maxItems: 5
items:
- const: uart
- pattern: '^clk_uart_baud[0-3]$'
@@ -164,7 +164,6 @@ patternProperties:
interrupts:
minItems: 1
- maxItems: 2
items:
- description: UART core irq
- description: Wakeup irq (RX GPIO)
@@ -100,7 +100,6 @@ patternProperties:
properties:
reg:
minItems: 2 # On AM437x one of two PRUSS units don't contain Shared RAM.
- maxItems: 3
items:
- description: Address and size of the Data RAM0.
- description: Address and size of the Data RAM1.
@@ -111,7 +110,6 @@ patternProperties:
reg-names:
minItems: 2
- maxItems: 3
items:
- const: dram0
- const: dram1
@@ -28,7 +28,6 @@ properties:
minItems: 2
clock-names:
- minItems: 2
items:
- const: pll_a
- const: plla_out0
@@ -34,7 +34,6 @@ properties:
clocks:
minItems: 1
- maxItems: 2
items:
- description: I2S bit clock
- description:
@@ -48,7 +47,6 @@ properties:
clock-names:
minItems: 1
- maxItems: 2
items:
- const: i2s
- const: sync_input
@@ -26,7 +26,6 @@ properties:
- description: Base address and size of SAI common register set.
- description: Base address and size of SAI identification register set.
minItems: 1
- maxItems: 2
ranges:
maxItems: 1
@@ -81,14 +80,12 @@ patternProperties:
- description: sai_ck clock feeding the internal clock generator.
- description: MCLK clock from a SAI set as master clock provider.
minItems: 1
- maxItems: 2
clock-names:
items:
- const: sai_ck
- const: MCLK
minItems: 1
- maxItems: 2
dmas:
maxItems: 1
@@ -35,7 +35,6 @@ properties:
clocks:
minItems: 1
- maxItems: 2
items:
- description: controller register bus clock
- description: baud rate generator and delay control clock
@@ -56,7 +56,6 @@ properties:
reg-names:
minItems: 1
- maxItems: 5
items:
- const: mspi
- const: bspi
@@ -71,7 +70,6 @@ properties:
interrupt-names:
oneOf:
- minItems: 1
- maxItems: 7
items:
- const: mspi_done
- const: mspi_halted
@@ -23,14 +23,12 @@ properties:
clocks:
minItems: 1
- maxItems: 2
items:
- description: Bus Clock
- description: Module Clock
clock-names:
minItems: 1
- maxItems: 2
items:
- const: bus
- const: mod
@@ -77,7 +77,6 @@ properties:
nvmem-cell-names:
minItems: 1
- maxItems: 2
items:
- const: calib
- enum:
@@ -24,7 +24,6 @@ properties:
interrupts:
minItems: 2
- maxItems: 4
items:
- description: Timer 0 Interrupt
- description: Timer 1 Interrupt
@@ -35,7 +35,6 @@ properties:
interrupts:
minItems: 1
- maxItems: 5
items:
- description: secure timer irq
- description: non-secure timer irq
@@ -71,14 +71,12 @@ patternProperties:
interrupts:
minItems: 1
- maxItems: 2
items:
- description: physical timer irq
- description: virtual timer irq
reg:
minItems: 1
- maxItems: 2
items:
- description: 1st view base address
- description: 2nd optional view base address
@@ -22,7 +22,6 @@ properties:
interrupts:
minItems: 1
- maxItems: 2
items:
- description: Timer 1 interrupt
- description: Timer 2 interrupt
@@ -30,14 +30,12 @@ properties:
- description: usb irq from max3420
- description: vbus detection irq
minItems: 1
- maxItems: 2
interrupt-names:
items:
- const: udc
- const: vbus
minItems: 1
- maxItems: 2
spi-max-frequency:
maximum: 26000000
@@ -25,7 +25,6 @@ properties:
reg:
minItems: 2
- maxItems: 3
items:
- description: XUSB device controller registers
- description: XUSB device PCI Config registers
@@ -33,7 +32,6 @@ properties:
reg-names:
minItems: 2
- maxItems: 3
items:
- const: base
- const: fpci
@@ -45,7 +43,6 @@ properties:
clocks:
minItems: 4
- maxItems: 5
items:
- description: Clock to enable core XUSB dev clock.
- description: Clock to enable XUSB super speed clock.
@@ -55,7 +52,6 @@ properties:
clock-names:
minItems: 4
- maxItems: 5
items:
- const: dev
- const: ss
@@ -53,7 +53,6 @@ properties:
clocks:
minItems: 1
- maxItems: 3
items:
- description: USB 2.0 host
- description: USB 2.0 peripheral
@@ -86,7 +85,6 @@ properties:
dma-names:
minItems: 2
- maxItems: 4
items:
- const: ch0
- const: ch1
@@ -100,7 +98,6 @@ properties:
resets:
minItems: 1
- maxItems: 2
items:
- description: USB 2.0 host
- description: USB 2.0 peripheral
@@ -27,7 +27,6 @@ properties:
- description: Low speed clock
- description: Optional peripheral clock
minItems: 1
- maxItems: 2
clock-names:
items: