Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
Hi, On Tue, Jan 24, 2023 at 05:02:28PM -0600, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- > [...] > diff --git a/Documentation/devicetree/bindings/power/supply/ti,lp8727.yaml > b/Documentation/devicetree/bindings/power/supply/ti,lp8727.yaml > index ce6fbdba8f6b..0542d4126cf5 100644 > --- a/Documentation/devicetree/bindings/power/supply/ti,lp8727.yaml > +++ b/Documentation/devicetree/bindings/power/supply/ti,lp8727.yaml > @@ -28,6 +28,7 @@ properties: > patternProperties: >'^(ac|usb)$': > type: object > +additionalProperties: false > description: USB/AC charging parameters > properties: >charger-type: Acked-by: Sebastian Reichel -- Sebastian signature.asc Description: PGP signature
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Tue, Jan 24, 2023 at 5:00 PM Rob Herring wrote: > > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- For the mailbox change, Acked-by: Jassi Brar
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Tue, 24 Jan 2023, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- > To: Krzysztof Kozlowski > To: David Airlie > To: Daniel Vetter > To: Bartosz Golaszewski > To: Jean Delvare > To: Guenter Roeck > To: Thomas Gleixner > To: Marc Zyngier > To: Jassi Brar > To: Mauro Carvalho Chehab > To: Lee Jones > To: Ulf Hansson > To: Richard Weinberger > To: Vignesh Raghavendra > To: Sebastian Reichel > To: Mark Brown > To: "Rafael J. Wysocki" > To: Daniel Lezcano > To: Greg Kroah-Hartman > Cc: linux-arm-ker...@lists.infradead.org > Cc: dri-devel@lists.freedesktop.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: open...@lists.ozlabs.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@lists.infradead.org > Cc: linux...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@vger.kernel.org > --- > .../devicetree/bindings/arm/arm,vexpress-juno.yaml | 1 + > .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml | 5 +++-- > .../bindings/arm/tegra/nvidia,tegra20-pmc.yaml | 4 > .../bindings/bus/allwinner,sun50i-a64-de2.yaml | 1 + > .../bindings/bus/allwinner,sun8i-a23-rsb.yaml | 1 + > .../bus/intel,ixp4xx-expansion-bus-controller.yaml | 6 ++ > Documentation/devicetree/bindings/bus/palmbus.yaml | 1 + > .../devicetree/bindings/display/msm/qcom,mdss.yaml | 5 + > Documentation/devicetree/bindings/example-schema.yaml | 2 ++ > .../devicetree/bindings/gpio/x-powers,axp209-gpio.yaml | 1 + > .../devicetree/bindings/hwmon/adi,ltc2992.yaml | 1 + > .../bindings/interrupt-controller/arm,gic-v3.yaml | 2 ++ > .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml | 1 + > .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++ > .../bindings/memory-controllers/arm,pl35x-smc.yaml | 1 + > .../bindings/memory-controllers/exynos-srom.yaml | 1 + > .../memory-controllers/nvidia,tegra124-emc.yaml| 1 + > .../bindings/memory-controllers/st,stm32-fmc2-ebi.yaml | 1 + > .../devicetree/bindings/mfd/mediatek,mt6370.yaml | 2 ++ > .../devicetree/bindings/mmc/aspeed,sdhci.yaml | 1 + > Documentation/devicetree/bindings/mtd/mtd.yaml | 1 + > .../devicetree/bindings/power/supply/ti,lp8727.yaml| 1 + > .../devicetree/bindings/soc/imx/fsl,imx93-src.yaml | 3 ++- > .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 1 + > Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 1 + > .../devicetree/bindings/sound/marvell,mmp-sspa.yaml| 1 + > .../devicetree/bindings/sound/qcom,wcd934x.yaml| 1 + > .../devicetree/bindings/sound/samsung,odroid.yaml | 2 ++ > .../devicetree/bindings/soundwire/qcom,soundwire.yaml | 1 + > .../bindings/spi/allwinner,sun4i-a10-spi.yaml | 1 + > .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + > .../devicetree/bindings/spi/spi-controller.yaml| 1 + > .../sram/allwinner,sun4i-a10-system-control.yaml | 10 +- > Documentation/devicetree/bindings/sram/qcom,ocmem.yaml | 1 + > .../devicetree/bindings/thermal/thermal-zones.yaml | 1 + > Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 + > 36 files changed, 65 insertions(+), 8 deletions(-) Is this the same as the patch I just reviewed? -- Lee Jones [李琼斯]
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Tue, 24 Jan 2023, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- > To: Krzysztof Kozlowski > To: David Airlie > To: Daniel Vetter > To: Bartosz Golaszewski > To: Jean Delvare > To: Guenter Roeck > To: Thomas Gleixner > To: Marc Zyngier > To: Jassi Brar > To: Mauro Carvalho Chehab > To: Lee Jones > To: Ulf Hansson > To: Richard Weinberger > To: Vignesh Raghavendra > To: Sebastian Reichel > To: Mark Brown > To: "Rafael J. Wysocki" > To: Daniel Lezcano > To: Greg Kroah-Hartman > Cc: linux-arm-ker...@lists.infradead.org > Cc: dri-devel@lists.freedesktop.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: open...@lists.ozlabs.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@lists.infradead.org > Cc: linux...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@vger.kernel.org > --- > .../devicetree/bindings/arm/arm,vexpress-juno.yaml | 1 + > .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml | 5 +++-- > .../bindings/arm/tegra/nvidia,tegra20-pmc.yaml | 4 > .../bindings/bus/allwinner,sun50i-a64-de2.yaml | 1 + > .../bindings/bus/allwinner,sun8i-a23-rsb.yaml | 1 + > .../bus/intel,ixp4xx-expansion-bus-controller.yaml | 6 ++ > Documentation/devicetree/bindings/bus/palmbus.yaml | 1 + > .../devicetree/bindings/display/msm/qcom,mdss.yaml | 5 + > Documentation/devicetree/bindings/example-schema.yaml | 2 ++ > .../devicetree/bindings/gpio/x-powers,axp209-gpio.yaml | 1 + > .../devicetree/bindings/hwmon/adi,ltc2992.yaml | 1 + > .../bindings/interrupt-controller/arm,gic-v3.yaml | 2 ++ > .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml | 1 + > .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++ > .../bindings/memory-controllers/arm,pl35x-smc.yaml | 1 + > .../bindings/memory-controllers/exynos-srom.yaml | 1 + > .../memory-controllers/nvidia,tegra124-emc.yaml| 1 + > .../bindings/memory-controllers/st,stm32-fmc2-ebi.yaml | 1 + > .../devicetree/bindings/mfd/mediatek,mt6370.yaml | 2 ++ > .../devicetree/bindings/mmc/aspeed,sdhci.yaml | 1 + > Documentation/devicetree/bindings/mtd/mtd.yaml | 1 + > .../devicetree/bindings/power/supply/ti,lp8727.yaml| 1 + > .../devicetree/bindings/soc/imx/fsl,imx93-src.yaml | 3 ++- > .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 1 + > Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 1 + > .../devicetree/bindings/sound/marvell,mmp-sspa.yaml| 1 + > .../devicetree/bindings/sound/qcom,wcd934x.yaml| 1 + > .../devicetree/bindings/sound/samsung,odroid.yaml | 2 ++ > .../devicetree/bindings/soundwire/qcom,soundwire.yaml | 1 + > .../bindings/spi/allwinner,sun4i-a10-spi.yaml | 1 + > .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + > .../devicetree/bindings/spi/spi-controller.yaml| 1 + > .../sram/allwinner,sun4i-a10-system-control.yaml | 10 +- > Documentation/devicetree/bindings/sram/qcom,ocmem.yaml | 1 + > .../devicetree/bindings/thermal/thermal-zones.yaml | 1 + > Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 + > 36 files changed, 65 insertions(+), 8 deletions(-) Acked-by: Lee Jones -- Lee Jones [李琼斯]
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Tue, Jan 24, 2023 at 05:02:28PM -0600, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring Acked-by: Greg Kroah-Hartman
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Tue, Jan 24, 2023 at 05:00:48PM -0600, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- > To: Krzysztof Kozlowski > To: David Airlie > To: Daniel Vetter > To: Bartosz Golaszewski > To: Jean Delvare > To: Guenter Roeck > To: Thomas Gleixner > To: Marc Zyngier > To: Jassi Brar > To: Mauro Carvalho Chehab > To: Lee Jones > To: Ulf Hansson > To: Richard Weinberger > To: Vignesh Raghavendra > To: Sebastian Reichel > To: Mark Brown > To: "Rafael J. Wysocki" > To: Daniel Lezcano > To: Greg Kroah-Hartman > Cc: linux-arm-ker...@lists.infradead.org > Cc: dri-devel@lists.freedesktop.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: open...@lists.ozlabs.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@lists.infradead.org > Cc: linux...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@vger.kernel.org Acked-by: Greg Kroah-Hartman
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Wed, Jan 25, 2023 at 12:00 AM Rob Herring wrote: > > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- [...] > .../devicetree/bindings/gpio/x-powers,axp209-gpio.yaml | 1 + [...] > diff --git a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml > b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml > index 7f26f6b1eea1..31906c253940 100644 > --- a/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml > +++ b/Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml > @@ -35,6 +35,7 @@ properties: > patternProperties: >"^.*-pins?$": > $ref: /schemas/pinctrl/pinmux-node.yaml# > +additionalProperties: false > > properties: >pins: Acked-by: Bartosz Golaszewski [...]
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Tue, Jan 24, 2023 at 05:02:28PM -0600, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. Acked-by: Mark Brown signature.asc Description: PGP signature
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Tue, Jan 24, 2023 at 05:00:48PM -0600, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. Acked-by: Mark Brown signature.asc Description: PGP signature
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Wed, 25 Jan 2023 at 00:00, Rob Herring wrote: > > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring Acked-by: Ulf Hansson # For MMC Kind regards Uffe > --- > To: Krzysztof Kozlowski > To: David Airlie > To: Daniel Vetter > To: Bartosz Golaszewski > To: Jean Delvare > To: Guenter Roeck > To: Thomas Gleixner > To: Marc Zyngier > To: Jassi Brar > To: Mauro Carvalho Chehab > To: Lee Jones > To: Ulf Hansson > To: Richard Weinberger > To: Vignesh Raghavendra > To: Sebastian Reichel > To: Mark Brown > To: "Rafael J. Wysocki" > To: Daniel Lezcano > To: Greg Kroah-Hartman > Cc: linux-arm-ker...@lists.infradead.org > Cc: dri-devel@lists.freedesktop.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: open...@lists.ozlabs.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@lists.infradead.org > Cc: linux...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@vger.kernel.org > --- > .../devicetree/bindings/arm/arm,vexpress-juno.yaml | 1 + > .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml | 5 +++-- > .../bindings/arm/tegra/nvidia,tegra20-pmc.yaml | 4 > .../bindings/bus/allwinner,sun50i-a64-de2.yaml | 1 + > .../bindings/bus/allwinner,sun8i-a23-rsb.yaml | 1 + > .../bus/intel,ixp4xx-expansion-bus-controller.yaml | 6 ++ > Documentation/devicetree/bindings/bus/palmbus.yaml | 1 + > .../devicetree/bindings/display/msm/qcom,mdss.yaml | 5 + > Documentation/devicetree/bindings/example-schema.yaml | 2 ++ > .../devicetree/bindings/gpio/x-powers,axp209-gpio.yaml | 1 + > .../devicetree/bindings/hwmon/adi,ltc2992.yaml | 1 + > .../bindings/interrupt-controller/arm,gic-v3.yaml | 2 ++ > .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml | 1 + > .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++ > .../bindings/memory-controllers/arm,pl35x-smc.yaml | 1 + > .../bindings/memory-controllers/exynos-srom.yaml | 1 + > .../memory-controllers/nvidia,tegra124-emc.yaml| 1 + > .../bindings/memory-controllers/st,stm32-fmc2-ebi.yaml | 1 + > .../devicetree/bindings/mfd/mediatek,mt6370.yaml | 2 ++ > .../devicetree/bindings/mmc/aspeed,sdhci.yaml | 1 + > Documentation/devicetree/bindings/mtd/mtd.yaml | 1 + > .../devicetree/bindings/power/supply/ti,lp8727.yaml| 1 + > .../devicetree/bindings/soc/imx/fsl,imx93-src.yaml | 3 ++- > .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 1 + > Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 1 + > .../devicetree/bindings/sound/marvell,mmp-sspa.yaml| 1 + > .../devicetree/bindings/sound/qcom,wcd934x.yaml| 1 + > .../devicetree/bindings/sound/samsung,odroid.yaml | 2 ++ > .../devicetree/bindings/soundwire/qcom,soundwire.yaml | 1 + > .../bindings/spi/allwinner,sun4i-a10-spi.yaml | 1 + > .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + > .../devicetree/bindings/spi/spi-controller.yaml| 1 + > .../sram/allwinner,sun4i-a10-system-control.yaml | 10 +- > Documentation/devicetree/bindings/sram/qcom,ocmem.yaml | 1 + > .../devicetree/bindings/thermal/thermal-zones.yaml | 1 + > Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 + > 36 files changed, 65 insertions(+), 8 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > index eec190a96225..09c319f803ba 100644 > --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > @@ -144,6 +144,7 @@ patternProperties: >it is stricter and always has two compatibles. > type: object > $ref: '/schemas/simple-bus.yaml' > +unevaluatedProperties: false > > properties: >compatible: > diff --git > a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml > b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml > index b369b374fc4a..39e3c248f5b7 100644 > --- > a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml > +++ > b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml > @@ -30,6 +30,7 @@ properties: > >clocks: > type: object > +additionalProperties: false > > properties: >compatible: > @@ -47,6 +48,7 @@ properties: > >reset: > type: object > +additionalProperties: false > > properties: >
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On 25/01/2023 00:02, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof
Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
On Tue, Jan 24, 2023 at 05:02:28PM -0600, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- > To: Krzysztof Kozlowski > To: David Airlie > To: Daniel Vetter > To: Bartosz Golaszewski > To: Jean Delvare > To: Guenter Roeck > To: Thomas Gleixner > To: Marc Zyngier > To: Jassi Brar > To: Mauro Carvalho Chehab > To: Lee Jones > To: Ulf Hansson > To: Richard Weinberger > To: Vignesh Raghavendra > To: Sebastian Reichel > To: Mark Brown > To: "Rafael J. Wysocki" > To: Daniel Lezcano > To: Greg Kroah-Hartman > Cc: linux-arm-ker...@lists.infradead.org > Cc: dri-devel@lists.freedesktop.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org For hwmon: Acked-by: Guenter Roeck > Cc: linux-me...@vger.kernel.org > Cc: open...@lists.ozlabs.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@lists.infradead.org > Cc: linux...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Cc: linux-...@vger.kernel.org > --- > .../devicetree/bindings/arm/arm,vexpress-juno.yaml | 1 + > .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml | 5 +++-- > .../bindings/arm/tegra/nvidia,tegra20-pmc.yaml | 4 > .../bindings/bus/allwinner,sun50i-a64-de2.yaml | 1 + > .../bindings/bus/allwinner,sun8i-a23-rsb.yaml | 1 + > .../bus/intel,ixp4xx-expansion-bus-controller.yaml | 6 ++ > Documentation/devicetree/bindings/bus/palmbus.yaml | 1 + > .../devicetree/bindings/display/msm/qcom,mdss.yaml | 5 + > Documentation/devicetree/bindings/example-schema.yaml | 2 ++ > .../devicetree/bindings/gpio/x-powers,axp209-gpio.yaml | 1 + > .../devicetree/bindings/hwmon/adi,ltc2992.yaml | 1 + > .../bindings/interrupt-controller/arm,gic-v3.yaml | 2 ++ > .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml | 1 + > .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++ > .../bindings/memory-controllers/arm,pl35x-smc.yaml | 1 + > .../bindings/memory-controllers/exynos-srom.yaml | 1 + > .../memory-controllers/nvidia,tegra124-emc.yaml| 1 + > .../bindings/memory-controllers/st,stm32-fmc2-ebi.yaml | 1 + > .../devicetree/bindings/mfd/mediatek,mt6370.yaml | 2 ++ > .../devicetree/bindings/mmc/aspeed,sdhci.yaml | 1 + > Documentation/devicetree/bindings/mtd/mtd.yaml | 1 + > .../devicetree/bindings/power/supply/ti,lp8727.yaml| 1 + > .../devicetree/bindings/soc/imx/fsl,imx93-src.yaml | 3 ++- > .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 1 + > Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 1 + > .../devicetree/bindings/sound/marvell,mmp-sspa.yaml| 1 + > .../devicetree/bindings/sound/qcom,wcd934x.yaml| 1 + > .../devicetree/bindings/sound/samsung,odroid.yaml | 2 ++ > .../devicetree/bindings/soundwire/qcom,soundwire.yaml | 1 + > .../bindings/spi/allwinner,sun4i-a10-spi.yaml | 1 + > .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + > .../devicetree/bindings/spi/spi-controller.yaml| 1 + > .../sram/allwinner,sun4i-a10-system-control.yaml | 10 +- > Documentation/devicetree/bindings/sram/qcom,ocmem.yaml | 1 + > .../devicetree/bindings/thermal/thermal-zones.yaml | 1 + > Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 + > 36 files changed, 65 insertions(+), 8 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > index eec190a96225..09c319f803ba 100644 > --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml > @@ -144,6 +144,7 @@ patternProperties: >it is stricter and always has two compatibles. > type: object > $ref: '/schemas/simple-bus.yaml' > +unevaluatedProperties: false > > properties: >compatible: > diff --git > a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml > b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml > index b369b374fc4a..39e3c248f5b7 100644 > --- > a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml > +++ > b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml > @@ -30,6 +30,7 @@ properties: > >clocks: > type: object > +additionalProperties: false > > properties: >compatible: > @@ -47,6 +48,7 @@ properties: > >reset: > type: object > +additionalProperties: false > >
[PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
Just as unevaluatedProperties or additionalProperties are required at the top level of schemas, they should (and will) also be required for child node schemas. That ensures only documented properties are present. Add unevaluatedProperties or additionalProperties as appropriate, and then add any missing properties flagged by the addition. Signed-off-by: Rob Herring --- To: Krzysztof Kozlowski To: David Airlie To: Daniel Vetter To: Bartosz Golaszewski To: Jean Delvare To: Guenter Roeck To: Thomas Gleixner To: Marc Zyngier To: Jassi Brar To: Mauro Carvalho Chehab To: Lee Jones To: Ulf Hansson To: Richard Weinberger To: Vignesh Raghavendra To: Sebastian Reichel To: Mark Brown To: "Rafael J. Wysocki" To: Daniel Lezcano To: Greg Kroah-Hartman Cc: linux-arm-ker...@lists.infradead.org Cc: dri-devel@lists.freedesktop.org Cc: linux-g...@vger.kernel.org Cc: linux-hw...@vger.kernel.org Cc: linux-me...@vger.kernel.org Cc: open...@lists.ozlabs.org Cc: linux-...@vger.kernel.org Cc: linux-...@lists.infradead.org Cc: linux...@vger.kernel.org Cc: alsa-de...@alsa-project.org Cc: linux-...@vger.kernel.org Cc: linux-...@vger.kernel.org --- .../devicetree/bindings/arm/arm,vexpress-juno.yaml | 1 + .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml | 5 +++-- .../bindings/arm/tegra/nvidia,tegra20-pmc.yaml | 4 .../bindings/bus/allwinner,sun50i-a64-de2.yaml | 1 + .../bindings/bus/allwinner,sun8i-a23-rsb.yaml | 1 + .../bus/intel,ixp4xx-expansion-bus-controller.yaml | 6 ++ Documentation/devicetree/bindings/bus/palmbus.yaml | 1 + .../devicetree/bindings/display/msm/qcom,mdss.yaml | 5 + Documentation/devicetree/bindings/example-schema.yaml | 2 ++ .../devicetree/bindings/gpio/x-powers,axp209-gpio.yaml | 1 + .../devicetree/bindings/hwmon/adi,ltc2992.yaml | 1 + .../bindings/interrupt-controller/arm,gic-v3.yaml | 2 ++ .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml | 1 + .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++ .../bindings/memory-controllers/arm,pl35x-smc.yaml | 1 + .../bindings/memory-controllers/exynos-srom.yaml | 1 + .../memory-controllers/nvidia,tegra124-emc.yaml| 1 + .../bindings/memory-controllers/st,stm32-fmc2-ebi.yaml | 1 + .../devicetree/bindings/mfd/mediatek,mt6370.yaml | 2 ++ .../devicetree/bindings/mmc/aspeed,sdhci.yaml | 1 + Documentation/devicetree/bindings/mtd/mtd.yaml | 1 + .../devicetree/bindings/power/supply/ti,lp8727.yaml| 1 + .../devicetree/bindings/soc/imx/fsl,imx93-src.yaml | 3 ++- .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 1 + Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 1 + .../devicetree/bindings/sound/marvell,mmp-sspa.yaml| 1 + .../devicetree/bindings/sound/qcom,wcd934x.yaml| 1 + .../devicetree/bindings/sound/samsung,odroid.yaml | 2 ++ .../devicetree/bindings/soundwire/qcom,soundwire.yaml | 1 + .../bindings/spi/allwinner,sun4i-a10-spi.yaml | 1 + .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + .../devicetree/bindings/spi/spi-controller.yaml| 1 + .../sram/allwinner,sun4i-a10-system-control.yaml | 10 +- Documentation/devicetree/bindings/sram/qcom,ocmem.yaml | 1 + .../devicetree/bindings/thermal/thermal-zones.yaml | 1 + Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 + 36 files changed, 65 insertions(+), 8 deletions(-) diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml index eec190a96225..09c319f803ba 100644 --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml @@ -144,6 +144,7 @@ patternProperties: it is stricter and always has two compatibles. type: object $ref: '/schemas/simple-bus.yaml' +unevaluatedProperties: false properties: compatible: diff --git a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml index b369b374fc4a..39e3c248f5b7 100644 --- a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml +++ b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml @@ -30,6 +30,7 @@ properties: clocks: type: object +additionalProperties: false properties: compatible: @@ -47,6 +48,7 @@ properties: reset: type: object +additionalProperties: false properties: compatible: @@ -63,6 +65,7 @@ properties: pwm: type: object +additionalProperties: false properties: compatible: @@ -76,8 +79,6 @@ properties: - compatible - "#pwm-cells" -additionalProperties: false - required: - compatible - mboxes diff --git
[PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas
Just as unevaluatedProperties or additionalProperties are required at the top level of schemas, they should (and will) also be required for child node schemas. That ensures only documented properties are present. Add unevaluatedProperties or additionalProperties as appropriate, and then add any missing properties flagged by the addition. Signed-off-by: Rob Herring --- To: Krzysztof Kozlowski To: David Airlie To: Daniel Vetter To: Bartosz Golaszewski To: Jean Delvare To: Guenter Roeck To: Thomas Gleixner To: Marc Zyngier To: Jassi Brar To: Mauro Carvalho Chehab To: Lee Jones To: Ulf Hansson To: Richard Weinberger To: Vignesh Raghavendra To: Sebastian Reichel To: Mark Brown To: "Rafael J. Wysocki" To: Daniel Lezcano To: Greg Kroah-Hartman Cc: linux-arm-ker...@lists.infradead.org Cc: dri-devel@lists.freedesktop.org Cc: linux-g...@vger.kernel.org Cc: linux-hw...@vger.kernel.org Cc: linux-me...@vger.kernel.org Cc: open...@lists.ozlabs.org Cc: linux-...@vger.kernel.org Cc: linux-...@lists.infradead.org Cc: linux...@vger.kernel.org Cc: alsa-de...@alsa-project.org Cc: linux-...@vger.kernel.org Cc: linux-...@vger.kernel.org --- .../devicetree/bindings/arm/arm,vexpress-juno.yaml | 1 + .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml | 5 +++-- .../bindings/arm/tegra/nvidia,tegra20-pmc.yaml | 4 .../bindings/bus/allwinner,sun50i-a64-de2.yaml | 1 + .../bindings/bus/allwinner,sun8i-a23-rsb.yaml | 1 + .../bus/intel,ixp4xx-expansion-bus-controller.yaml | 6 ++ Documentation/devicetree/bindings/bus/palmbus.yaml | 1 + .../devicetree/bindings/display/msm/qcom,mdss.yaml | 5 + Documentation/devicetree/bindings/example-schema.yaml | 2 ++ .../devicetree/bindings/gpio/x-powers,axp209-gpio.yaml | 1 + .../devicetree/bindings/hwmon/adi,ltc2992.yaml | 1 + .../bindings/interrupt-controller/arm,gic-v3.yaml | 2 ++ .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml | 1 + .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++ .../bindings/memory-controllers/arm,pl35x-smc.yaml | 1 + .../bindings/memory-controllers/exynos-srom.yaml | 1 + .../memory-controllers/nvidia,tegra124-emc.yaml| 1 + .../bindings/memory-controllers/st,stm32-fmc2-ebi.yaml | 1 + .../devicetree/bindings/mfd/mediatek,mt6370.yaml | 2 ++ .../devicetree/bindings/mmc/aspeed,sdhci.yaml | 1 + Documentation/devicetree/bindings/mtd/mtd.yaml | 1 + .../devicetree/bindings/power/supply/ti,lp8727.yaml| 1 + .../devicetree/bindings/soc/imx/fsl,imx93-src.yaml | 3 ++- .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 1 + Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 1 + .../devicetree/bindings/sound/marvell,mmp-sspa.yaml| 1 + .../devicetree/bindings/sound/qcom,wcd934x.yaml| 1 + .../devicetree/bindings/sound/samsung,odroid.yaml | 2 ++ .../devicetree/bindings/soundwire/qcom,soundwire.yaml | 1 + .../bindings/spi/allwinner,sun4i-a10-spi.yaml | 1 + .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + .../devicetree/bindings/spi/spi-controller.yaml| 1 + .../sram/allwinner,sun4i-a10-system-control.yaml | 10 +- Documentation/devicetree/bindings/sram/qcom,ocmem.yaml | 1 + .../devicetree/bindings/thermal/thermal-zones.yaml | 1 + Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 + 36 files changed, 65 insertions(+), 8 deletions(-) diff --git a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml index eec190a96225..09c319f803ba 100644 --- a/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml +++ b/Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml @@ -144,6 +144,7 @@ patternProperties: it is stricter and always has two compatibles. type: object $ref: '/schemas/simple-bus.yaml' +unevaluatedProperties: false properties: compatible: diff --git a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml index b369b374fc4a..39e3c248f5b7 100644 --- a/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml +++ b/Documentation/devicetree/bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml @@ -30,6 +30,7 @@ properties: clocks: type: object +additionalProperties: false properties: compatible: @@ -47,6 +48,7 @@ properties: reset: type: object +additionalProperties: false properties: compatible: @@ -63,6 +65,7 @@ properties: pwm: type: object +additionalProperties: false properties: compatible: @@ -76,8 +79,6 @@ properties: - compatible - "#pwm-cells" -additionalProperties: false - required: - compatible - mboxes diff --git