Skip to contentSkip to conversation

dt-bindings: additionalProperties clean-up

PATCH In review Rob Herring 12 participants 27 messages Mar 25, 2020 – Mar 31, 2020

Subsystems (73)
lore
Share & tools
lore

Rob Herring

author maintainer
#0 lore raw

cover dt-bindings: additionalProperties clean-up

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

This series is a bunch of fixes in patches 1-3 found by setting 'additionalProperties: false' and then patch 4 sets additionalProperties on a bunch of schema.

Rob

Rob Herring (4):
dt-bindings: iio/accel: Drop duplicate adi,adxl345/6 from
trivial-devices.yaml
dt-bindings: sram: qcom: Clean-up 'ranges' and child node names
dt-bindings: Clean-up schema errors due to missing
'addtionalProperties: false'
dt-bindings: Add missing 'additionalProperties: false'

 .../arm/altera/socfpga-clk-manager.yaml       |  2 ++
 .../amlogic/amlogic,meson-gx-ao-secure.yaml   |  2 ++
 .../bindings/arm/msm/qcom,llcc.yaml           |  2 ++
 .../devicetree/bindings/arm/renesas,prr.yaml  |  2 ++
 .../bindings/arm/samsung/exynos-chipid.yaml   |  2 ++
 .../devicetree/bindings/arm/samsung/pmu.yaml  |  2 ++
 .../arm/samsung/samsung-secure-firmware.yaml  |  2 ++
 .../bindings/arm/stm32/st,stm32-syscon.yaml   |  2 ++
 .../devicetree/bindings/clock/fsl,plldig.yaml |  5 ++++
 .../bindings/clock/imx8mn-clock.yaml          |  2 ++
 .../bindings/clock/imx8mp-clock.yaml          |  2 ++
 .../bindings/clock/milbeaut-clock.yaml        |  2 ++
 .../bindings/clock/qcom,gcc-apq8064.yaml      |  2 ++
 .../bindings/clock/qcom,gcc-ipq8074.yaml      |  2 ++
 .../bindings/clock/qcom,gcc-msm8996.yaml      |  2 ++
 .../bindings/clock/qcom,gcc-msm8998.yaml      |  2 ++
 .../bindings/clock/qcom,gcc-qcs404.yaml       |  2 ++
 .../bindings/clock/qcom,gcc-sc7180.yaml       |  2 ++
 .../bindings/clock/qcom,gcc-sm8150.yaml       |  2 ++
 .../devicetree/bindings/clock/qcom,gcc.yaml   |  2 ++
 .../devicetree/bindings/clock/qcom,mmcc.yaml  |  2 ++
 .../bindings/clock/qcom,msm8998-gpucc.yaml    |  2 ++
 .../bindings/clock/qcom,rpmhcc.yaml           |  2 ++
 .../bindings/clock/qcom,sc7180-dispcc.yaml    |  2 ++
 .../bindings/clock/qcom,sc7180-gpucc.yaml     |  2 ++
 .../bindings/clock/qcom,sc7180-videocc.yaml   |  2 ++
 .../bindings/clock/qcom,sdm845-dispcc.yaml    |  2 ++
 .../bindings/clock/qcom,sdm845-gpucc.yaml     |  2 ++
 .../bindings/clock/qcom,sdm845-videocc.yaml   |  2 ++
 .../bindings/display/amlogic,meson-vpu.yaml   |  2 ++
 .../bindings/dma/sifive,fu540-c000-pdma.yaml  |  2 ++
 .../devicetree/bindings/dsp/fsl,dsp.yaml      |  2 ++
 .../devicetree/bindings/eeprom/at24.yaml      |  2 ++
 ...ntel,ixp4xx-network-processing-engine.yaml |  3 +++
 .../bindings/gpio/brcm,xgs-iproc-gpio.yaml    |  2 ++
 .../gpio/socionext,uniphier-gpio.yaml         |  4 +++
 .../bindings/gpio/xylon,logicvc-gpio.yaml     |  2 ++
 .../bindings/gpu/arm,mali-bifrost.yaml        |  8 +++---
 .../bindings/gpu/arm,mali-midgard.yaml        |  5 ++++
 .../bindings/gpu/arm,mali-utgard.yaml         |  2 ++
 .../bindings/gpu/samsung-rotator.yaml         |  2 ++
 .../bindings/hwmon/adi,adm1177.yaml           |  2 ++
 .../bindings/hwmon/adi,ltc2947.yaml           |  2 ++
 .../bindings/hwmon/pmbus/ti,ucd90320.yaml     |  2 ++
 .../devicetree/bindings/hwmon/ti,tmp513.yaml  |  2 ++
 .../bindings/iio/accel/adi,adxl345.yaml       | 10 +++++---
 .../bindings/iio/accel/bosch,bma400.yaml      |  2 ++
 .../bindings/iio/adc/adi,ad7192.yaml          |  1 -
 .../bindings/iio/adc/adi,ad7780.yaml          |  2 ++
 .../bindings/iio/adc/avia-hx711.yaml          |  2 ++
 .../bindings/iio/adc/lltc,ltc2496.yaml        |  2 ++
 .../bindings/iio/adc/microchip,mcp3911.yaml   |  2 ++
 .../bindings/iio/adc/st,stm32-dfsdm-adc.yaml  |  2 ++
 .../iio/chemical/plantower,pms7003.yaml       |  2 ++
 .../iio/chemical/sensirion,sps30.yaml         |  2 ++
 .../bindings/iio/dac/lltc,ltc1660.yaml        |  2 ++
 .../bindings/iio/light/adux1020.yaml          |  2 ++
 .../devicetree/bindings/iio/light/bh1750.yaml |  2 ++
 .../bindings/iio/light/isl29018.yaml          |  2 ++
 .../bindings/iio/light/noa1305.yaml           |  2 ++
 .../bindings/iio/light/stk33xx.yaml           |  2 ++
 .../bindings/iio/light/tsl2583.yaml           |  2 ++
 .../bindings/iio/light/tsl2772.yaml           |  2 ++
 .../bindings/iio/light/veml6030.yaml          |  2 ++
 .../bindings/iio/pressure/asc,dlhl60d.yaml    |  2 ++
 .../bindings/iio/pressure/bmp085.yaml         |  5 ++++
 .../iio/proximity/devantech-srf04.yaml        |  2 ++
 .../bindings/iio/proximity/parallax-ping.yaml |  2 ++
 .../bindings/iio/temperature/adi,ltc2983.yaml |  2 ++
 .../bindings/input/gpio-vibrator.yaml         |  2 ++
 .../bindings/input/max77650-onkey.yaml        |  3 +++
 .../intel,ixp4xx-interrupt.yaml               |  2 ++
 .../bindings/iommu/samsung,sysmmu.yaml        |  2 ++
 .../bindings/leds/leds-max77650.yaml          |  3 +++
 .../bindings/leds/rohm,bd71828-leds.yaml      |  3 +++
 .../mailbox/amlogic,meson-gxbb-mhu.yaml       |  2 ++
 .../bindings/media/amlogic,gx-vdec.yaml       |  2 ++
 .../media/amlogic,meson-gx-ao-cec.yaml        | 11 +++++---
 .../bindings/media/renesas,ceu.yaml           |  2 ++
 .../devicetree/bindings/mfd/max77650.yaml     |  2 ++
 .../bindings/mfd/rohm,bd71828-pmic.yaml       |  5 ++++
 .../misc/intel,ixp4xx-ahb-queue-manager.yaml  |  2 ++
 .../bindings/net/ti,cpsw-switch.yaml          | 25 +++++++++++++------
 .../phy/allwinner,sun50i-h6-usb3-phy.yaml     |  2 ++
 .../phy/amlogic,meson-g12a-usb3-pcie-phy.yaml |  2 ++
 .../bindings/phy/intel,lgm-emmc-phy.yaml      |  2 ++
 .../bindings/phy/marvell,mmp3-hsic-phy.yaml   |  2 ++
 .../bindings/phy/ti,phy-j721e-wiz.yaml        |  2 ++
 .../pinctrl/aspeed,ast2400-pinctrl.yaml       |  2 ++
 .../pinctrl/aspeed,ast2500-pinctrl.yaml       |  2 ++
 .../pinctrl/aspeed,ast2600-pinctrl.yaml       |  2 ++
 .../bindings/pinctrl/st,stm32-pinctrl.yaml    |  2 ++
 .../bindings/power/amlogic,meson-ee-pwrc.yaml |  2 ++
 .../bindings/power/reset/syscon-poweroff.yaml |  2 ++
 .../bindings/power/reset/syscon-reboot.yaml   |  2 ++
 .../power/supply/max77650-charger.yaml        |  3 +++
 .../devicetree/bindings/ptp/ptp-idtcm.yaml    |  2 ++
 .../regulator/max77650-regulator.yaml         |  5 +++-
 .../bindings/reset/amlogic,meson-reset.yaml   |  2 ++
 .../reset/brcm,bcm7216-pcie-sata-rescal.yaml  |  2 ++
 .../bindings/rng/amlogic,meson-rng.yaml       |  2 ++
 .../devicetree/bindings/rng/brcm,bcm2835.yaml |  2 ++
 .../bindings/rtc/renesas,sh-rtc.yaml          |  2 ++
 .../devicetree/bindings/rtc/st,stm32-rtc.yaml |  2 ++
 .../bindings/serial/amlogic,meson-uart.yaml   |  2 ++
 .../bindings/soc/amlogic/amlogic,canvas.yaml  |  2 ++
 .../bindings/sound/adi,adau7118.yaml          |  2 ++
 .../bindings/sound/qcom,wcd934x.yaml          |  2 ++
 .../bindings/sound/renesas,fsi.yaml           |  2 ++
 .../bindings/sound/samsung,odroid.yaml        |  2 ++
 .../bindings/sound/samsung-i2s.yaml           |  2 ++
 .../devicetree/bindings/sram/qcom,ocmem.yaml  | 14 ++++++-----
 .../bindings/thermal/amlogic,thermal.yaml     |  4 +++
 .../bindings/timer/arm,arch_timer.yaml        |  2 ++
 .../bindings/timer/arm,arch_timer_mmio.yaml   |  4 +++
 .../bindings/timer/arm,global_timer.yaml      |  2 ++
 .../bindings/timer/intel,ixp4xx-timer.yaml    |  2 ++
 .../timer/samsung,exynos4210-mct.yaml         |  2 ++
 .../devicetree/bindings/trivial-devices.yaml  |  6 ++---
 119 files changed, 296 insertions(+), 29 deletions(-)

--
2.20.1
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

author maintainer
#1 lore raw

PATCH 1/4 dt-bindings: iio/accel: Drop duplicate adi,adxl345/6 from trivial-devices.yaml

The 'adi,adxl345' definition is a duplicate as there's a full binding in:
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml

The trivial-devices binding doesn't capture that 'adi,adxl346' has a fallback compatible 'adi,adxl345', so let's add it to adi,adxl345.yaml.

Cc Michael Hennerich <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
---
 .../devicetree/bindings/iio/accel/adi,adxl345.yaml     | 10 +++++++---
 Documentation/devicetree/bindings/trivial-devices.yaml |  4 ----
 2 files changed, 7 insertions(+), 7 deletions(-)
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +7 -3
index c602b6fe1c0c..d124eba1ce54 100644
@@ -17,9 +17,13 @@ description: |
 
 properties:
   compatible:
-    enum:
-      - adi,adxl345
-      - adi,adxl375
+    oneOf:
+      - items:
+          - const: adi,adxl346
+          - const: adi,adxl345
+      - enum:
+          - adi,adxl345
+          - adi,adxl375
 
   reg:
     maxItems: 1
Documentation/devicetree/bindings/trivial-devices.yaml +0 -4
index 978de7d37c66..51d1f6e43c02 100644
@@ -42,10 +42,6 @@ properties:
           - adi,adt7476
             # +/-1C TDM Extended Temp Range I.C
           - adi,adt7490
-            # Three-Axis Digital Accelerometer
-          - adi,adxl345
-            # Three-Axis Digital Accelerometer (backward-compatibility value "adi,adxl345" must be listed too)
-          - adi,adxl346
             # AMS iAQ-Core VOC Sensor
           - ams,iaq-core
             # i2c serial eeprom  (24cxx)
Signature · 1 line
2.20.1
Acked-by on v1 1/4
#2 lore raw

Re: [PATCH 1/4] dt-bindings: iio/accel: Drop duplicate adi,adxl345/6 from trivial-devices.yaml

Quoted 8 lines · Rob Herring

[External]

The 'adi,adxl345' definition is a duplicate as there's a full binding in:
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml

The trivial-devices binding doesn't capture that 'adi,adxl346' has a fallback compatible 'adi,adxl345', so let's add it to adi,adxl345.yaml.

Quoted 50 lines
Cc Michael Hennerich <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
---
 .../devicetree/bindings/iio/accel/adi,adxl345.yaml     | 10 +++++++---
 Documentation/devicetree/bindings/trivial-devices.yaml |  4 ----
 2 files changed, 7 insertions(+), 7 deletions(-)
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +7 -3
index c602b6fe1c0c..d124eba1ce54 100644
@@ -17,9 +17,13 @@ description: |
 
 properties:
   compatible:
-    enum:
-      - adi,adxl345
-      - adi,adxl375
+    oneOf:
+      - items:
+          - const: adi,adxl346
+          - const: adi,adxl345
+      - enum:
+          - adi,adxl345
+          - adi,adxl375
 
   reg:
     maxItems: 1
Documentation/devicetree/bindings/trivial-devices.yaml +0 -4
index 978de7d37c66..51d1f6e43c02 100644
@@ -42,10 +42,6 @@ properties:
           - adi,adt7476
             # +/-1C TDM Extended Temp Range I.C
           - adi,adt7490
-            # Three-Axis Digital Accelerometer
-          - adi,adxl345
-            # Three-Axis Digital Accelerometer (backward-compatibility value
 "adi,adxl345" must be listed too)
-          - adi,adxl346
             # AMS iAQ-Core VOC Sensor
           - ams,iaq-core
Quoted 1 line
             # i2c serial eeprom  (24cxx)

_______________________________________________
linux-arm-kernel mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

#3 lore raw

Re: [PATCH 1/4] dt-bindings: iio/accel: Drop duplicate adi, adxl345/6 from trivial-devices.yaml

Quoted 11 lines · Alexandru
Quoted 8 lines · Rob Herring

[External]

The 'adi,adxl345' definition is a duplicate as there's a full binding in:
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml

The trivial-devices binding doesn't capture that 'adi,adxl346' has a fallback compatible 'adi,adxl345', so let's add it to adi,adxl345.yaml.

Quoted 48 lines
Quoted 48 lines
Cc Michael Hennerich <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
---
 .../devicetree/bindings/iio/accel/adi,adxl345.yaml     | 10 +++++++---
 Documentation/devicetree/bindings/trivial-devices.yaml |  4 ----
 2 files changed, 7 insertions(+), 7 deletions(-)
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +7 -3
index c602b6fe1c0c..d124eba1ce54 100644
@@ -17,9 +17,13 @@ description: |
 
 properties:
   compatible:
-    enum:
-      - adi,adxl345
-      - adi,adxl375
+    oneOf:
+      - items:
+          - const: adi,adxl346
+          - const: adi,adxl345
+      - enum:
+          - adi,adxl345
+          - adi,adxl375
 
   reg:
     maxItems: 1
Documentation/devicetree/bindings/trivial-devices.yaml +0 -3
index 978de7d37c66..51d1f6e43c02 100644
@@ -42,10 +42,6 @@ properties:
           - adi,adt7476
             # +/-1C TDM Extended Temp Range I.C
           - adi,adt7490
-            # Three-Axis Digital Accelerometer
-          - adi,adxl345
-            # Three-Axis Digital Accelerometer (backward-compatibility value
 "adi,adxl345" must be listed too)
Quoted 8 lines
Quoted 4 lines
Documentation/devicetree/bindings/trivial-devices.yaml +0 -1
-          - adi,adxl346
             # AMS iAQ-Core VOC Sensor
           - ams,iaq-core
             # i2c serial eeprom  (24cxx)  

_______________________________________________
linux-arm-kernel mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

_______________________________________________
linux-arm-kernel mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

#4 lore raw

Re: [PATCH 1/4] dt-bindings: iio/accel: Drop duplicate adi, adxl345/6 from trivial-devices.yaml

In reply to Rob Herring

Hi Rob.

Quoted 26 lines · Rob Herring

The 'adi,adxl345' definition is a duplicate as there's a full binding in:
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml

The trivial-devices binding doesn't capture that 'adi,adxl346' has a fallback compatible 'adi,adxl345', so let's add it to adi,adxl345.yaml.

Cc Michael Hennerich <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
---
 .../devicetree/bindings/iio/accel/adi,adxl345.yaml     | 10 +++++++---
 Documentation/devicetree/bindings/trivial-devices.yaml |  4 ----
 2 files changed, 7 insertions(+), 7 deletions(-)
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +0 -0
index c602b6fe1c0c..d124eba1ce54 100644
@@ -17,9 +17,13 @@ description: |
 
 properties:
   compatible:

Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml description: | · line 26

Quoted 10 lines
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +7 -3
-    enum:
-      - adi,adxl345
-      - adi,adxl375
+    oneOf:
+      - items:
+          - const: adi,adxl346
+          - const: adi,adxl345
+      - enum:
+          - adi,adxl345
+          - adi,adxl375

I assume it is my schema understanding that is poor. But I cannot parse the above.

The mix of items, enum and const confuses me.

I guess that if I am confused then others may end in the same situation. Can we improve readability here or amybe add a comment?

Sam

Quoted 25 lines
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +0 -0
   reg:
     maxItems: 1
Documentation/devicetree/bindings/trivial-devices.yaml +0 -4
index 978de7d37c66..51d1f6e43c02 100644
@@ -42,10 +42,6 @@ properties:
           - adi,adt7476
             # +/-1C TDM Extended Temp Range I.C
           - adi,adt7490
-            # Three-Axis Digital Accelerometer
-          - adi,adxl345
-            # Three-Axis Digital Accelerometer (backward-compatibility value "adi,adxl345" must be listed too)
-          - adi,adxl346
             # AMS iAQ-Core VOC Sensor
           - ams,iaq-core
             # i2c serial eeprom  (24cxx)
Signature · 6 lines
2.20.1

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

author maintainer
#5 lore raw

Re: [PATCH 1/4] dt-bindings: iio/accel: Drop duplicate adi, adxl345/6 from trivial-devices.yaml

Quoted 44 lines · Sam Ravnborg

Hi Rob.

Quoted 26 lines · Rob Herring

The 'adi,adxl345' definition is a duplicate as there's a full binding in:
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml

The trivial-devices binding doesn't capture that 'adi,adxl346' has a fallback compatible 'adi,adxl345', so let's add it to adi,adxl345.yaml.

Cc Michael Hennerich <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
---
 .../devicetree/bindings/iio/accel/adi,adxl345.yaml     | 10 +++++++---
 Documentation/devicetree/bindings/trivial-devices.yaml |  4 ----
 2 files changed, 7 insertions(+), 7 deletions(-)
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +0 -0
index c602b6fe1c0c..d124eba1ce54 100644
@@ -17,9 +17,13 @@ description: |
 
 properties:
   compatible:

Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml description: | · line 26

Quoted 10 lines
Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml +7 -3
-    enum:
-      - adi,adxl345
-      - adi,adxl375
+    oneOf:
+      - items:
+          - const: adi,adxl346
+          - const: adi,adxl345
+      - enum:
+          - adi,adxl345
+          - adi,adxl375

I assume it is my schema understanding that is poor. But I cannot parse the above.

Quoted 1 line

The mix of items, enum and const confuses me.

compatible can be one of 3 possibilities:
"adi,adxl346", "adi,adxl345"
"adi,adxl345"
"adi,adxl375"

For a single entry, 'items' can be omitted.

Quoted 2 lines

I guess that if I am confused then others may end in the same situation. Can we improve readability here or amybe add a comment?

example-schema.yaml explains this to some extent. I'd rather improve that.

Rob
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

author maintainer
#6 lore raw

PATCH 2/4 dt-bindings: sram: qcom: Clean-up 'ranges' and child node names

In reply to Rob Herring

The regex for child nodes doesn't match the example. This wasn't flagged with 'additionalProperties: false' missing. The child node schema was also incorrect with 'ranges' property as it applies to child nodes and should be moved up to the parent node.

Fixes 957fd69d396b ("dt-bindings: soc: qcom: add On Chip MEMory (OCMEM) bindings")
Cc Brian Masney <[email protected]>
Cc Bjorn Andersson <[email protected]>
---
 .../devicetree/bindings/sram/qcom,ocmem.yaml         | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
Documentation/devicetree/bindings/sram/qcom,ocmem.yaml +6 -6
index 222990f9923c..469cec133647 100644
@@ -43,6 +43,9 @@ properties:
   '#size-cells':
     const: 1
 
+  ranges:
+    maxItems: 1
+
 required:
   - compatible
   - reg
@@ -51,9 +54,10 @@ required:
   - clock-names
   - '#address-cells'
   - '#size-cells'
+  - ranges
 
 patternProperties:
-  "^.+-sram$":
+  "-sram@[0-9a-f]+$":
     type: object
     description: A region of reserved memory.
 
@@ -61,12 +65,8 @@ patternProperties:
       reg:
         maxItems: 1
 
-      ranges:
-        maxItems: 1
-
     required:
       - reg
-      - ranges
 
 examples:
   - |
@@ -88,9 +88,9 @@ examples:
 
         #address-cells = <1>;
         #size-cells = <1>;
+        ranges = <0 0xfec00000 0x100000>;
 
         gmu-sram@0 {
                 reg = <0x0 0x100000>;
-                ranges = <0 0 0xfec00000 0x100000>;
         };
       };
Signature · 6 lines
2.20.1

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Reviewed-by on v1 2/4
#7 lore raw

Re: [PATCH 2/4] dt-bindings: sram: qcom: Clean-up 'ranges' and child node names

Quoted 10 lines · Rob Herring

The regex for child nodes doesn't match the example. This wasn't flagged with 'additionalProperties: false' missing. The child node schema was also incorrect with 'ranges' property as it applies to child nodes and should be moved up to the parent node.

Fixes 957fd69d396b ("dt-bindings: soc: qcom: add On Chip MEMory (OCMEM) bindings")
Cc Brian Masney <[email protected]>
Cc Bjorn Andersson <[email protected]>
author maintainer
#8 lore raw

PATCH 3/4 dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'

In reply to Rob Herring

Numerous schemas are missing 'additionalProperties: false' statements which ensures a binding doesn't have any extra undocumented properties or child nodes. Fixing this reveals various missing properties, so let's fix all those occurrences.

Cc Stephen Boyd <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Bartosz Golaszewski <[email protected]>
Cc Masahiro Yamada <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>
Cc Kevin Hilman <[email protected]>
Cc Lee Jones <[email protected]>
Cc "David S. Miller" <[email protected]>
Cc Liam Girdwood <[email protected]>
Cc Mark Brown <[email protected]>
Cc Guillaume La Roque <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Thomas Gleixner <[email protected]>
---
 .../devicetree/bindings/clock/fsl,plldig.yaml |  3 +++
 .../gpio/socionext,uniphier-gpio.yaml         |  2 ++
 .../bindings/gpu/arm,mali-bifrost.yaml        |  6 ++---
 .../bindings/gpu/arm,mali-midgard.yaml        |  3 +++
 .../bindings/iio/adc/adi,ad7192.yaml          |  1 -
 .../bindings/iio/pressure/bmp085.yaml         |  3 +++
 .../media/amlogic,meson-gx-ao-cec.yaml        |  9 +++++---
 .../bindings/mfd/rohm,bd71828-pmic.yaml       |  3 +++
 .../bindings/net/ti,cpsw-switch.yaml          | 23 ++++++++++++-------
 .../regulator/max77650-regulator.yaml         |  2 +-
 .../bindings/thermal/amlogic,thermal.yaml     |  2 ++
 .../bindings/timer/arm,arch_timer_mmio.yaml   |  2 ++
 12 files changed, 43 insertions(+), 16 deletions(-)
Documentation/devicetree/bindings/clock/fsl,plldig.yaml +3 -0
index c8350030b374..d1c040228cf7 100644
@@ -21,6 +21,9 @@ properties:
   reg:
     maxItems: 1
 
+  clocks:
+    maxItems: 1
+
   '#clock-cells':
     const: 0
 
Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml +2 -0
index 580a39e09d39..c58ff9a94f45 100644
@@ -41,6 +41,8 @@ properties:
     minimum: 0
     maximum: 512
 
+  gpio-ranges: true
+
   gpio-ranges-group-names:
     $ref: /schemas/types.yaml#/definitions/string-array
 
Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +3 -3
index e8b99adcb1bd..05fd9a404ff7 100644
@@ -43,6 +43,9 @@ properties:
 
   operating-points-v2: true
 
+  resets:
+    maxItems: 2
+
 required:
   - compatible
   - reg
@@ -57,9 +60,6 @@ allOf:
           contains:
             const: amlogic,meson-g12a-mali
     then:
-      properties:
-        resets:
-          minItems: 2
       required:
         - resets
 
Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +3 -0
index 8d966f3ff3db..6819cde050df 100644
@@ -75,6 +75,9 @@ properties:
 
   mali-supply: true
 
+  power-domains:
+    maxItems: 1
+
   resets:
     minItems: 1
     maxItems: 2
Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml +0 -1
index 84d25bd39488..d0913034b1d8 100644
@@ -106,7 +106,6 @@ examples:
         spi-cpha;
         clocks = <&ad7192_mclk>;
         clock-names = "mclk";
-        #interrupt-cells = <2>;
         interrupts = <25 0x2>;
         interrupt-parent = <&gpio>;
         dvdd-supply = <&dvdd>;
Documentation/devicetree/bindings/iio/pressure/bmp085.yaml +3 -0
index 519137e5c170..5d4aec0e0d24 100644
@@ -25,6 +25,9 @@ properties:
       - bosch,bmp280
       - bosch,bme280
 
+  reg:
+    maxItems: 1
+
   vddd-supply:
     description:
       digital voltage regulator (see regulator/regulator.txt)
Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml +6 -3
index 41197578f19a..e8ce37fcbfec 100644
@@ -24,6 +24,12 @@ properties:
   reg:
     maxItems: 1
 
+  clocks:
+    maxItems: 1
+
+  clock-names:
+    maxItems: 1
+
   interrupts:
     maxItems: 1
 
@@ -47,7 +53,6 @@ allOf:
             - description: AO-CEC clock
 
         clock-names:
-          maxItems: 1
           items:
             - const: core
 
@@ -66,7 +71,6 @@ allOf:
             - description: AO-CEC clock generator source
 
         clock-names:
-          maxItems: 1
           items:
             - const: oscin
 
@@ -88,4 +92,3 @@ examples:
         clock-names = "core";
         hdmi-phandle = <&hdmi_tx>;
     };
-
Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml +3 -0
index 4fbb9e734284..38dc4f8b0ceb 100644
@@ -41,6 +41,9 @@ properties:
   "#clock-cells":
     const: 0
 
+  clock-output-names:
+    const: bd71828-32k-out
+
   rohm,charger-sense-resistor-ohms:
     minimum: 10000000
     maximum: 50000000
Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +15 -8
index ac8c76369a86..b9e9696da5be 100644
@@ -37,6 +37,12 @@ properties:
     description:
        The physical base address and size of full the CPSW module IO range
 
+  '#address-cells':
+    const: 1
+
+  '#size-cells':
+    const: 1
+
   ranges: true
 
   clocks:
@@ -111,13 +117,6 @@ properties:
             - reg
             - phys
 
-  mdio:
-    type: object
-    allOf:
-      - $ref: "ti,davinci-mdio.yaml#"
-    description:
-      CPSW MDIO bus.
-
   cpts:
     type: object
     description:
@@ -148,6 +147,15 @@ properties:
       - clocks
       - clock-names
 
+patternProperties:
+  "^mdio@":
+    type: object
+    allOf:
+      - $ref: "ti,davinci-mdio.yaml#"
+    description:
+      CPSW MDIO bus.
+
+
 required:
   - compatible
   - reg
@@ -174,7 +182,6 @@ examples:
         #address-cells = <1>;
         #size-cells = <1>;
         syscon = <&scm_conf>;
-        inctrl-names = "default", "sleep";
 
         interrupts = <GIC_SPI 334 IRQ_TYPE_LEVEL_HIGH>,
                      <GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH>,
Documentation/devicetree/bindings/regulator/max77650-regulator.yaml +1 -1
index 7d724159f890..50690487edc8 100644
@@ -24,7 +24,7 @@ properties:
     const: maxim,max77650-regulator
 
 patternProperties:
-  "^regulator@[0-3]$":
+  "^regulator-(ldo|sbb[0-2])$":
     $ref: "regulator.yaml#"
 
 required:
Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2 -0
index f761681e4c0d..93fe7b10a82e 100644
@@ -32,6 +32,8 @@ properties:
     description: phandle to the ao-secure syscon
     $ref: '/schemas/types.yaml#/definitions/phandle'
 
+  '#thermal-sensor-cells':
+    const: 0
 
 required:
   - compatible
Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2 -0
index 102f319833d9..f7ef6646bade 100644
@@ -32,6 +32,8 @@ properties:
   '#size-cells':
     const: 1
 
+  ranges: true
+
   clock-frequency:
     description: The frequency of the main counter, in Hz. Should be present
       only where necessary to work around broken firmware which does not configure
Signature · 6 lines
2.20.1

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Acked-by on v1 3/4
#9 lore raw

Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'

Quoted 5 lines · Rob Herring

Numerous schemas are missing 'additionalProperties: false' statements which ensures a binding doesn't have any extra undocumented properties or child nodes. Fixing this reveals various missing properties, so let's fix all those occurrences.

For 'bindings/iio/adc/adi,ad7192.yaml'

Quoted 296 lines
Cc Stephen Boyd <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Bartosz Golaszewski <[email protected]>
Cc Masahiro Yamada <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>
Cc Kevin Hilman <[email protected]>
Cc Lee Jones <[email protected]>
Cc "David S. Miller" <[email protected]>
Cc Liam Girdwood <[email protected]>
Cc Mark Brown <[email protected]>
Cc Guillaume La Roque <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Thomas Gleixner <[email protected]>
---
 .../devicetree/bindings/clock/fsl,plldig.yaml |  3 +++
 .../gpio/socionext,uniphier-gpio.yaml         |  2 ++
 .../bindings/gpu/arm,mali-bifrost.yaml        |  6 ++---
 .../bindings/gpu/arm,mali-midgard.yaml        |  3 +++
 .../bindings/iio/adc/adi,ad7192.yaml          |  1 -
 .../bindings/iio/pressure/bmp085.yaml         |  3 +++
 .../media/amlogic,meson-gx-ao-cec.yaml        |  9 +++++---
 .../bindings/mfd/rohm,bd71828-pmic.yaml       |  3 +++
 .../bindings/net/ti,cpsw-switch.yaml          | 23 ++++++++++++-------
 .../regulator/max77650-regulator.yaml         |  2 +-
 .../bindings/thermal/amlogic,thermal.yaml     |  2 ++
 .../bindings/timer/arm,arch_timer_mmio.yaml   |  2 ++
 12 files changed, 43 insertions(+), 16 deletions(-)
Documentation/devicetree/bindings/clock/fsl,plldig.yaml +3 -0
index c8350030b374..d1c040228cf7 100644
@@ -21,6 +21,9 @@ properties:
   reg:
     maxItems: 1
 
+  clocks:
+    maxItems: 1
+
   '#clock-cells':
     const: 0
 
(quoted diff) +0 -0

gpio.yaml b/Documentation/devicetree/bindings/gpio/socionext,uniphier- gpio.yaml
index 580a39e09d39..c58ff9a94f45 100644

Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml +2 -0
@@ -41,6 +41,8 @@ properties:
     minimum: 0
     maximum: 512
 
+  gpio-ranges: true
+
   gpio-ranges-group-names:
     $ref: /schemas/types.yaml#/definitions/string-array
 
Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +3 -3
index e8b99adcb1bd..05fd9a404ff7 100644
@@ -43,6 +43,9 @@ properties:
 
   operating-points-v2: true
 
+  resets:
+    maxItems: 2
+
 required:
   - compatible
   - reg
@@ -57,9 +60,6 @@ allOf:
           contains:
             const: amlogic,meson-g12a-mali
     then:
-      properties:
-        resets:
-          minItems: 2
       required:
         - resets
 
Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +3 -0
index 8d966f3ff3db..6819cde050df 100644
@@ -75,6 +75,9 @@ properties:
 
   mali-supply: true
 
+  power-domains:
+    maxItems: 1
+
   resets:
     minItems: 1
     maxItems: 2
Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml +0 -1
index 84d25bd39488..d0913034b1d8 100644
@@ -106,7 +106,6 @@ examples:
         spi-cpha;
         clocks = <&ad7192_mclk>;
         clock-names = "mclk";
-        #interrupt-cells = <2>;
         interrupts = <25 0x2>;
         interrupt-parent = <&gpio>;
         dvdd-supply = <&dvdd>;
Documentation/devicetree/bindings/iio/pressure/bmp085.yaml +3 -0
index 519137e5c170..5d4aec0e0d24 100644
@@ -25,6 +25,9 @@ properties:
       - bosch,bmp280
       - bosch,bme280
 
+  reg:
+    maxItems: 1
+
   vddd-supply:
     description:
       digital voltage regulator (see regulator/regulator.txt)
(quoted diff) +0 -0

cec.yaml b/Documentation/devicetree/bindings/media/amlogic,meson-gx-ao- cec.yaml
index 41197578f19a..e8ce37fcbfec 100644

Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml +6 -3
@@ -24,6 +24,12 @@ properties:
   reg:
     maxItems: 1
 
+  clocks:
+    maxItems: 1
+
+  clock-names:
+    maxItems: 1
+
   interrupts:
     maxItems: 1
 
@@ -47,7 +53,6 @@ allOf:
             - description: AO-CEC clock
 
         clock-names:
-          maxItems: 1
           items:
             - const: core
 
@@ -66,7 +71,6 @@ allOf:
             - description: AO-CEC clock generator source
 
         clock-names:
-          maxItems: 1
           items:
             - const: oscin
 
@@ -88,4 +92,3 @@ examples:
         clock-names = "core";
         hdmi-phandle = <&hdmi_tx>;
     };
-
Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml +3 -0
index 4fbb9e734284..38dc4f8b0ceb 100644
@@ -41,6 +41,9 @@ properties:
   "#clock-cells":
     const: 0
 
+  clock-output-names:
+    const: bd71828-32k-out
+
   rohm,charger-sense-resistor-ohms:
     minimum: 10000000
     maximum: 50000000
Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +15 -8
index ac8c76369a86..b9e9696da5be 100644
@@ -37,6 +37,12 @@ properties:
     description:
        The physical base address and size of full the CPSW module IO range
 
+  '#address-cells':
+    const: 1
+
+  '#size-cells':
+    const: 1
+
   ranges: true
 
   clocks:
@@ -111,13 +117,6 @@ properties:
             - reg
             - phys
 
-  mdio:
-    type: object
-    allOf:
-      - $ref: "ti,davinci-mdio.yaml#"
-    description:
-      CPSW MDIO bus.
-
   cpts:
     type: object
     description:
@@ -148,6 +147,15 @@ properties:
       - clocks
       - clock-names
 
+patternProperties:
+  "^mdio@":
+    type: object
+    allOf:
+      - $ref: "ti,davinci-mdio.yaml#"
+    description:
+      CPSW MDIO bus.
+
+
 required:
   - compatible
   - reg
@@ -174,7 +182,6 @@ examples:
         #address-cells = <1>;
         #size-cells = <1>;
         syscon = <&scm_conf>;
-        inctrl-names = "default", "sleep";
 
         interrupts = <GIC_SPI 334 IRQ_TYPE_LEVEL_HIGH>,
                      <GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH>,
(quoted diff) +0 -0

regulator.yaml b/Documentation/devicetree/bindings/regulator/max77650- regulator.yaml
index 7d724159f890..50690487edc8 100644

Documentation/devicetree/bindings/regulator/max77650-regulator.yaml +1 -1
@@ -24,7 +24,7 @@ properties:
     const: maxim,max77650-regulator
 
 patternProperties:
-  "^regulator@[0-3]$":
+  "^regulator-(ldo|sbb[0-2])$":
     $ref: "regulator.yaml#"
 
 required:
Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2 -0
index f761681e4c0d..93fe7b10a82e 100644
@@ -32,6 +32,8 @@ properties:
     description: phandle to the ao-secure syscon
     $ref: '/schemas/types.yaml#/definitions/phandle'
 
+  '#thermal-sensor-cells':
+    const: 0
 
 required:
   - compatible
Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2 -0
index 102f319833d9..f7ef6646bade 100644
@@ -32,6 +32,8 @@ properties:
   '#size-cells':
     const: 1
 
+  ranges: true
+
   clock-frequency:
     description: The frequency of the main counter, in Hz. Should be present
       only where necessary to work around broken firmware which does not
Quoted 1 line

configure

_______________________________________________
linux-arm-kernel mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Reviewed-by on v1 3/4
#10 lore raw

Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'

In reply to Rob Herring
Quoted 34 lines · Rob Herring

Numerous schemas are missing 'additionalProperties: false' statements which ensures a binding doesn't have any extra undocumented properties or child nodes. Fixing this reveals various missing properties, so let's fix all those occurrences.

Cc Stephen Boyd <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Bartosz Golaszewski <[email protected]>
Cc Masahiro Yamada <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>
Cc Kevin Hilman <[email protected]>
Cc Lee Jones <[email protected]>
Cc "David S. Miller" <[email protected]>
Cc Liam Girdwood <[email protected]>
Cc Mark Brown <[email protected]>
Cc Guillaume La Roque <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Thomas Gleixner <[email protected]>
Quoted 15 lines
---
 .../devicetree/bindings/clock/fsl,plldig.yaml |  3 +++
 .../gpio/socionext,uniphier-gpio.yaml         |  2 ++
 .../bindings/gpu/arm,mali-bifrost.yaml        |  6 ++---
 .../bindings/gpu/arm,mali-midgard.yaml        |  3 +++
 .../bindings/iio/adc/adi,ad7192.yaml          |  1 -
 .../bindings/iio/pressure/bmp085.yaml         |  3 +++
 .../media/amlogic,meson-gx-ao-cec.yaml        |  9 +++++---
 .../bindings/mfd/rohm,bd71828-pmic.yaml       |  3 +++
 .../bindings/net/ti,cpsw-switch.yaml          | 23 ++++++++++++-------
 .../regulator/max77650-regulator.yaml         |  2 +-
 .../bindings/thermal/amlogic,thermal.yaml     |  2 ++
 .../bindings/timer/arm,arch_timer_mmio.yaml   |  2 ++
 12 files changed, 43 insertions(+), 16 deletions(-)

For:

  .../bindings/gpu/arm,mali-bifrost.yaml        |  6 ++---
  .../bindings/gpu/arm,mali-midgard.yaml        |  3 +++
  .../media/amlogic,meson-gx-ao-cec.yaml        |  9 +++++---
  .../bindings/thermal/amlogic,thermal.yaml     |  2 ++
Acked-by on v1 3/4
#11 lore raw

Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'

In reply to Rob Herring
Quoted 26 lines · On Wed, 25 Mar 2020 16:05:40 -0600 Rob Herring <[email protected]> wrote:

Numerous schemas are missing 'additionalProperties: false' statements which ensures a binding doesn't have any extra undocumented properties or child nodes. Fixing this reveals various missing properties, so let's fix all those occurrences.

Cc Stephen Boyd <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Bartosz Golaszewski <[email protected]>
Cc Masahiro Yamada <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>
Cc Kevin Hilman <[email protected]>
Cc Lee Jones <[email protected]>
Cc "David S. Miller" <[email protected]>
Cc Liam Girdwood <[email protected]>
Cc Mark Brown <[email protected]>
Cc Guillaume La Roque <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Thomas Gleixner <[email protected]>
Quoted 8 lines
Quoted 252 lines
---
 .../devicetree/bindings/clock/fsl,plldig.yaml |  3 +++
 .../gpio/socionext,uniphier-gpio.yaml         |  2 ++
 .../bindings/gpu/arm,mali-bifrost.yaml        |  6 ++---
 .../bindings/gpu/arm,mali-midgard.yaml        |  3 +++
 .../bindings/iio/adc/adi,ad7192.yaml          |  1 -
 .../bindings/iio/pressure/bmp085.yaml         |  3 +++
 .../media/amlogic,meson-gx-ao-cec.yaml        |  9 +++++---
 .../bindings/mfd/rohm,bd71828-pmic.yaml       |  3 +++
 .../bindings/net/ti,cpsw-switch.yaml          | 23 ++++++++++++-------
 .../regulator/max77650-regulator.yaml         |  2 +-
 .../bindings/thermal/amlogic,thermal.yaml     |  2 ++
 .../bindings/timer/arm,arch_timer_mmio.yaml   |  2 ++
 12 files changed, 43 insertions(+), 16 deletions(-)
Documentation/devicetree/bindings/clock/fsl,plldig.yaml +3 -0
index c8350030b374..d1c040228cf7 100644
@@ -21,6 +21,9 @@ properties:
   reg:
     maxItems: 1
 
+  clocks:
+    maxItems: 1
+
   '#clock-cells':
     const: 0
 
Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml +2 -0
index 580a39e09d39..c58ff9a94f45 100644
@@ -41,6 +41,8 @@ properties:
     minimum: 0
     maximum: 512
 
+  gpio-ranges: true
+
   gpio-ranges-group-names:
     $ref: /schemas/types.yaml#/definitions/string-array
 
Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +3 -3
index e8b99adcb1bd..05fd9a404ff7 100644
@@ -43,6 +43,9 @@ properties:
 
   operating-points-v2: true
 
+  resets:
+    maxItems: 2
+
 required:
   - compatible
   - reg
@@ -57,9 +60,6 @@ allOf:
           contains:
             const: amlogic,meson-g12a-mali
     then:
-      properties:
-        resets:
-          minItems: 2
       required:
         - resets
 
Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +3 -0
index 8d966f3ff3db..6819cde050df 100644
@@ -75,6 +75,9 @@ properties:
 
   mali-supply: true
 
+  power-domains:
+    maxItems: 1
+
   resets:
     minItems: 1
     maxItems: 2
Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml +0 -1
index 84d25bd39488..d0913034b1d8 100644
@@ -106,7 +106,6 @@ examples:
         spi-cpha;
         clocks = <&ad7192_mclk>;
         clock-names = "mclk";
-        #interrupt-cells = <2>;
         interrupts = <25 0x2>;
         interrupt-parent = <&gpio>;
         dvdd-supply = <&dvdd>;
Documentation/devicetree/bindings/iio/pressure/bmp085.yaml +3 -0
index 519137e5c170..5d4aec0e0d24 100644
@@ -25,6 +25,9 @@ properties:
       - bosch,bmp280
       - bosch,bme280
 
+  reg:
+    maxItems: 1
+
   vddd-supply:
     description:
       digital voltage regulator (see regulator/regulator.txt)
Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml +6 -3
index 41197578f19a..e8ce37fcbfec 100644
@@ -24,6 +24,12 @@ properties:
   reg:
     maxItems: 1
 
+  clocks:
+    maxItems: 1
+
+  clock-names:
+    maxItems: 1
+
   interrupts:
     maxItems: 1
 
@@ -47,7 +53,6 @@ allOf:
             - description: AO-CEC clock
 
         clock-names:
-          maxItems: 1
           items:
             - const: core
 
@@ -66,7 +71,6 @@ allOf:
             - description: AO-CEC clock generator source
 
         clock-names:
-          maxItems: 1
           items:
             - const: oscin
 
@@ -88,4 +92,3 @@ examples:
         clock-names = "core";
         hdmi-phandle = <&hdmi_tx>;
     };
-
Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml +3 -0
index 4fbb9e734284..38dc4f8b0ceb 100644
@@ -41,6 +41,9 @@ properties:
   "#clock-cells":
     const: 0
 
+  clock-output-names:
+    const: bd71828-32k-out
+
   rohm,charger-sense-resistor-ohms:
     minimum: 10000000
     maximum: 50000000
Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +15 -8
index ac8c76369a86..b9e9696da5be 100644
@@ -37,6 +37,12 @@ properties:
     description:
        The physical base address and size of full the CPSW module IO range
 
+  '#address-cells':
+    const: 1
+
+  '#size-cells':
+    const: 1
+
   ranges: true
 
   clocks:
@@ -111,13 +117,6 @@ properties:
             - reg
             - phys
 
-  mdio:
-    type: object
-    allOf:
-      - $ref: "ti,davinci-mdio.yaml#"
-    description:
-      CPSW MDIO bus.
-
   cpts:
     type: object
     description:
@@ -148,6 +147,15 @@ properties:
       - clocks
       - clock-names
 
+patternProperties:
+  "^mdio@":
+    type: object
+    allOf:
+      - $ref: "ti,davinci-mdio.yaml#"
+    description:
+      CPSW MDIO bus.
+
+
 required:
   - compatible
   - reg
@@ -174,7 +182,6 @@ examples:
         #address-cells = <1>;
         #size-cells = <1>;
         syscon = <&scm_conf>;
-        inctrl-names = "default", "sleep";
 
         interrupts = <GIC_SPI 334 IRQ_TYPE_LEVEL_HIGH>,
                      <GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH>,
Documentation/devicetree/bindings/regulator/max77650-regulator.yaml +1 -1
index 7d724159f890..50690487edc8 100644
@@ -24,7 +24,7 @@ properties:
     const: maxim,max77650-regulator
 
 patternProperties:
-  "^regulator@[0-3]$":
+  "^regulator-(ldo|sbb[0-2])$":
     $ref: "regulator.yaml#"
 
 required:
Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2 -0
index f761681e4c0d..93fe7b10a82e 100644
@@ -32,6 +32,8 @@ properties:
     description: phandle to the ao-secure syscon
     $ref: '/schemas/types.yaml#/definitions/phandle'
 
+  '#thermal-sensor-cells':
+    const: 0
 
 required:
   - compatible
Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2 -0
index 102f319833d9..f7ef6646bade 100644
@@ -32,6 +32,8 @@ properties:
   '#size-cells':
     const: 1
 
+  ranges: true
+
   clock-frequency:
     description: The frequency of the main counter, in Hz. Should be present
       only where necessary to work around broken firmware which does not configure
maintainer Acked-by on v1 3/4
#12 lore raw

Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'

In reply to Rob Herring

Quoting Rob Herring (2020-03-25 15:05:40)

Quoted 34 lines

Numerous schemas are missing 'additionalProperties: false' statements which ensures a binding doesn't have any extra undocumented properties or child nodes. Fixing this reveals various missing properties, so let's fix all those occurrences.

Cc Stephen Boyd <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Bartosz Golaszewski <[email protected]>
Cc Masahiro Yamada <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>
Cc Kevin Hilman <[email protected]>
Cc Lee Jones <[email protected]>
Cc "David S. Miller" <[email protected]>
Cc Liam Girdwood <[email protected]>
Cc Mark Brown <[email protected]>
Cc Guillaume La Roque <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Thomas Gleixner <[email protected]>
Quoted 3 lines
---
 .../devicetree/bindings/clock/fsl,plldig.yaml |  3 +++

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Acked-by on v1 3/4
#13 lore raw

Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'

In reply to Rob Herring
Quoted 43 lines · Rob Herring

Numerous schemas are missing 'additionalProperties: false' statements which ensures a binding doesn't have any extra undocumented properties or child nodes. Fixing this reveals various missing properties, so let's fix all those occurrences.

Cc Stephen Boyd <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Bartosz Golaszewski <[email protected]>
Cc Masahiro Yamada <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>
Cc Kevin Hilman <[email protected]>
Cc Lee Jones <[email protected]>
Cc "David S. Miller" <[email protected]>
Cc Liam Girdwood <[email protected]>
Cc Mark Brown <[email protected]>
Cc Guillaume La Roque <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Thomas Gleixner <[email protected]>
---
 .../devicetree/bindings/clock/fsl,plldig.yaml |  3 +++
 .../gpio/socionext,uniphier-gpio.yaml         |  2 ++
 .../bindings/gpu/arm,mali-bifrost.yaml        |  6 ++---
 .../bindings/gpu/arm,mali-midgard.yaml        |  3 +++
 .../bindings/iio/adc/adi,ad7192.yaml          |  1 -
 .../bindings/iio/pressure/bmp085.yaml         |  3 +++
 .../media/amlogic,meson-gx-ao-cec.yaml        |  9 +++++---
Quoted 1 line
 .../bindings/mfd/rohm,bd71828-pmic.yaml       |  3 +++
Signature · 4 lines
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Reviewed-by on v1 3/4
#14 lore raw

Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'

In reply to Rob Herring
Quoted 26 lines · Rob Herring

Numerous schemas are missing 'additionalProperties: false' statements which ensures a binding doesn't have any extra undocumented properties or child nodes. Fixing this reveals various missing properties, so let's fix all those occurrences.

Cc Stephen Boyd <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Bartosz Golaszewski <[email protected]>
Cc Masahiro Yamada <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>
Cc Kevin Hilman <[email protected]>
Cc Lee Jones <[email protected]>
Cc "David S. Miller" <[email protected]>
Cc Liam Girdwood <[email protected]>
Cc Mark Brown <[email protected]>
Cc Guillaume La Roque <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Thomas Gleixner <[email protected]>
Quoted 8 lines

Sorry for errors caused by me,

Yours,
Linus Walleij

Acked-by on v1 3/4
#15 lore raw

Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'

In reply to Rob Herring
Quoted 35 lines · Rob Herring

Numerous schemas are missing 'additionalProperties: false' statements which ensures a binding doesn't have any extra undocumented properties or child nodes. Fixing this reveals various missing properties, so let's fix all those occurrences.

Cc Stephen Boyd <[email protected]>
Cc Linus Walleij <[email protected]>
Cc Bartosz Golaszewski <[email protected]>
Cc Masahiro Yamada <[email protected]>
Cc Jonathan Cameron <[email protected]>
Cc Hartmut Knaack <[email protected]>
Cc Lars-Peter Clausen <[email protected]>
Cc Peter Meerwald-Stadler <[email protected]>
Cc Neil Armstrong <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>
Cc Kevin Hilman <[email protected]>
Cc Lee Jones <[email protected]>
Cc "David S. Miller" <[email protected]>
Cc Liam Girdwood <[email protected]>
Cc Mark Brown <[email protected]>
Cc Guillaume La Roque <[email protected]>
Cc Zhang Rui <[email protected]>
Cc Daniel Lezcano <[email protected]>
Cc Thomas Gleixner <[email protected]>
Quoted 5 lines

---

 .../gpio/socionext,uniphier-gpio.yaml         |  2 ++

You may have already queue this up, but just in case.

Signature · 2 lines
Best Regards
Masahiro Yamada
author maintainer
#16 lore raw

PATCH 4/4 dt-bindings: Add missing 'additionalProperties: false'

In reply to Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

---
 .../devicetree/bindings/arm/altera/socfpga-clk-manager.yaml    | 2 ++
 .../bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/renesas,prr.yaml         | 2 ++
 .../devicetree/bindings/arm/samsung/exynos-chipid.yaml         | 2 ++
 Documentation/devicetree/bindings/arm/samsung/pmu.yaml         | 2 ++
 .../bindings/arm/samsung/samsung-secure-firmware.yaml          | 2 ++
 .../devicetree/bindings/arm/stm32/st,stm32-syscon.yaml         | 2 ++
 Documentation/devicetree/bindings/clock/fsl,plldig.yaml        | 2 ++
 Documentation/devicetree/bindings/clock/imx8mn-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/imx8mp-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/milbeaut-clock.yaml    | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,mmcc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,msm8998-gpucc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml       | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-videocc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-videocc.yaml         | 2 ++
 .../devicetree/bindings/display/amlogic,meson-vpu.yaml         | 2 ++
 .../devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml        | 2 ++
 Documentation/devicetree/bindings/dsp/fsl,dsp.yaml             | 2 ++
 Documentation/devicetree/bindings/eeprom/at24.yaml             | 2 ++
 .../firmware/intel,ixp4xx-network-processing-engine.yaml       | 3 +++
 .../devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml          | 2 ++
 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++
 Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml     | 2 ++
 Documentation/devicetree/bindings/gpu/samsung-rotator.yaml     | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml | 2 ++
 Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml         | 2 ++
 Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml  | 2 ++
 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml    | 2 ++
 .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml         | 2 ++
 .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml        | 2 ++
 .../devicetree/bindings/iio/chemical/plantower,pms7003.yaml    | 2 ++
 .../devicetree/bindings/iio/chemical/sensirion,sps30.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml    | 2 ++
 Documentation/devicetree/bindings/iio/light/adux1020.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/bh1750.yaml        | 2 ++
 Documentation/devicetree/bindings/iio/light/isl29018.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/noa1305.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/stk33xx.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2583.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2772.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/veml6030.yaml      | 2 ++
 .../devicetree/bindings/iio/pressure/asc,dlhl60d.yaml          | 2 ++
 Documentation/devicetree/bindings/iio/pressure/bmp085.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/devantech-srf04.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/parallax-ping.yaml       | 2 ++
 .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml       | 2 ++
 Documentation/devicetree/bindings/input/gpio-vibrator.yaml     | 2 ++
 Documentation/devicetree/bindings/input/max77650-onkey.yaml    | 3 +++
 .../bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml  | 2 ++
 Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml    | 2 ++
 Documentation/devicetree/bindings/leds/leds-max77650.yaml      | 3 +++
 Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml  | 3 +++
 .../devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml    | 2 ++
 Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml   | 2 ++
 .../devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml     | 2 ++
 Documentation/devicetree/bindings/media/renesas,ceu.yaml       | 2 ++
 Documentation/devicetree/bindings/mfd/max77650.yaml            | 2 ++
 Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml   | 2 ++
 .../bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml          | 2 ++
 Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml      | 2 ++
 .../devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml  | 2 ++
 .../bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml  | 2 ++
 .../devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml          | 2 ++
 .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-poweroff.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-reboot.yaml         | 2 ++
 .../devicetree/bindings/power/supply/max77650-charger.yaml     | 3 +++
 Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml           | 2 ++
 .../devicetree/bindings/regulator/max77650-regulator.yaml      | 3 +++
 .../devicetree/bindings/reset/amlogic,meson-reset.yaml         | 2 ++
 .../bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml          | 2 ++
 Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml   | 2 ++
 Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml        | 2 ++
 Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml      | 2 ++
 Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml        | 2 ++
 .../devicetree/bindings/serial/amlogic,meson-uart.yaml         | 2 ++
 .../devicetree/bindings/soc/amlogic/amlogic,canvas.yaml        | 2 ++
 Documentation/devicetree/bindings/sound/adi,adau7118.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/renesas,fsi.yaml       | 2 ++
 Documentation/devicetree/bindings/sound/samsung,odroid.yaml    | 2 ++
 Documentation/devicetree/bindings/sound/samsung-i2s.yaml       | 2 ++
 Documentation/devicetree/bindings/sram/qcom,ocmem.yaml         | 2 ++
 Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml | 2 ++
 Documentation/devicetree/bindings/timer/arm,arch_timer.yaml    | 2 ++
 .../devicetree/bindings/timer/arm,arch_timer_mmio.yaml         | 2 ++
 Documentation/devicetree/bindings/timer/arm,global_timer.yaml  | 2 ++
 .../devicetree/bindings/timer/intel,ixp4xx-timer.yaml          | 2 ++
 .../devicetree/bindings/timer/samsung,exynos4210-mct.yaml      | 2 ++
 Documentation/devicetree/bindings/trivial-devices.yaml         | 2 ++
 117 files changed, 240 insertions(+)
Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml +2 -0
index e4131fa42b26..572381306681 100644
@@ -21,6 +21,8 @@ properties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     clkmgr@ffd04000 {
Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml +2 -0
index 853d7d2b56f5..66213bd95e6e 100644
@@ -43,6 +43,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     ao-secure@140 {
Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml +2 -0
index 79902f470e4b..c3a8604dfa80 100644
@@ -43,6 +43,8 @@ required:
   - reg-names
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
Documentation/devicetree/bindings/arm/renesas,prr.yaml +2 -0
index 7f8d17f33983..dd087643a9f8 100644
@@ -27,6 +27,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     prr: chipid@ff000044 {
Documentation/devicetree/bindings/arm/samsung/exynos-chipid.yaml +2 -0
index afcd70803c12..0425d333b50d 100644
@@ -30,6 +30,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     chipid@10000000 {
Documentation/devicetree/bindings/arm/samsung/pmu.yaml +2 -0
index 73b56fc5bf58..c9651892710e 100644
@@ -89,6 +89,8 @@ required:
   - clock-names
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos5250.h>
Documentation/devicetree/bindings/arm/samsung/samsung-secure-firmware.yaml +2 -0
index 51d23b6f8a94..3d9abad3c749 100644
@@ -23,6 +23,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     firmware@203f000 {
Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml +2 -0
index 0dedf94c8578..baff80197d5a 100644
@@ -29,6 +29,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/stm32mp1-clks.h>
Documentation/devicetree/bindings/clock/fsl,plldig.yaml +2 -0
index d1c040228cf7..a203d5d498db 100644
@@ -44,6 +44,8 @@ required:
   - clocks
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Display PIXEL Clock node:
   - |
Documentation/devicetree/bindings/clock/imx8mn-clock.yaml +2 -0
index cd0b8a341321..03386b0861a2 100644
@@ -52,6 +52,8 @@ required:
   - clock-names
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock Control Module node:
   - |
Documentation/devicetree/bindings/clock/imx8mp-clock.yaml +2 -0
index 89aee63c9019..4351a1dbb4f7 100644
@@ -52,6 +52,8 @@ required:
   - clock-names
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock Control Module node:
   - |
Documentation/devicetree/bindings/clock/milbeaut-clock.yaml +2 -0
index f0b804a7f096..0e8b07710451 100644
@@ -35,6 +35,8 @@ required:
   - clocks
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock controller node:
   - |
Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml +2 -0
index 3647007f82ca..eacccc88bbf6 100644
@@ -68,6 +68,8 @@ required:
   - nvmem-cell-names
   - '#thermal-sensor-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@900000 {
Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml +2 -0
index 89c6e070e7ac..98572b4a9b60 100644
@@ -40,6 +40,8 @@ required:
   - '#clock-cells'
   - '#reset-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@1800000 {
Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml +2 -0
index 18e4e77b8cfa..5a5b2214f0ca 100644
@@ -56,6 +56,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@300000 {
Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml +2 -0
index 1d3cae980471..a0bb713929b0 100644
@@ -66,6 +66,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmcc.h>
Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml +2 -0
index 8cdece395eba..ce06f3f8c3e3 100644
@@ -40,6 +40,8 @@ required:
   - '#clock-cells'
   - '#reset-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@1800000 {
Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml +2 -0
index ee4f968e2909..a345320e0e49 100644
@@ -58,6 +58,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml +2 -0
index 888e9a708390..36f3b3668ced 100644
@@ -56,6 +56,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,gcc.yaml +2 -0
index d18f8ab9eeee..e533bb0cfd2b 100644
@@ -74,6 +74,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   # Example for GCC for MSM8960:
   - |
Documentation/devicetree/bindings/clock/qcom,mmcc.yaml +2 -0
index 85518494ce43..f684fe67db84 100644
@@ -74,6 +74,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 if:
   properties:
     compatible:
Documentation/devicetree/bindings/clock/qcom,msm8998-gpucc.yaml +2 -0
index 7d853c1a85e5..d747bb58f0a7 100644
@@ -50,6 +50,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-msm8998.h>
Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml +2 -0
index 2cd158f13bab..c9fd748b4d7c 100644
@@ -35,6 +35,8 @@ required:
   - compatible
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Example for GCC for SDM845: The below node should be defined inside
   # &apps_rsc node.
Documentation/devicetree/bindings/clock/qcom,sc7180-dispcc.yaml +2 -0
index 0429062f1585..58cdfd5924d3 100644
@@ -58,6 +58,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sc7180.h>
Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml +2 -0
index 5785192cc4be..8635e35fd3f0 100644
@@ -52,6 +52,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sc7180.h>
Documentation/devicetree/bindings/clock/qcom,sc7180-videocc.yaml +2 -0
index 31df901884ac..0071b9701960 100644
@@ -48,6 +48,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-dispcc.yaml +2 -0
index 89269ddfbdcd..ad47d747a3e4 100644
@@ -67,6 +67,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml +2 -0
index bac04f1c5d79..7a052ac5dc00 100644
@@ -52,6 +52,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-videocc.yaml +2 -0
index 9d216c0f11d4..2a6a81ab0318 100644
@@ -48,6 +48,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/display/amlogic,meson-vpu.yaml +2 -0
index d1205a6697a0..d8e573eeb5ec 100644
@@ -107,6 +107,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 examples:
   - |
     vpu: vpu@d0100000 {
Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml +2 -0
index 2ca3ddbe1ff4..e7f2ad7dab5e 100644
@@ -43,6 +43,8 @@ required:
   - interrupts
   - '#dma-cells'
 
+additionalProperties: false
+
 examples:
   - |
     dma@3000000 {
Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +2 -0
index f04870d84542..a5dc070d0ca7 100644
@@ -68,6 +68,8 @@ required:
   - mbox-names
   - memory-region
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/firmware/imx/rsrc.h>
Documentation/devicetree/bindings/eeprom/at24.yaml +2 -0
index 0f6d8db18d6c..a15787e504f0 100644
@@ -172,6 +172,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml +3 -0
index 878a2079ebb6..1bd2870c3a9c 100644
@@ -34,9 +34,12 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     npe@c8006000 {
          compatible = "intel,ixp4xx-network-processing-engine";
          reg = <0xc8006000 0x1000>, <0xc8007000 0x1000>, <0xc8008000 0x1000>;
     };
+...
Documentation/devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml +2 -0
index 64e279a4bc10..5f1ed20e43ee 100644
@@ -47,6 +47,8 @@ required:
   - "#gpio-cells"
   - gpio-controller
 
+additionalProperties: false
+
 dependencies:
   interrupt-controller: [ interrupts ]
 
Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml +2 -0
index c58ff9a94f45..1a54db04f29d 100644
@@ -64,6 +64,8 @@ required:
   - gpio-ranges
   - socionext,interrupt-ranges
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml +2 -0
index d102888c1be7..a36aec27069c 100644
@@ -49,6 +49,8 @@ required:
   - "#gpio-cells"
   - gpio-controller
 
+additionalProperties: false
+
 examples:
   - |
     logicvc: logicvc@43c00000 {
Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +2 -0
index 05fd9a404ff7..0b229a7d4a98 100644
@@ -53,6 +53,8 @@ required:
   - interrupt-names
   - clocks
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +2 -0
index 6819cde050df..0407e45eb8c4 100644
@@ -94,6 +94,8 @@ required:
   - interrupt-names
   - clocks
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml +2 -0
index afde81be3c29..f5401cc8de4a 100644
@@ -115,6 +115,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +2 -0
index f4dfa6fc724c..665c6e3b31d3 100644
@@ -36,6 +36,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     rotator@12810000 {
Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml +2 -0
index 2a9822075b36..154bee851139 100644
@@ -47,6 +47,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml +2 -0
index 6a742a51e2f9..44a63fffb4be 100644
@@ -87,6 +87,8 @@ required:
   - reg
 
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml +2 -0
index 5d42e1304202..e8feee38c76c 100644
@@ -32,6 +32,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml +2 -0
index 168235ad5d81..3f043e943668 100644
@@ -76,6 +76,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml +2 -0
index c1c6d6f223cf..8723a336229e 100644
@@ -36,6 +36,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml +2 -0
index 9acde6d2e2d9..a67ba67dab51 100644
@@ -67,6 +67,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml +2 -0
index 91ab9c842273..77605f17901c 100644
@@ -53,6 +53,8 @@ required:
   - dout-gpios
   - avdd-supply
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml +2 -0
index 59009997dca0..118809a03279 100644
@@ -32,6 +32,8 @@ required:
   - vref-supply
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml +2 -0
index 0ce290473fb0..8ffeceb6abae 100644
@@ -52,6 +52,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +2 -0
index acf36eef728b..b1627441a0b2 100644
@@ -69,6 +69,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 patternProperties:
   "^filter@[0-9]+$":
     type: object
Documentation/devicetree/bindings/iio/chemical/plantower,pms7003.yaml +2 -0
index 19e53930ebf6..1fe561574019 100644
@@ -38,6 +38,8 @@ required:
   - compatible
   - vcc-supply
 
+additionalProperties: false
+
 examples:
   - |
     serial {
Documentation/devicetree/bindings/iio/chemical/sensirion,sps30.yaml +2 -0
index 50a50a0d7070..a93d1972a5c2 100644
@@ -24,6 +24,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml +2 -0
index a285eaba7125..e51a585bd5a3 100644
@@ -34,6 +34,8 @@ required:
   - reg
   - vref-supply
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/light/adux1020.yaml +2 -0
index 69bd5c06319d..d7d14f2f1c20 100644
@@ -28,6 +28,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/bh1750.yaml +2 -0
index 1cc60d7ecfa0..1a88b3c253d5 100644
@@ -28,6 +28,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/isl29018.yaml +2 -0
index cbb00be8f359..0ea278b07d1c 100644
@@ -38,6 +38,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/noa1305.yaml +2 -0
index 17e7f140b69b..fe7bfe1adbda 100644
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/stk33xx.yaml +2 -0
index aae8a6d627c9..f92bf7b2b7f0 100644
@@ -30,6 +30,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/tsl2583.yaml +2 -0
index e86ef64ecf03..7b92ba8cbb9f 100644
@@ -32,6 +32,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/tsl2772.yaml +2 -0
index ed2c3d5eadf5..e8f7d1ada57b 100644
@@ -62,6 +62,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/veml6030.yaml +2 -0
index 0ff9b11f9d18..fb19a2d7a849 100644
@@ -45,6 +45,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/pressure/asc,dlhl60d.yaml +2 -0
index 9f5ca9c42025..64c18f1693f0 100644
@@ -33,6 +33,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/pressure/bmp085.yaml +2 -0
index 5d4aec0e0d24..49257f9251e8 100644
@@ -52,6 +52,8 @@ required:
   - vddd-supply
   - vdda-supply
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml +2 -0
index 4e80ea7c1475..7ac5eb7560e0 100644
@@ -56,6 +56,8 @@ required:
   - trig-gpios
   - echo-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/proximity/parallax-ping.yaml +2 -0
index a079c9921af6..ada55f186f3c 100644
@@ -42,6 +42,8 @@ required:
   - compatible
   - ping-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml +2 -0
index d4922f9f0376..acc030c1b20e 100644
@@ -398,6 +398,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/input/gpio-vibrator.yaml +2 -0
index b98bf9363c8f..2384465eaa19 100644
@@ -26,6 +26,8 @@ required:
   - compatible
   - enable-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/input/max77650-onkey.yaml +3 -0
index 2f2e0b6ebbbd..3a2ad6ec64db 100644
@@ -33,3 +33,6 @@ properties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml +2 -0
index 507c141ea760..ccc507f384d2 100644
@@ -44,6 +44,8 @@ required:
   - interrupt-controller
   - '#interrupt-cells'
 
+additionalProperties: false
+
 examples:
   - |
     intcon: interrupt-controller@c8003000 {
Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml +2 -0
index 7cdd3aaa2ba4..0e33cd9e010e 100644
@@ -80,6 +80,8 @@ required:
   - clock-names
   - "#iommu-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos5250.h>
Documentation/devicetree/bindings/leds/leds-max77650.yaml +3 -0
index 8c43f1e1bf7d..c6f96cabd4d1 100644
@@ -49,3 +49,6 @@ required:
   - compatible
   - "#address-cells"
   - "#size-cells"
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml +3 -0
index b50f4bcc98f1..90edf9d33b33 100644
@@ -50,3 +50,6 @@ patternProperties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml +2 -0
index 319280563648..aa2b3bf56b57 100644
@@ -41,6 +41,8 @@ required:
   - interrupts
   - "#mbox-cells"
 
+additionalProperties: false
+
 examples:
   - |
     mailbox@c883c404 {
Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml +2 -0
index 335717e15970..37d77e065491 100644
@@ -124,6 +124,8 @@ required:
   - amlogic,ao-sysctrl
   - amlogic,canvas
 
+additionalProperties: false
+
 examples:
   - |
     vdec: video-decoder@c8820000 {
Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml +2 -0
index e8ce37fcbfec..95ffa8bc0533 100644
@@ -82,6 +82,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     cec_AO: cec@100 {
Documentation/devicetree/bindings/media/renesas,ceu.yaml +2 -0
index 8e9251a0f9ef..fcb5f13704a5 100644
@@ -59,6 +59,8 @@ required:
   - interrupts
   - port
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
Documentation/devicetree/bindings/mfd/max77650.yaml +2 -0
index 480385789394..b0a0f0d3d9d4 100644
@@ -73,6 +73,8 @@ required:
   - gpio-controller
   - "#gpio-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml +2 -0
index 38dc4f8b0ceb..3a6a1a26e2b3 100644
@@ -77,6 +77,8 @@ required:
   - gpio-controller
   - "#gpio-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml +2 -0
index 0ea21a6f70b4..38ab0499102d 100644
@@ -38,6 +38,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +2 -0
index b9e9696da5be..976f139bb66e 100644
@@ -167,6 +167,8 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml +2 -0
index e5922b427342..c03b83103e87 100644
@@ -34,6 +34,8 @@ required:
   - resets
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/sun50i-h6-ccu.h>
Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml +2 -0
index 346f9c35427c..453c083cf44c 100644
@@ -44,6 +44,8 @@ required:
   - reset-names
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     phy@46000 {
Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml +2 -0
index 0ccee64c6962..9a346d6290d9 100644
@@ -40,6 +40,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     sysconf: chiptop@e0200000 {
Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +2 -0
index 5ab436189f3b..00609ace677c 100644
@@ -31,6 +31,8 @@ required:
   - reset-gpios
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml +2 -0
index 452cee1aed32..fd1982c56104 100644
@@ -159,6 +159,8 @@ required:
   - "#reset-cells"
   - ranges
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/soc/ti,sci_pm_domain.h>
Documentation/devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml +2 -0
index 135c7dfbc180..7651a675ab2d 100644
@@ -57,6 +57,8 @@ patternProperties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     syscon: scu@1e6e2000 {
Documentation/devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml +2 -0
index 824f7fd1d51b..36feaf5e2dff 100644
@@ -70,6 +70,8 @@ required:
   - compatible
   - aspeed,external-nodes
 
+additionalProperties: false
+
 examples:
   - |
     apb {
Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml +2 -0
index ac8d1c30a8ed..45af29bc3202 100644
@@ -92,6 +92,8 @@ patternProperties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     syscon: scu@1e6e2000 {
Documentation/devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml +2 -0
index ef4de32cb17c..46a0478cb924 100644
@@ -194,6 +194,8 @@ required:
   - ranges
   - pins-are-numbered
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/pinctrl/stm32-pinfunc.h>
Documentation/devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml +2 -0
index d3098c924b25..6c6079fe1351 100644
@@ -68,6 +68,8 @@ required:
   - "#power-domain-cells"
   - amlogic,ao-sysctrl
 
+additionalProperties: false
+
 examples:
   - |
     pwrc: power-controller {
Documentation/devicetree/bindings/power/reset/syscon-poweroff.yaml +2 -0
index 520e07e6f21b..3412fe7e1e80 100644
@@ -41,6 +41,8 @@ required:
   - regmap
   - offset
 
+additionalProperties: false
+
 allOf:
   - if:
       not:
Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +2 -0
index d38006b1f1f4..b80772cb9f06 100644
@@ -41,6 +41,8 @@ required:
   - regmap
   - offset
 
+additionalProperties: false
+
 allOf:
   - if:
       not:
Documentation/devicetree/bindings/power/supply/max77650-charger.yaml +3 -0
index deef010ec535..62eeddb65aed 100644
@@ -32,3 +32,6 @@ properties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml +2 -0
index 9e21b83d717e..239b49fad805 100644
@@ -55,6 +55,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c@1 {
Documentation/devicetree/bindings/regulator/max77650-regulator.yaml +3 -0
index 50690487edc8..ce0a4021ae7f 100644
@@ -29,3 +29,6 @@ patternProperties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/reset/amlogic,meson-reset.yaml +2 -0
index b3f57d81f007..92922d3afd14 100644
@@ -29,6 +29,8 @@ required:
   - reg
   - "#reset-cells"
 
+additionalProperties: false
+
 examples:
   - |
     reset-controller@c884404 {
Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml +2 -0
index 411bd76f1b64..512a33bdb208 100644
@@ -28,6 +28,8 @@ required:
   - reg
   - "#reset-cells"
 
+additionalProperties: false
+
 examples:
   - |
     reset-controller@8b2c800 {
Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml +2 -0
index a9ff3cb35c5e..444be32a8a29 100644
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     rng@c8834000 {
Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml +2 -0
index 42d9a38e4e1a..89ab67f20a7f 100644
@@ -35,6 +35,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     rng {
Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml +2 -0
index dcff573cbdb1..b95cb017f469 100644
@@ -51,6 +51,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/r7s72100-clock.h>
Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml +2 -0
index 0a54296d7218..48c6cafca90c 100644
@@ -111,6 +111,8 @@ required:
   - clocks
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/mfd/stm32f4-rcc.h>
Documentation/devicetree/bindings/serial/amlogic,meson-uart.yaml +2 -0
index 214fe8beddc3..d4178ab0d675 100644
@@ -62,6 +62,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     serial@84c0 {
Documentation/devicetree/bindings/soc/amlogic/amlogic,canvas.yaml +2 -0
index f548594d020b..cb008fd188d8 100644
@@ -40,6 +40,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     canvas: video-lut@48 {
Documentation/devicetree/bindings/sound/adi,adau7118.yaml +2 -0
index 75e0cbe6be70..76ee695097bf 100644
@@ -59,6 +59,8 @@ required:
   - iovdd-supply
   - dvdd-supply
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml +2 -0
index 38eaf0c028f9..a495d5fc0d23 100644
@@ -139,6 +139,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 examples:
   - |
     codec@1,0{
Documentation/devicetree/bindings/sound/renesas,fsi.yaml +2 -0
index 140a37fc3c0b..d1b65554e681 100644
@@ -63,6 +63,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     sh_fsi2: sound@ec230000 {
Documentation/devicetree/bindings/sound/samsung,odroid.yaml +2 -0
index c6b244352d05..8ff2d39e7d17 100644
@@ -69,6 +69,8 @@ required:
   - cpu
   - codec
 
+additionalProperties: false
+
 examples:
   - |
     sound {
Documentation/devicetree/bindings/sound/samsung-i2s.yaml +2 -0
index 53e3bad4178c..b2ad093d94df 100644
@@ -115,6 +115,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos-audss-clk.h>
Documentation/devicetree/bindings/sram/qcom,ocmem.yaml +2 -0
index 469cec133647..930188bc5e6a 100644
@@ -56,6 +56,8 @@ required:
   - '#size-cells'
   - ranges
 
+additionalProperties: false
+
 patternProperties:
   "-sram@[0-9a-f]+$":
     type: object
Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2 -0
index 93fe7b10a82e..e43ec50bda37 100644
@@ -42,6 +42,8 @@ required:
   - clocks
   - amlogic,ao-secure
 
+additionalProperties: false
+
 examples:
   - |
         cpu_temp: temperature-sensor@ff634800 {
Documentation/devicetree/bindings/timer/arm,arch_timer.yaml +2 -0
index 6deead07728e..fa255672e8e5 100644
@@ -82,6 +82,8 @@ properties:
 required:
   - compatible
 
+additionalProperties: false
+
 oneOf:
   - required:
       - interrupts
Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2 -0
index f7ef6646bade..582bbef62b95 100644
@@ -95,6 +95,8 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+additionalProperties: false
+
 examples:
   - |
     timer@f0000000 {
Documentation/devicetree/bindings/timer/arm,global_timer.yaml +2 -0
index 21c24a8e28fd..4956c8f409d2 100644
@@ -35,6 +35,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     timer@2c000600 {
Documentation/devicetree/bindings/timer/intel,ixp4xx-timer.yaml +2 -0
index 2807225db902..1a721d8af67a 100644
@@ -32,6 +32,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml +2 -0
index 273e359854dd..37bd01a62c52 100644
@@ -52,6 +52,8 @@ required:
   - interrupts
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     // In this example, the IP contains two local timers, using separate
Documentation/devicetree/bindings/trivial-devices.yaml +2 -0
index 51d1f6e43c02..bcae5f9b1d7f 100644
@@ -369,4 +369,6 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 ...
Signature · 6 lines
2.20.1

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
maintainer Acked-by on v1 4/4
#17 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

Quoted 10 lines · Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Acked-by on v1 4/4
#18 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

In reply to Rob Herring
Quoted 11 lines · Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

Quoted 3 lines

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

For Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml Documentation/devicetree/bindings/sound/adi,adau7118.yaml

[also, if i missed any, also for any other ADI or LTC bindings]

Quoted 1,784 lines
---
 .../devicetree/bindings/arm/altera/socfpga-clk-manager.yaml    | 2 ++
 .../bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/renesas,prr.yaml         | 2 ++
 .../devicetree/bindings/arm/samsung/exynos-chipid.yaml         | 2 ++
 Documentation/devicetree/bindings/arm/samsung/pmu.yaml         | 2 ++
 .../bindings/arm/samsung/samsung-secure-firmware.yaml          | 2 ++
 .../devicetree/bindings/arm/stm32/st,stm32-syscon.yaml         | 2 ++
 Documentation/devicetree/bindings/clock/fsl,plldig.yaml        | 2 ++
 Documentation/devicetree/bindings/clock/imx8mn-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/imx8mp-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/milbeaut-clock.yaml    | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,mmcc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,msm8998-gpucc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml       | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-videocc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-videocc.yaml         | 2 ++
 .../devicetree/bindings/display/amlogic,meson-vpu.yaml         | 2 ++
 .../devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml        | 2 ++
 Documentation/devicetree/bindings/dsp/fsl,dsp.yaml             | 2 ++
 Documentation/devicetree/bindings/eeprom/at24.yaml             | 2 ++
 .../firmware/intel,ixp4xx-network-processing-engine.yaml       | 3 +++
 .../devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml          | 2 ++
 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++
 Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml     | 2 ++
 Documentation/devicetree/bindings/gpu/samsung-rotator.yaml     | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml | 2 ++
 Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml         | 2 ++
 Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml  | 2 ++
 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml    | 2 ++
 .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml         | 2 ++
 .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml        | 2 ++
 .../devicetree/bindings/iio/chemical/plantower,pms7003.yaml    | 2 ++
 .../devicetree/bindings/iio/chemical/sensirion,sps30.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml    | 2 ++
 Documentation/devicetree/bindings/iio/light/adux1020.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/bh1750.yaml        | 2 ++
 Documentation/devicetree/bindings/iio/light/isl29018.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/noa1305.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/stk33xx.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2583.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2772.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/veml6030.yaml      | 2 ++
 .../devicetree/bindings/iio/pressure/asc,dlhl60d.yaml          | 2 ++
 Documentation/devicetree/bindings/iio/pressure/bmp085.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/devantech-srf04.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/parallax-ping.yaml       | 2 ++
 .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml       | 2 ++
 Documentation/devicetree/bindings/input/gpio-vibrator.yaml     | 2 ++
 Documentation/devicetree/bindings/input/max77650-onkey.yaml    | 3 +++
 .../bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml  | 2 ++
 Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml    | 2 ++
 Documentation/devicetree/bindings/leds/leds-max77650.yaml      | 3 +++
 Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml  | 3 +++
 .../devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml    | 2 ++
 Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml   | 2 ++
 .../devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml     | 2 ++
 Documentation/devicetree/bindings/media/renesas,ceu.yaml       | 2 ++
 Documentation/devicetree/bindings/mfd/max77650.yaml            | 2 ++
 Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml   | 2 ++
 .../bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml          | 2 ++
 Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml      | 2 ++
 .../devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml  | 2 ++
 .../bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml  | 2 ++
 .../devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml          | 2 ++
 .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-poweroff.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-reboot.yaml         | 2 ++
 .../devicetree/bindings/power/supply/max77650-charger.yaml     | 3 +++
 Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml           | 2 ++
 .../devicetree/bindings/regulator/max77650-regulator.yaml      | 3 +++
 .../devicetree/bindings/reset/amlogic,meson-reset.yaml         | 2 ++
 .../bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml          | 2 ++
 Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml   | 2 ++
 Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml        | 2 ++
 Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml      | 2 ++
 Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml        | 2 ++
 .../devicetree/bindings/serial/amlogic,meson-uart.yaml         | 2 ++
 .../devicetree/bindings/soc/amlogic/amlogic,canvas.yaml        | 2 ++
 Documentation/devicetree/bindings/sound/adi,adau7118.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/renesas,fsi.yaml       | 2 ++
 Documentation/devicetree/bindings/sound/samsung,odroid.yaml    | 2 ++
 Documentation/devicetree/bindings/sound/samsung-i2s.yaml       | 2 ++
 Documentation/devicetree/bindings/sram/qcom,ocmem.yaml         | 2 ++
 Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml | 2 ++
 Documentation/devicetree/bindings/timer/arm,arch_timer.yaml    | 2 ++
 .../devicetree/bindings/timer/arm,arch_timer_mmio.yaml         | 2 ++
 Documentation/devicetree/bindings/timer/arm,global_timer.yaml  | 2 ++
 .../devicetree/bindings/timer/intel,ixp4xx-timer.yaml          | 2 ++
 .../devicetree/bindings/timer/samsung,exynos4210-mct.yaml      | 2 ++
 Documentation/devicetree/bindings/trivial-devices.yaml         | 2 ++
 117 files changed, 240 insertions(+)
(quoted diff) +0 -0

manager.yaml b/Documentation/devicetree/bindings/arm/altera/socfpga-clk- manager.yaml
index e4131fa42b26..572381306681 100644

Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml +2 -0
@@ -21,6 +21,8 @@ properties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     clkmgr@ffd04000 {
(quoted diff) +0 -0

ao-secure.yaml b/Documentation/devicetree/bindings/arm/amlogic/amlogic,meson- gx-ao-secure.yaml
index 853d7d2b56f5..66213bd95e6e 100644
--- a/Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao- secure.yaml
+++ b/Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao- secure.yaml

Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml +2 -0
@@ -43,6 +43,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     ao-secure@140 {
Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml +2 -0
index 79902f470e4b..c3a8604dfa80 100644
@@ -43,6 +43,8 @@ required:
   - reg-names
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
Documentation/devicetree/bindings/arm/renesas,prr.yaml +2 -0
index 7f8d17f33983..dd087643a9f8 100644
@@ -27,6 +27,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     prr: chipid@ff000044 {
Documentation/devicetree/bindings/arm/samsung/exynos-chipid.yaml +2 -0
index afcd70803c12..0425d333b50d 100644
@@ -30,6 +30,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     chipid@10000000 {
Documentation/devicetree/bindings/arm/samsung/pmu.yaml +2 -0
index 73b56fc5bf58..c9651892710e 100644
@@ -89,6 +89,8 @@ required:
   - clock-names
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos5250.h>
(quoted diff) +0 -0

firmware.yaml b/Documentation/devicetree/bindings/arm/samsung/samsung-secure- firmware.yaml
index 51d23b6f8a94..3d9abad3c749 100644
--- a/Documentation/devicetree/bindings/arm/samsung/samsung-secure- firmware.yaml
+++ b/Documentation/devicetree/bindings/arm/samsung/samsung-secure- firmware.yaml

Documentation/devicetree/bindings/arm/samsung/samsung-secure-firmware.yaml +2 -0
@@ -23,6 +23,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     firmware@203f000 {
Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml +2 -0
index 0dedf94c8578..baff80197d5a 100644
@@ -29,6 +29,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/stm32mp1-clks.h>
Documentation/devicetree/bindings/clock/fsl,plldig.yaml +2 -0
index d1c040228cf7..a203d5d498db 100644
@@ -44,6 +44,8 @@ required:
   - clocks
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Display PIXEL Clock node:
   - |
Documentation/devicetree/bindings/clock/imx8mn-clock.yaml +2 -0
index cd0b8a341321..03386b0861a2 100644
@@ -52,6 +52,8 @@ required:
   - clock-names
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock Control Module node:
   - |
Documentation/devicetree/bindings/clock/imx8mp-clock.yaml +2 -0
index 89aee63c9019..4351a1dbb4f7 100644
@@ -52,6 +52,8 @@ required:
   - clock-names
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock Control Module node:
   - |
Documentation/devicetree/bindings/clock/milbeaut-clock.yaml +2 -0
index f0b804a7f096..0e8b07710451 100644
@@ -35,6 +35,8 @@ required:
   - clocks
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock controller node:
   - |
Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml +2 -0
index 3647007f82ca..eacccc88bbf6 100644
@@ -68,6 +68,8 @@ required:
   - nvmem-cell-names
   - '#thermal-sensor-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@900000 {
Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml +2 -0
index 89c6e070e7ac..98572b4a9b60 100644
@@ -40,6 +40,8 @@ required:
   - '#clock-cells'
   - '#reset-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@1800000 {
Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml +2 -0
index 18e4e77b8cfa..5a5b2214f0ca 100644
@@ -56,6 +56,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@300000 {
Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml +2 -0
index 1d3cae980471..a0bb713929b0 100644
@@ -66,6 +66,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmcc.h>
Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml +2 -0
index 8cdece395eba..ce06f3f8c3e3 100644
@@ -40,6 +40,8 @@ required:
   - '#clock-cells'
   - '#reset-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@1800000 {
Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml +2 -0
index ee4f968e2909..a345320e0e49 100644
@@ -58,6 +58,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml +2 -0
index 888e9a708390..36f3b3668ced 100644
@@ -56,6 +56,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,gcc.yaml +2 -0
index d18f8ab9eeee..e533bb0cfd2b 100644
@@ -74,6 +74,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   # Example for GCC for MSM8960:
   - |
Documentation/devicetree/bindings/clock/qcom,mmcc.yaml +2 -0
index 85518494ce43..f684fe67db84 100644
@@ -74,6 +74,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 if:
   properties:
     compatible:
Documentation/devicetree/bindings/clock/qcom,msm8998-gpucc.yaml +2 -0
index 7d853c1a85e5..d747bb58f0a7 100644
@@ -50,6 +50,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-msm8998.h>
Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml +2 -0
index 2cd158f13bab..c9fd748b4d7c 100644
@@ -35,6 +35,8 @@ required:
   - compatible
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Example for GCC for SDM845: The below node should be defined inside
   # &apps_rsc node.
Documentation/devicetree/bindings/clock/qcom,sc7180-dispcc.yaml +2 -0
index 0429062f1585..58cdfd5924d3 100644
@@ -58,6 +58,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sc7180.h>
Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml +2 -0
index 5785192cc4be..8635e35fd3f0 100644
@@ -52,6 +52,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sc7180.h>
Documentation/devicetree/bindings/clock/qcom,sc7180-videocc.yaml +2 -0
index 31df901884ac..0071b9701960 100644
@@ -48,6 +48,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-dispcc.yaml +2 -0
index 89269ddfbdcd..ad47d747a3e4 100644
@@ -67,6 +67,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml +2 -0
index bac04f1c5d79..7a052ac5dc00 100644
@@ -52,6 +52,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-videocc.yaml +2 -0
index 9d216c0f11d4..2a6a81ab0318 100644
@@ -48,6 +48,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/display/amlogic,meson-vpu.yaml +2 -0
index d1205a6697a0..d8e573eeb5ec 100644
@@ -107,6 +107,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 examples:
   - |
     vpu: vpu@d0100000 {
Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml +2 -0
index 2ca3ddbe1ff4..e7f2ad7dab5e 100644
@@ -43,6 +43,8 @@ required:
   - interrupts
   - '#dma-cells'
 
+additionalProperties: false
+
 examples:
   - |
     dma@3000000 {
Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +2 -0
index f04870d84542..a5dc070d0ca7 100644
@@ -68,6 +68,8 @@ required:
   - mbox-names
   - memory-region
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/firmware/imx/rsrc.h>
Documentation/devicetree/bindings/eeprom/at24.yaml +2 -0
index 0f6d8db18d6c..a15787e504f0 100644
@@ -172,6 +172,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
(quoted diff) +0 -0

processing-engine.yaml b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing- engine.yaml
index 878a2079ebb6..1bd2870c3a9c 100644
--- a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network- processing-engine.yaml
+++ b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network- processing-engine.yaml

Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml +3 -0
@@ -34,9 +34,12 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     npe@c8006000 {
          compatible = "intel,ixp4xx-network-processing-engine";
          reg = <0xc8006000 0x1000>, <0xc8007000 0x1000>, <0xc8008000 0x1000>;
     };
+...
Documentation/devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml +2 -0
index 64e279a4bc10..5f1ed20e43ee 100644
@@ -47,6 +47,8 @@ required:
   - "#gpio-cells"
   - gpio-controller
 
+additionalProperties: false
+
 dependencies:
   interrupt-controller: [ interrupts ]
 
(quoted diff) +0 -0

gpio.yaml b/Documentation/devicetree/bindings/gpio/socionext,uniphier- gpio.yaml
index c58ff9a94f45..1a54db04f29d 100644

Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml +2 -0
@@ -64,6 +64,8 @@ required:
   - gpio-ranges
   - socionext,interrupt-ranges
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml +2 -0
index d102888c1be7..a36aec27069c 100644
@@ -49,6 +49,8 @@ required:
   - "#gpio-cells"
   - gpio-controller
 
+additionalProperties: false
+
 examples:
   - |
     logicvc: logicvc@43c00000 {
Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +2 -0
index 05fd9a404ff7..0b229a7d4a98 100644
@@ -53,6 +53,8 @@ required:
   - interrupt-names
   - clocks
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +2 -0
index 6819cde050df..0407e45eb8c4 100644
@@ -94,6 +94,8 @@ required:
   - interrupt-names
   - clocks
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml +2 -0
index afde81be3c29..f5401cc8de4a 100644
@@ -115,6 +115,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +2 -0
index f4dfa6fc724c..665c6e3b31d3 100644
@@ -36,6 +36,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     rotator@12810000 {
Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml +2 -0
index 2a9822075b36..154bee851139 100644
@@ -47,6 +47,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml +2 -0
index 6a742a51e2f9..44a63fffb4be 100644
@@ -87,6 +87,8 @@ required:
   - reg
 
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml +2 -0
index 5d42e1304202..e8feee38c76c 100644
@@ -32,6 +32,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml +2 -0
index 168235ad5d81..3f043e943668 100644
@@ -76,6 +76,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml +2 -0
index c1c6d6f223cf..8723a336229e 100644
@@ -36,6 +36,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml +2 -0
index 9acde6d2e2d9..a67ba67dab51 100644
@@ -67,6 +67,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml +2 -0
index 91ab9c842273..77605f17901c 100644
@@ -53,6 +53,8 @@ required:
   - dout-gpios
   - avdd-supply
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml +2 -0
index 59009997dca0..118809a03279 100644
@@ -32,6 +32,8 @@ required:
   - vref-supply
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml +2 -0
index 0ce290473fb0..8ffeceb6abae 100644
@@ -52,6 +52,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +2 -0
index acf36eef728b..b1627441a0b2 100644
@@ -69,6 +69,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 patternProperties:
   "^filter@[0-9]+$":
     type: object

diff --git a/Documentation/devicetree/bindings/iio/chemical/plantower,pms7003.yaml b/Documentation/devicetree/bindings/iio/chemical/plantower,pms7003.yaml index 19e53930ebf6..1fe561574019 100644

Documentation/devicetree/bindings/iio/chemical/plantower,pms7003.yaml +2 -0
@@ -38,6 +38,8 @@ required:
   - compatible
   - vcc-supply
 
+additionalProperties: false
+
 examples:
   - |
     serial {

diff --git a/Documentation/devicetree/bindings/iio/chemical/sensirion,sps30.yaml b/Documentation/devicetree/bindings/iio/chemical/sensirion,sps30.yaml index 50a50a0d7070..a93d1972a5c2 100644

Documentation/devicetree/bindings/iio/chemical/sensirion,sps30.yaml +2 -0
@@ -24,6 +24,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml +2 -0
index a285eaba7125..e51a585bd5a3 100644
@@ -34,6 +34,8 @@ required:
   - reg
   - vref-supply
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/light/adux1020.yaml +2 -0
index 69bd5c06319d..d7d14f2f1c20 100644
@@ -28,6 +28,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/bh1750.yaml +2 -0
index 1cc60d7ecfa0..1a88b3c253d5 100644
@@ -28,6 +28,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/isl29018.yaml +2 -0
index cbb00be8f359..0ea278b07d1c 100644
@@ -38,6 +38,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/noa1305.yaml +2 -0
index 17e7f140b69b..fe7bfe1adbda 100644
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/stk33xx.yaml +2 -0
index aae8a6d627c9..f92bf7b2b7f0 100644
@@ -30,6 +30,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/tsl2583.yaml +2 -0
index e86ef64ecf03..7b92ba8cbb9f 100644
@@ -32,6 +32,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/tsl2772.yaml +2 -0
index ed2c3d5eadf5..e8f7d1ada57b 100644
@@ -62,6 +62,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/veml6030.yaml +2 -0
index 0ff9b11f9d18..fb19a2d7a849 100644
@@ -45,6 +45,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/pressure/asc,dlhl60d.yaml +2 -0
index 9f5ca9c42025..64c18f1693f0 100644
@@ -33,6 +33,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/pressure/bmp085.yaml +2 -0
index 5d4aec0e0d24..49257f9251e8 100644
@@ -52,6 +52,8 @@ required:
   - vddd-supply
   - vdda-supply
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
(quoted diff) +0 -0

srf04.yaml b/Documentation/devicetree/bindings/iio/proximity/devantech- srf04.yaml
index 4e80ea7c1475..7ac5eb7560e0 100644

Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml +2 -0
@@ -56,6 +56,8 @@ required:
   - trig-gpios
   - echo-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
(quoted diff) +0 -0

ping.yaml b/Documentation/devicetree/bindings/iio/proximity/parallax-ping.yaml index a079c9921af6..ada55f186f3c 100644

Documentation/devicetree/bindings/iio/proximity/parallax-ping.yaml +2 -0
@@ -42,6 +42,8 @@ required:
   - compatible
   - ping-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>

diff --git a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml index d4922f9f0376..acc030c1b20e 100644

Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml +2 -0
@@ -398,6 +398,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/input/gpio-vibrator.yaml +2 -0
index b98bf9363c8f..2384465eaa19 100644
@@ -26,6 +26,8 @@ required:
   - compatible
   - enable-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/input/max77650-onkey.yaml +3 -0
index 2f2e0b6ebbbd..3a2ad6ec64db 100644
@@ -33,3 +33,6 @@ properties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
(quoted diff) +0 -0

controller/intel,ixp4xx-interrupt.yaml b/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx- interrupt.yaml
index 507c141ea760..ccc507f384d2 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx- interrupt.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx- interrupt.yaml

Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml +2 -0
@@ -44,6 +44,8 @@ required:
   - interrupt-controller
   - '#interrupt-cells'
 
+additionalProperties: false
+
 examples:
   - |
     intcon: interrupt-controller@c8003000 {
Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml +2 -0
index 7cdd3aaa2ba4..0e33cd9e010e 100644
@@ -80,6 +80,8 @@ required:
   - clock-names
   - "#iommu-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos5250.h>
Documentation/devicetree/bindings/leds/leds-max77650.yaml +3 -0
index 8c43f1e1bf7d..c6f96cabd4d1 100644
@@ -49,3 +49,6 @@ required:
   - compatible
   - "#address-cells"
   - "#size-cells"
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml +3 -0
index b50f4bcc98f1..90edf9d33b33 100644
@@ -50,3 +50,6 @@ patternProperties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
(quoted diff) +0 -0

mhu.yaml b/Documentation/devicetree/bindings/mailbox/amlogic,meson-gxbb- mhu.yaml
index 319280563648..aa2b3bf56b57 100644

Documentation/devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml +2 -0
@@ -41,6 +41,8 @@ required:
   - interrupts
   - "#mbox-cells"
 
+additionalProperties: false
+
 examples:
   - |
     mailbox@c883c404 {
Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml +2 -0
index 335717e15970..37d77e065491 100644
@@ -124,6 +124,8 @@ required:
   - amlogic,ao-sysctrl
   - amlogic,canvas
 
+additionalProperties: false
+
 examples:
   - |
     vdec: video-decoder@c8820000 {
(quoted diff) +0 -0

cec.yaml b/Documentation/devicetree/bindings/media/amlogic,meson-gx-ao- cec.yaml
index e8ce37fcbfec..95ffa8bc0533 100644

Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml +2 -0
@@ -82,6 +82,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     cec_AO: cec@100 {
Documentation/devicetree/bindings/media/renesas,ceu.yaml +2 -0
index 8e9251a0f9ef..fcb5f13704a5 100644
@@ -59,6 +59,8 @@ required:
   - interrupts
   - port
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
Documentation/devicetree/bindings/mfd/max77650.yaml +2 -0
index 480385789394..b0a0f0d3d9d4 100644
@@ -73,6 +73,8 @@ required:
   - gpio-controller
   - "#gpio-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml +2 -0
index 38dc4f8b0ceb..3a6a1a26e2b3 100644
@@ -77,6 +77,8 @@ required:
   - gpio-controller
   - "#gpio-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
(quoted diff) +0 -0

manager.yaml b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue- manager.yaml
index 0ea21a6f70b4..38ab0499102d 100644
--- a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue- manager.yaml
+++ b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue- manager.yaml

Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml +2 -0
@@ -38,6 +38,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +2 -0
index b9e9696da5be..976f139bb66e 100644
@@ -167,6 +167,8 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
(quoted diff) +0 -0

phy.yaml b/Documentation/devicetree/bindings/phy/allwinner,sun50i-h6-usb3- phy.yaml
index e5922b427342..c03b83103e87 100644

Documentation/devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml +2 -0
@@ -34,6 +34,8 @@ required:
   - resets
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/sun50i-h6-ccu.h>
(quoted diff) +0 -0

pcie-phy.yaml b/Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3- pcie-phy.yaml
index 346f9c35427c..453c083cf44c 100644
--- a/Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie- phy.yaml
+++ b/Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie- phy.yaml

Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml +2 -0
@@ -44,6 +44,8 @@ required:
   - reset-names
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     phy@46000 {
Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml +2 -0
index 0ccee64c6962..9a346d6290d9 100644
@@ -40,6 +40,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     sysconf: chiptop@e0200000 {
Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +2 -0
index 5ab436189f3b..00609ace677c 100644
@@ -31,6 +31,8 @@ required:
   - reset-gpios
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml +2 -0
index 452cee1aed32..fd1982c56104 100644
@@ -159,6 +159,8 @@ required:
   - "#reset-cells"
   - ranges
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/soc/ti,sci_pm_domain.h>
(quoted diff) +0 -0

pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/aspeed,ast2400- pinctrl.yaml
index 135c7dfbc180..7651a675ab2d 100644

Documentation/devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml +2 -0
@@ -57,6 +57,8 @@ patternProperties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     syscon: scu@1e6e2000 {
(quoted diff) +0 -0

pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/aspeed,ast2500- pinctrl.yaml
index 824f7fd1d51b..36feaf5e2dff 100644

Documentation/devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml +2 -0
@@ -70,6 +70,8 @@ required:
   - compatible
   - aspeed,external-nodes
 
+additionalProperties: false
+
 examples:
   - |
     apb {
(quoted diff) +0 -0

pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/aspeed,ast2600- pinctrl.yaml
index ac8d1c30a8ed..45af29bc3202 100644

Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml +2 -0
@@ -92,6 +92,8 @@ patternProperties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     syscon: scu@1e6e2000 {
Documentation/devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml +2 -0
index ef4de32cb17c..46a0478cb924 100644
@@ -194,6 +194,8 @@ required:
   - ranges
   - pins-are-numbered
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/pinctrl/stm32-pinfunc.h>
(quoted diff) +0 -0

pwrc.yaml b/Documentation/devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml index d3098c924b25..6c6079fe1351 100644

Documentation/devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml +2 -0
@@ -68,6 +68,8 @@ required:
   - "#power-domain-cells"
   - amlogic,ao-sysctrl
 
+additionalProperties: false
+
 examples:
   - |
     pwrc: power-controller {
(quoted diff) +0 -0

poweroff.yaml b/Documentation/devicetree/bindings/power/reset/syscon- poweroff.yaml
index 520e07e6f21b..3412fe7e1e80 100644

Documentation/devicetree/bindings/power/reset/syscon-poweroff.yaml +2 -0
@@ -41,6 +41,8 @@ required:
   - regmap
   - offset
 
+additionalProperties: false
+
 allOf:
   - if:
       not:
Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +2 -0
index d38006b1f1f4..b80772cb9f06 100644
@@ -41,6 +41,8 @@ required:
   - regmap
   - offset
 
+additionalProperties: false
+
 allOf:
   - if:
       not:
(quoted diff) +0 -0

charger.yaml b/Documentation/devicetree/bindings/power/supply/max77650- charger.yaml
index deef010ec535..62eeddb65aed 100644

Documentation/devicetree/bindings/power/supply/max77650-charger.yaml +3 -0
@@ -32,3 +32,6 @@ properties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml +2 -0
index 9e21b83d717e..239b49fad805 100644
@@ -55,6 +55,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c@1 {
(quoted diff) +0 -0

regulator.yaml b/Documentation/devicetree/bindings/regulator/max77650- regulator.yaml
index 50690487edc8..ce0a4021ae7f 100644

Documentation/devicetree/bindings/regulator/max77650-regulator.yaml +3 -0
@@ -29,3 +29,6 @@ patternProperties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/reset/amlogic,meson-reset.yaml +2 -0
index b3f57d81f007..92922d3afd14 100644
@@ -29,6 +29,8 @@ required:
   - reg
   - "#reset-cells"
 
+additionalProperties: false
+
 examples:
   - |
     reset-controller@c884404 {
(quoted diff) +0 -0

rescal.yaml b/Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata- rescal.yaml
index 411bd76f1b64..512a33bdb208 100644
--- a/Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata- rescal.yaml
+++ b/Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata- rescal.yaml

Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml +2 -0
@@ -28,6 +28,8 @@ required:
   - reg
   - "#reset-cells"
 
+additionalProperties: false
+
 examples:
   - |
     reset-controller@8b2c800 {
Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml +2 -0
index a9ff3cb35c5e..444be32a8a29 100644
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     rng@c8834000 {
Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml +2 -0
index 42d9a38e4e1a..89ab67f20a7f 100644
@@ -35,6 +35,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     rng {
Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml +2 -0
index dcff573cbdb1..b95cb017f469 100644
@@ -51,6 +51,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/r7s72100-clock.h>
Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml +2 -0
index 0a54296d7218..48c6cafca90c 100644
@@ -111,6 +111,8 @@ required:
   - clocks
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/mfd/stm32f4-rcc.h>
Documentation/devicetree/bindings/serial/amlogic,meson-uart.yaml +2 -0
index 214fe8beddc3..d4178ab0d675 100644
@@ -62,6 +62,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     serial@84c0 {
Documentation/devicetree/bindings/soc/amlogic/amlogic,canvas.yaml +2 -0
index f548594d020b..cb008fd188d8 100644
@@ -40,6 +40,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     canvas: video-lut@48 {
Documentation/devicetree/bindings/sound/adi,adau7118.yaml +2 -0
index 75e0cbe6be70..76ee695097bf 100644
@@ -59,6 +59,8 @@ required:
   - iovdd-supply
   - dvdd-supply
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml +2 -0
index 38eaf0c028f9..a495d5fc0d23 100644
@@ -139,6 +139,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 examples:
   - |
     codec@1,0{
Documentation/devicetree/bindings/sound/renesas,fsi.yaml +2 -0
index 140a37fc3c0b..d1b65554e681 100644
@@ -63,6 +63,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     sh_fsi2: sound@ec230000 {
Documentation/devicetree/bindings/sound/samsung,odroid.yaml +2 -0
index c6b244352d05..8ff2d39e7d17 100644
@@ -69,6 +69,8 @@ required:
   - cpu
   - codec
 
+additionalProperties: false
+
 examples:
   - |
     sound {
Documentation/devicetree/bindings/sound/samsung-i2s.yaml +2 -0
index 53e3bad4178c..b2ad093d94df 100644
@@ -115,6 +115,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos-audss-clk.h>
Documentation/devicetree/bindings/sram/qcom,ocmem.yaml +2 -0
index 469cec133647..930188bc5e6a 100644
@@ -56,6 +56,8 @@ required:
   - '#size-cells'
   - ranges
 
+additionalProperties: false
+
 patternProperties:
   "-sram@[0-9a-f]+$":
     type: object
Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2 -0
index 93fe7b10a82e..e43ec50bda37 100644
@@ -42,6 +42,8 @@ required:
   - clocks
   - amlogic,ao-secure
 
+additionalProperties: false
+
 examples:
   - |
         cpu_temp: temperature-sensor@ff634800 {
Documentation/devicetree/bindings/timer/arm,arch_timer.yaml +2 -0
index 6deead07728e..fa255672e8e5 100644
@@ -82,6 +82,8 @@ properties:
 required:
   - compatible
 
+additionalProperties: false
+
 oneOf:
   - required:
       - interrupts
Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2 -0
index f7ef6646bade..582bbef62b95 100644
@@ -95,6 +95,8 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+additionalProperties: false
+
 examples:
   - |
     timer@f0000000 {
Documentation/devicetree/bindings/timer/arm,global_timer.yaml +2 -0
index 21c24a8e28fd..4956c8f409d2 100644
@@ -35,6 +35,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     timer@2c000600 {
Documentation/devicetree/bindings/timer/intel,ixp4xx-timer.yaml +2 -0
index 2807225db902..1a721d8af67a 100644
@@ -32,6 +32,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
(quoted diff) +0 -0

mct.yaml b/Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml index 273e359854dd..37bd01a62c52 100644

Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml +2 -0
@@ -52,6 +52,8 @@ required:
   - interrupts
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     // In this example, the IP contains two local timers, using separate
Documentation/devicetree/bindings/trivial-devices.yaml +0 -0
index 51d1f6e43c02..bcae5f9b1d7f 100644

Documentation/devicetree/bindings/trivial-devices.yaml required: · line 373

Quoted 6 lines
Documentation/devicetree/bindings/trivial-devices.yaml +2 -0
@@ -369,4 +369,6 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
...

_______________________________________________
linux-arm-kernel mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Reviewed-by on v1 4/4
#19 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

In reply to Rob Herring
Quoted 127 lines · Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

---
 .../devicetree/bindings/arm/altera/socfpga-clk-manager.yaml    | 2 ++
 .../bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/renesas,prr.yaml         | 2 ++
 .../devicetree/bindings/arm/samsung/exynos-chipid.yaml         | 2 ++
 Documentation/devicetree/bindings/arm/samsung/pmu.yaml         | 2 ++
 .../bindings/arm/samsung/samsung-secure-firmware.yaml          | 2 ++
 .../devicetree/bindings/arm/stm32/st,stm32-syscon.yaml         | 2 ++
 Documentation/devicetree/bindings/clock/fsl,plldig.yaml        | 2 ++
 Documentation/devicetree/bindings/clock/imx8mn-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/imx8mp-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/milbeaut-clock.yaml    | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,mmcc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,msm8998-gpucc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml       | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-videocc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-videocc.yaml         | 2 ++
 .../devicetree/bindings/display/amlogic,meson-vpu.yaml         | 2 ++
 .../devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml        | 2 ++
 Documentation/devicetree/bindings/dsp/fsl,dsp.yaml             | 2 ++
 Documentation/devicetree/bindings/eeprom/at24.yaml             | 2 ++
 .../firmware/intel,ixp4xx-network-processing-engine.yaml       | 3 +++
 .../devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml          | 2 ++
 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++
 Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml     | 2 ++
 Documentation/devicetree/bindings/gpu/samsung-rotator.yaml     | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml | 2 ++
 Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml         | 2 ++
 Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml  | 2 ++
 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml    | 2 ++
 .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml         | 2 ++
 .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml        | 2 ++
 .../devicetree/bindings/iio/chemical/plantower,pms7003.yaml    | 2 ++
 .../devicetree/bindings/iio/chemical/sensirion,sps30.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml    | 2 ++
 Documentation/devicetree/bindings/iio/light/adux1020.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/bh1750.yaml        | 2 ++
 Documentation/devicetree/bindings/iio/light/isl29018.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/noa1305.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/stk33xx.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2583.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2772.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/veml6030.yaml      | 2 ++
 .../devicetree/bindings/iio/pressure/asc,dlhl60d.yaml          | 2 ++
 Documentation/devicetree/bindings/iio/pressure/bmp085.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/devantech-srf04.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/parallax-ping.yaml       | 2 ++
 .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml       | 2 ++
 Documentation/devicetree/bindings/input/gpio-vibrator.yaml     | 2 ++
 Documentation/devicetree/bindings/input/max77650-onkey.yaml    | 3 +++
 .../bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml  | 2 ++
 Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml    | 2 ++
 Documentation/devicetree/bindings/leds/leds-max77650.yaml      | 3 +++
 Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml  | 3 +++
 .../devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml    | 2 ++
 Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml   | 2 ++
 .../devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml     | 2 ++
 Documentation/devicetree/bindings/media/renesas,ceu.yaml       | 2 ++
 Documentation/devicetree/bindings/mfd/max77650.yaml            | 2 ++
 Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml   | 2 ++
 .../bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml          | 2 ++
 Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml      | 2 ++
 .../devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml  | 2 ++
 .../bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml  | 2 ++
 .../devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml          | 2 ++
 .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-poweroff.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-reboot.yaml         | 2 ++
 .../devicetree/bindings/power/supply/max77650-charger.yaml     | 3 +++
 Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml           | 2 ++
 .../devicetree/bindings/regulator/max77650-regulator.yaml      | 3 +++
 .../devicetree/bindings/reset/amlogic,meson-reset.yaml         | 2 ++
 .../bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml          | 2 ++
 Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml   | 2 ++
 Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml        | 2 ++
 Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml      | 2 ++
 Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml        | 2 ++
 .../devicetree/bindings/serial/amlogic,meson-uart.yaml         | 2 ++
 .../devicetree/bindings/soc/amlogic/amlogic,canvas.yaml        | 2 ++
 Documentation/devicetree/bindings/sound/adi,adau7118.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/renesas,fsi.yaml       | 2 ++
 Documentation/devicetree/bindings/sound/samsung,odroid.yaml    | 2 ++
 Documentation/devicetree/bindings/sound/samsung-i2s.yaml       | 2 ++
 Documentation/devicetree/bindings/sram/qcom,ocmem.yaml         | 2 ++
Quoted 9 lines
 Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml | 2 ++
 Documentation/devicetree/bindings/timer/arm,arch_timer.yaml    | 2 ++
 .../devicetree/bindings/timer/arm,arch_timer_mmio.yaml         | 2 ++
 Documentation/devicetree/bindings/timer/arm,global_timer.yaml  | 2 ++
 .../devicetree/bindings/timer/intel,ixp4xx-timer.yaml          | 2 ++
 .../devicetree/bindings/timer/samsung,exynos4210-mct.yaml      | 2 ++
 Documentation/devicetree/bindings/trivial-devices.yaml         | 2 ++
 117 files changed, 240 insertions(+)

For:

 .../bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml       | 2 ++
 .../devicetree/bindings/display/amlogic,meson-vpu.yaml         | 2 ++
 .../devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml    | 2 ++
 .../devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml     | 2 ++
 Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml   | 2 ++
 .../bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml         | 2 ++
 .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml       | 2 ++
 .../devicetree/bindings/reset/amlogic,meson-reset.yaml         | 2 ++
 Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml   | 2 ++
 .../devicetree/bindings/serial/amlogic,meson-uart.yaml         | 2 ++
 .../devicetree/bindings/soc/amlogic/amlogic,canvas.yaml        | 2 ++
 Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml | 2 ++

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Acked-by on v1 4/4
#20 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

In reply to Rob Herring
Quoted 11 lines · On Wed, 25 Mar 2020 16:05:41 -0600 Rob Herring <[email protected]> wrote:

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

Quoted 5 lines

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

for IIO ones.

Thanks.

Quoted 1,633 lines
---
 .../devicetree/bindings/arm/altera/socfpga-clk-manager.yaml    | 2 ++
 .../bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/renesas,prr.yaml         | 2 ++
 .../devicetree/bindings/arm/samsung/exynos-chipid.yaml         | 2 ++
 Documentation/devicetree/bindings/arm/samsung/pmu.yaml         | 2 ++
 .../bindings/arm/samsung/samsung-secure-firmware.yaml          | 2 ++
 .../devicetree/bindings/arm/stm32/st,stm32-syscon.yaml         | 2 ++
 Documentation/devicetree/bindings/clock/fsl,plldig.yaml        | 2 ++
 Documentation/devicetree/bindings/clock/imx8mn-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/imx8mp-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/milbeaut-clock.yaml    | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,mmcc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,msm8998-gpucc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml       | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-videocc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-videocc.yaml         | 2 ++
 .../devicetree/bindings/display/amlogic,meson-vpu.yaml         | 2 ++
 .../devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml        | 2 ++
 Documentation/devicetree/bindings/dsp/fsl,dsp.yaml             | 2 ++
 Documentation/devicetree/bindings/eeprom/at24.yaml             | 2 ++
 .../firmware/intel,ixp4xx-network-processing-engine.yaml       | 3 +++
 .../devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml          | 2 ++
 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++
 Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml     | 2 ++
 Documentation/devicetree/bindings/gpu/samsung-rotator.yaml     | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml | 2 ++
 Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml         | 2 ++
 Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml  | 2 ++
 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml    | 2 ++
 .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml         | 2 ++
 .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml        | 2 ++
 .../devicetree/bindings/iio/chemical/plantower,pms7003.yaml    | 2 ++
 .../devicetree/bindings/iio/chemical/sensirion,sps30.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml    | 2 ++
 Documentation/devicetree/bindings/iio/light/adux1020.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/bh1750.yaml        | 2 ++
 Documentation/devicetree/bindings/iio/light/isl29018.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/noa1305.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/stk33xx.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2583.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2772.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/veml6030.yaml      | 2 ++
 .../devicetree/bindings/iio/pressure/asc,dlhl60d.yaml          | 2 ++
 Documentation/devicetree/bindings/iio/pressure/bmp085.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/devantech-srf04.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/parallax-ping.yaml       | 2 ++
 .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml       | 2 ++
 Documentation/devicetree/bindings/input/gpio-vibrator.yaml     | 2 ++
 Documentation/devicetree/bindings/input/max77650-onkey.yaml    | 3 +++
 .../bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml  | 2 ++
 Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml    | 2 ++
 Documentation/devicetree/bindings/leds/leds-max77650.yaml      | 3 +++
 Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml  | 3 +++
 .../devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml    | 2 ++
 Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml   | 2 ++
 .../devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml     | 2 ++
 Documentation/devicetree/bindings/media/renesas,ceu.yaml       | 2 ++
 Documentation/devicetree/bindings/mfd/max77650.yaml            | 2 ++
 Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml   | 2 ++
 .../bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml          | 2 ++
 Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml      | 2 ++
 .../devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml  | 2 ++
 .../bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml  | 2 ++
 .../devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml          | 2 ++
 .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-poweroff.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-reboot.yaml         | 2 ++
 .../devicetree/bindings/power/supply/max77650-charger.yaml     | 3 +++
 Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml           | 2 ++
 .../devicetree/bindings/regulator/max77650-regulator.yaml      | 3 +++
 .../devicetree/bindings/reset/amlogic,meson-reset.yaml         | 2 ++
 .../bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml          | 2 ++
 Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml   | 2 ++
 Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml        | 2 ++
 Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml      | 2 ++
 Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml        | 2 ++
 .../devicetree/bindings/serial/amlogic,meson-uart.yaml         | 2 ++
 .../devicetree/bindings/soc/amlogic/amlogic,canvas.yaml        | 2 ++
 Documentation/devicetree/bindings/sound/adi,adau7118.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/renesas,fsi.yaml       | 2 ++
 Documentation/devicetree/bindings/sound/samsung,odroid.yaml    | 2 ++
 Documentation/devicetree/bindings/sound/samsung-i2s.yaml       | 2 ++
 Documentation/devicetree/bindings/sram/qcom,ocmem.yaml         | 2 ++
 Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml | 2 ++
 Documentation/devicetree/bindings/timer/arm,arch_timer.yaml    | 2 ++
 .../devicetree/bindings/timer/arm,arch_timer_mmio.yaml         | 2 ++
 Documentation/devicetree/bindings/timer/arm,global_timer.yaml  | 2 ++
 .../devicetree/bindings/timer/intel,ixp4xx-timer.yaml          | 2 ++
 .../devicetree/bindings/timer/samsung,exynos4210-mct.yaml      | 2 ++
 Documentation/devicetree/bindings/trivial-devices.yaml         | 2 ++
 117 files changed, 240 insertions(+)
Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml +2 -0
index e4131fa42b26..572381306681 100644
@@ -21,6 +21,8 @@ properties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     clkmgr@ffd04000 {
Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml +2 -0
index 853d7d2b56f5..66213bd95e6e 100644
@@ -43,6 +43,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     ao-secure@140 {
Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml +2 -0
index 79902f470e4b..c3a8604dfa80 100644
@@ -43,6 +43,8 @@ required:
   - reg-names
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
Documentation/devicetree/bindings/arm/renesas,prr.yaml +2 -0
index 7f8d17f33983..dd087643a9f8 100644
@@ -27,6 +27,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     prr: chipid@ff000044 {
Documentation/devicetree/bindings/arm/samsung/exynos-chipid.yaml +2 -0
index afcd70803c12..0425d333b50d 100644
@@ -30,6 +30,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     chipid@10000000 {
Documentation/devicetree/bindings/arm/samsung/pmu.yaml +2 -0
index 73b56fc5bf58..c9651892710e 100644
@@ -89,6 +89,8 @@ required:
   - clock-names
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos5250.h>
Documentation/devicetree/bindings/arm/samsung/samsung-secure-firmware.yaml +2 -0
index 51d23b6f8a94..3d9abad3c749 100644
@@ -23,6 +23,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     firmware@203f000 {
Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml +2 -0
index 0dedf94c8578..baff80197d5a 100644
@@ -29,6 +29,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/stm32mp1-clks.h>
Documentation/devicetree/bindings/clock/fsl,plldig.yaml +2 -0
index d1c040228cf7..a203d5d498db 100644
@@ -44,6 +44,8 @@ required:
   - clocks
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Display PIXEL Clock node:
   - |
Documentation/devicetree/bindings/clock/imx8mn-clock.yaml +2 -0
index cd0b8a341321..03386b0861a2 100644
@@ -52,6 +52,8 @@ required:
   - clock-names
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock Control Module node:
   - |
Documentation/devicetree/bindings/clock/imx8mp-clock.yaml +2 -0
index 89aee63c9019..4351a1dbb4f7 100644
@@ -52,6 +52,8 @@ required:
   - clock-names
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock Control Module node:
   - |
Documentation/devicetree/bindings/clock/milbeaut-clock.yaml +2 -0
index f0b804a7f096..0e8b07710451 100644
@@ -35,6 +35,8 @@ required:
   - clocks
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock controller node:
   - |
Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml +2 -0
index 3647007f82ca..eacccc88bbf6 100644
@@ -68,6 +68,8 @@ required:
   - nvmem-cell-names
   - '#thermal-sensor-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@900000 {
Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml +2 -0
index 89c6e070e7ac..98572b4a9b60 100644
@@ -40,6 +40,8 @@ required:
   - '#clock-cells'
   - '#reset-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@1800000 {
Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml +2 -0
index 18e4e77b8cfa..5a5b2214f0ca 100644
@@ -56,6 +56,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@300000 {
Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml +2 -0
index 1d3cae980471..a0bb713929b0 100644
@@ -66,6 +66,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmcc.h>
Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml +2 -0
index 8cdece395eba..ce06f3f8c3e3 100644
@@ -40,6 +40,8 @@ required:
   - '#clock-cells'
   - '#reset-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@1800000 {
Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml +2 -0
index ee4f968e2909..a345320e0e49 100644
@@ -58,6 +58,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml +2 -0
index 888e9a708390..36f3b3668ced 100644
@@ -56,6 +56,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,gcc.yaml +2 -0
index d18f8ab9eeee..e533bb0cfd2b 100644
@@ -74,6 +74,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   # Example for GCC for MSM8960:
   - |
Documentation/devicetree/bindings/clock/qcom,mmcc.yaml +2 -0
index 85518494ce43..f684fe67db84 100644
@@ -74,6 +74,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 if:
   properties:
     compatible:
Documentation/devicetree/bindings/clock/qcom,msm8998-gpucc.yaml +2 -0
index 7d853c1a85e5..d747bb58f0a7 100644
@@ -50,6 +50,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-msm8998.h>
Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml +2 -0
index 2cd158f13bab..c9fd748b4d7c 100644
@@ -35,6 +35,8 @@ required:
   - compatible
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Example for GCC for SDM845: The below node should be defined inside
   # &apps_rsc node.
Documentation/devicetree/bindings/clock/qcom,sc7180-dispcc.yaml +2 -0
index 0429062f1585..58cdfd5924d3 100644
@@ -58,6 +58,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sc7180.h>
Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml +2 -0
index 5785192cc4be..8635e35fd3f0 100644
@@ -52,6 +52,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sc7180.h>
Documentation/devicetree/bindings/clock/qcom,sc7180-videocc.yaml +2 -0
index 31df901884ac..0071b9701960 100644
@@ -48,6 +48,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-dispcc.yaml +2 -0
index 89269ddfbdcd..ad47d747a3e4 100644
@@ -67,6 +67,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml +2 -0
index bac04f1c5d79..7a052ac5dc00 100644
@@ -52,6 +52,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-videocc.yaml +2 -0
index 9d216c0f11d4..2a6a81ab0318 100644
@@ -48,6 +48,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/display/amlogic,meson-vpu.yaml +2 -0
index d1205a6697a0..d8e573eeb5ec 100644
@@ -107,6 +107,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 examples:
   - |
     vpu: vpu@d0100000 {
Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml +2 -0
index 2ca3ddbe1ff4..e7f2ad7dab5e 100644
@@ -43,6 +43,8 @@ required:
   - interrupts
   - '#dma-cells'
 
+additionalProperties: false
+
 examples:
   - |
     dma@3000000 {
Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +2 -0
index f04870d84542..a5dc070d0ca7 100644
@@ -68,6 +68,8 @@ required:
   - mbox-names
   - memory-region
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/firmware/imx/rsrc.h>
Documentation/devicetree/bindings/eeprom/at24.yaml +2 -0
index 0f6d8db18d6c..a15787e504f0 100644
@@ -172,6 +172,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml +3 -0
index 878a2079ebb6..1bd2870c3a9c 100644
@@ -34,9 +34,12 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     npe@c8006000 {
          compatible = "intel,ixp4xx-network-processing-engine";
          reg = <0xc8006000 0x1000>, <0xc8007000 0x1000>, <0xc8008000 0x1000>;
     };
+...
Documentation/devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml +2 -0
index 64e279a4bc10..5f1ed20e43ee 100644
@@ -47,6 +47,8 @@ required:
   - "#gpio-cells"
   - gpio-controller
 
+additionalProperties: false
+
 dependencies:
   interrupt-controller: [ interrupts ]
 
Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml +2 -0
index c58ff9a94f45..1a54db04f29d 100644
@@ -64,6 +64,8 @@ required:
   - gpio-ranges
   - socionext,interrupt-ranges
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml +2 -0
index d102888c1be7..a36aec27069c 100644
@@ -49,6 +49,8 @@ required:
   - "#gpio-cells"
   - gpio-controller
 
+additionalProperties: false
+
 examples:
   - |
     logicvc: logicvc@43c00000 {
Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +2 -0
index 05fd9a404ff7..0b229a7d4a98 100644
@@ -53,6 +53,8 @@ required:
   - interrupt-names
   - clocks
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +2 -0
index 6819cde050df..0407e45eb8c4 100644
@@ -94,6 +94,8 @@ required:
   - interrupt-names
   - clocks
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml +2 -0
index afde81be3c29..f5401cc8de4a 100644
@@ -115,6 +115,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +2 -0
index f4dfa6fc724c..665c6e3b31d3 100644
@@ -36,6 +36,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     rotator@12810000 {
Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml +2 -0
index 2a9822075b36..154bee851139 100644
@@ -47,6 +47,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml +2 -0
index 6a742a51e2f9..44a63fffb4be 100644
@@ -87,6 +87,8 @@ required:
   - reg
 
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml +2 -0
index 5d42e1304202..e8feee38c76c 100644
@@ -32,6 +32,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml +2 -0
index 168235ad5d81..3f043e943668 100644
@@ -76,6 +76,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml +2 -0
index c1c6d6f223cf..8723a336229e 100644
@@ -36,6 +36,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml +2 -0
index 9acde6d2e2d9..a67ba67dab51 100644
@@ -67,6 +67,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml +2 -0
index 91ab9c842273..77605f17901c 100644
@@ -53,6 +53,8 @@ required:
   - dout-gpios
   - avdd-supply
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml +2 -0
index 59009997dca0..118809a03279 100644
@@ -32,6 +32,8 @@ required:
   - vref-supply
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml +2 -0
index 0ce290473fb0..8ffeceb6abae 100644
@@ -52,6 +52,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +2 -0
index acf36eef728b..b1627441a0b2 100644
@@ -69,6 +69,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 patternProperties:
   "^filter@[0-9]+$":
     type: object
Documentation/devicetree/bindings/iio/chemical/plantower,pms7003.yaml +2 -0
index 19e53930ebf6..1fe561574019 100644
@@ -38,6 +38,8 @@ required:
   - compatible
   - vcc-supply
 
+additionalProperties: false
+
 examples:
   - |
     serial {
Documentation/devicetree/bindings/iio/chemical/sensirion,sps30.yaml +2 -0
index 50a50a0d7070..a93d1972a5c2 100644
@@ -24,6 +24,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml +2 -0
index a285eaba7125..e51a585bd5a3 100644
@@ -34,6 +34,8 @@ required:
   - reg
   - vref-supply
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/light/adux1020.yaml +2 -0
index 69bd5c06319d..d7d14f2f1c20 100644
@@ -28,6 +28,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/bh1750.yaml +2 -0
index 1cc60d7ecfa0..1a88b3c253d5 100644
@@ -28,6 +28,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/isl29018.yaml +2 -0
index cbb00be8f359..0ea278b07d1c 100644
@@ -38,6 +38,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/noa1305.yaml +2 -0
index 17e7f140b69b..fe7bfe1adbda 100644
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/stk33xx.yaml +2 -0
index aae8a6d627c9..f92bf7b2b7f0 100644
@@ -30,6 +30,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/tsl2583.yaml +2 -0
index e86ef64ecf03..7b92ba8cbb9f 100644
@@ -32,6 +32,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/tsl2772.yaml +2 -0
index ed2c3d5eadf5..e8f7d1ada57b 100644
@@ -62,6 +62,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/veml6030.yaml +2 -0
index 0ff9b11f9d18..fb19a2d7a849 100644
@@ -45,6 +45,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/pressure/asc,dlhl60d.yaml +2 -0
index 9f5ca9c42025..64c18f1693f0 100644
@@ -33,6 +33,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/pressure/bmp085.yaml +2 -0
index 5d4aec0e0d24..49257f9251e8 100644
@@ -52,6 +52,8 @@ required:
   - vddd-supply
   - vdda-supply
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml +2 -0
index 4e80ea7c1475..7ac5eb7560e0 100644
@@ -56,6 +56,8 @@ required:
   - trig-gpios
   - echo-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/proximity/parallax-ping.yaml +2 -0
index a079c9921af6..ada55f186f3c 100644
@@ -42,6 +42,8 @@ required:
   - compatible
   - ping-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml +2 -0
index d4922f9f0376..acc030c1b20e 100644
@@ -398,6 +398,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/input/gpio-vibrator.yaml +2 -0
index b98bf9363c8f..2384465eaa19 100644
@@ -26,6 +26,8 @@ required:
   - compatible
   - enable-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/input/max77650-onkey.yaml +3 -0
index 2f2e0b6ebbbd..3a2ad6ec64db 100644
@@ -33,3 +33,6 @@ properties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml +2 -0
index 507c141ea760..ccc507f384d2 100644
@@ -44,6 +44,8 @@ required:
   - interrupt-controller
   - '#interrupt-cells'
 
+additionalProperties: false
+
 examples:
   - |
     intcon: interrupt-controller@c8003000 {
Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml +2 -0
index 7cdd3aaa2ba4..0e33cd9e010e 100644
@@ -80,6 +80,8 @@ required:
   - clock-names
   - "#iommu-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos5250.h>
Documentation/devicetree/bindings/leds/leds-max77650.yaml +3 -0
index 8c43f1e1bf7d..c6f96cabd4d1 100644
@@ -49,3 +49,6 @@ required:
   - compatible
   - "#address-cells"
   - "#size-cells"
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml +3 -0
index b50f4bcc98f1..90edf9d33b33 100644
@@ -50,3 +50,6 @@ patternProperties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml +2 -0
index 319280563648..aa2b3bf56b57 100644
@@ -41,6 +41,8 @@ required:
   - interrupts
   - "#mbox-cells"
 
+additionalProperties: false
+
 examples:
   - |
     mailbox@c883c404 {
Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml +2 -0
index 335717e15970..37d77e065491 100644
@@ -124,6 +124,8 @@ required:
   - amlogic,ao-sysctrl
   - amlogic,canvas
 
+additionalProperties: false
+
 examples:
   - |
     vdec: video-decoder@c8820000 {
Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml +2 -0
index e8ce37fcbfec..95ffa8bc0533 100644
@@ -82,6 +82,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     cec_AO: cec@100 {
Documentation/devicetree/bindings/media/renesas,ceu.yaml +2 -0
index 8e9251a0f9ef..fcb5f13704a5 100644
@@ -59,6 +59,8 @@ required:
   - interrupts
   - port
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
Documentation/devicetree/bindings/mfd/max77650.yaml +2 -0
index 480385789394..b0a0f0d3d9d4 100644
@@ -73,6 +73,8 @@ required:
   - gpio-controller
   - "#gpio-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml +2 -0
index 38dc4f8b0ceb..3a6a1a26e2b3 100644
@@ -77,6 +77,8 @@ required:
   - gpio-controller
   - "#gpio-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml +2 -0
index 0ea21a6f70b4..38ab0499102d 100644
@@ -38,6 +38,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +2 -0
index b9e9696da5be..976f139bb66e 100644
@@ -167,6 +167,8 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml +2 -0
index e5922b427342..c03b83103e87 100644
@@ -34,6 +34,8 @@ required:
   - resets
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/sun50i-h6-ccu.h>
Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml +2 -0
index 346f9c35427c..453c083cf44c 100644
@@ -44,6 +44,8 @@ required:
   - reset-names
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     phy@46000 {
Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml +2 -0
index 0ccee64c6962..9a346d6290d9 100644
@@ -40,6 +40,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     sysconf: chiptop@e0200000 {
Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +2 -0
index 5ab436189f3b..00609ace677c 100644
@@ -31,6 +31,8 @@ required:
   - reset-gpios
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml +2 -0
index 452cee1aed32..fd1982c56104 100644
@@ -159,6 +159,8 @@ required:
   - "#reset-cells"
   - ranges
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/soc/ti,sci_pm_domain.h>
Documentation/devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml +2 -0
index 135c7dfbc180..7651a675ab2d 100644
@@ -57,6 +57,8 @@ patternProperties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     syscon: scu@1e6e2000 {
Documentation/devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml +2 -0
index 824f7fd1d51b..36feaf5e2dff 100644
@@ -70,6 +70,8 @@ required:
   - compatible
   - aspeed,external-nodes
 
+additionalProperties: false
+
 examples:
   - |
     apb {
Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml +2 -0
index ac8d1c30a8ed..45af29bc3202 100644
@@ -92,6 +92,8 @@ patternProperties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     syscon: scu@1e6e2000 {
Documentation/devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml +2 -0
index ef4de32cb17c..46a0478cb924 100644
@@ -194,6 +194,8 @@ required:
   - ranges
   - pins-are-numbered
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/pinctrl/stm32-pinfunc.h>
Documentation/devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml +2 -0
index d3098c924b25..6c6079fe1351 100644
@@ -68,6 +68,8 @@ required:
   - "#power-domain-cells"
   - amlogic,ao-sysctrl
 
+additionalProperties: false
+
 examples:
   - |
     pwrc: power-controller {
Documentation/devicetree/bindings/power/reset/syscon-poweroff.yaml +2 -0
index 520e07e6f21b..3412fe7e1e80 100644
@@ -41,6 +41,8 @@ required:
   - regmap
   - offset
 
+additionalProperties: false
+
 allOf:
   - if:
       not:
Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +2 -0
index d38006b1f1f4..b80772cb9f06 100644
@@ -41,6 +41,8 @@ required:
   - regmap
   - offset
 
+additionalProperties: false
+
 allOf:
   - if:
       not:
Documentation/devicetree/bindings/power/supply/max77650-charger.yaml +3 -0
index deef010ec535..62eeddb65aed 100644
@@ -32,3 +32,6 @@ properties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml +2 -0
index 9e21b83d717e..239b49fad805 100644
@@ -55,6 +55,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c@1 {
Documentation/devicetree/bindings/regulator/max77650-regulator.yaml +3 -0
index 50690487edc8..ce0a4021ae7f 100644
@@ -29,3 +29,6 @@ patternProperties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/reset/amlogic,meson-reset.yaml +2 -0
index b3f57d81f007..92922d3afd14 100644
@@ -29,6 +29,8 @@ required:
   - reg
   - "#reset-cells"
 
+additionalProperties: false
+
 examples:
   - |
     reset-controller@c884404 {
Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml +2 -0
index 411bd76f1b64..512a33bdb208 100644
@@ -28,6 +28,8 @@ required:
   - reg
   - "#reset-cells"
 
+additionalProperties: false
+
 examples:
   - |
     reset-controller@8b2c800 {
Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml +2 -0
index a9ff3cb35c5e..444be32a8a29 100644
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     rng@c8834000 {
Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml +2 -0
index 42d9a38e4e1a..89ab67f20a7f 100644
@@ -35,6 +35,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     rng {
Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml +2 -0
index dcff573cbdb1..b95cb017f469 100644
@@ -51,6 +51,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/r7s72100-clock.h>
Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml +2 -0
index 0a54296d7218..48c6cafca90c 100644
@@ -111,6 +111,8 @@ required:
   - clocks
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/mfd/stm32f4-rcc.h>
Documentation/devicetree/bindings/serial/amlogic,meson-uart.yaml +2 -0
index 214fe8beddc3..d4178ab0d675 100644
@@ -62,6 +62,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     serial@84c0 {
Documentation/devicetree/bindings/soc/amlogic/amlogic,canvas.yaml +2 -0
index f548594d020b..cb008fd188d8 100644
@@ -40,6 +40,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     canvas: video-lut@48 {
Documentation/devicetree/bindings/sound/adi,adau7118.yaml +2 -0
index 75e0cbe6be70..76ee695097bf 100644
@@ -59,6 +59,8 @@ required:
   - iovdd-supply
   - dvdd-supply
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml +2 -0
index 38eaf0c028f9..a495d5fc0d23 100644
@@ -139,6 +139,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 examples:
   - |
     codec@1,0{
Documentation/devicetree/bindings/sound/renesas,fsi.yaml +2 -0
index 140a37fc3c0b..d1b65554e681 100644
@@ -63,6 +63,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     sh_fsi2: sound@ec230000 {
Documentation/devicetree/bindings/sound/samsung,odroid.yaml +2 -0
index c6b244352d05..8ff2d39e7d17 100644
@@ -69,6 +69,8 @@ required:
   - cpu
   - codec
 
+additionalProperties: false
+
 examples:
   - |
     sound {
Documentation/devicetree/bindings/sound/samsung-i2s.yaml +2 -0
index 53e3bad4178c..b2ad093d94df 100644
@@ -115,6 +115,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos-audss-clk.h>
Documentation/devicetree/bindings/sram/qcom,ocmem.yaml +2 -0
index 469cec133647..930188bc5e6a 100644
@@ -56,6 +56,8 @@ required:
   - '#size-cells'
   - ranges
 
+additionalProperties: false
+
 patternProperties:
   "-sram@[0-9a-f]+$":
     type: object
Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2 -0
index 93fe7b10a82e..e43ec50bda37 100644
@@ -42,6 +42,8 @@ required:
   - clocks
   - amlogic,ao-secure
 
+additionalProperties: false
+
 examples:
   - |
         cpu_temp: temperature-sensor@ff634800 {
Documentation/devicetree/bindings/timer/arm,arch_timer.yaml +2 -0
index 6deead07728e..fa255672e8e5 100644
@@ -82,6 +82,8 @@ properties:
 required:
   - compatible
 
+additionalProperties: false
+
 oneOf:
   - required:
       - interrupts
Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2 -0
index f7ef6646bade..582bbef62b95 100644
@@ -95,6 +95,8 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+additionalProperties: false
+
 examples:
   - |
     timer@f0000000 {
Documentation/devicetree/bindings/timer/arm,global_timer.yaml +2 -0
index 21c24a8e28fd..4956c8f409d2 100644
@@ -35,6 +35,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     timer@2c000600 {
Documentation/devicetree/bindings/timer/intel,ixp4xx-timer.yaml +2 -0
index 2807225db902..1a721d8af67a 100644
@@ -32,6 +32,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml +2 -0
index 273e359854dd..37bd01a62c52 100644
@@ -52,6 +52,8 @@ required:
   - interrupts
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     // In this example, the IP contains two local timers, using separate
Documentation/devicetree/bindings/trivial-devices.yaml +2 -0
index 51d1f6e43c02..bcae5f9b1d7f 100644
@@ -369,4 +369,6 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
...
#21 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

In reply to Rob Herring
Quoted 12 lines · Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

Quoted 3 lines

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

for stm32 bindings:
Reviewed-by Benjamin Gaignard <[email protected]>

Quoted 1,642 lines
---
 .../devicetree/bindings/arm/altera/socfpga-clk-manager.yaml    | 2 ++
 .../bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/renesas,prr.yaml         | 2 ++
 .../devicetree/bindings/arm/samsung/exynos-chipid.yaml         | 2 ++
 Documentation/devicetree/bindings/arm/samsung/pmu.yaml         | 2 ++
 .../bindings/arm/samsung/samsung-secure-firmware.yaml          | 2 ++
 .../devicetree/bindings/arm/stm32/st,stm32-syscon.yaml         | 2 ++
 Documentation/devicetree/bindings/clock/fsl,plldig.yaml        | 2 ++
 Documentation/devicetree/bindings/clock/imx8mn-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/imx8mp-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/milbeaut-clock.yaml    | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,mmcc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,msm8998-gpucc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml       | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-videocc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-videocc.yaml         | 2 ++
 .../devicetree/bindings/display/amlogic,meson-vpu.yaml         | 2 ++
 .../devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml        | 2 ++
 Documentation/devicetree/bindings/dsp/fsl,dsp.yaml             | 2 ++
 Documentation/devicetree/bindings/eeprom/at24.yaml             | 2 ++
 .../firmware/intel,ixp4xx-network-processing-engine.yaml       | 3 +++
 .../devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml          | 2 ++
 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++
 Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml     | 2 ++
 Documentation/devicetree/bindings/gpu/samsung-rotator.yaml     | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml | 2 ++
 Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml         | 2 ++
 Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml  | 2 ++
 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml    | 2 ++
 .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml         | 2 ++
 .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml        | 2 ++
 .../devicetree/bindings/iio/chemical/plantower,pms7003.yaml    | 2 ++
 .../devicetree/bindings/iio/chemical/sensirion,sps30.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml    | 2 ++
 Documentation/devicetree/bindings/iio/light/adux1020.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/bh1750.yaml        | 2 ++
 Documentation/devicetree/bindings/iio/light/isl29018.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/noa1305.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/stk33xx.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2583.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2772.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/veml6030.yaml      | 2 ++
 .../devicetree/bindings/iio/pressure/asc,dlhl60d.yaml          | 2 ++
 Documentation/devicetree/bindings/iio/pressure/bmp085.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/devantech-srf04.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/parallax-ping.yaml       | 2 ++
 .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml       | 2 ++
 Documentation/devicetree/bindings/input/gpio-vibrator.yaml     | 2 ++
 Documentation/devicetree/bindings/input/max77650-onkey.yaml    | 3 +++
 .../bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml  | 2 ++
 Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml    | 2 ++
 Documentation/devicetree/bindings/leds/leds-max77650.yaml      | 3 +++
 Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml  | 3 +++
 .../devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml    | 2 ++
 Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml   | 2 ++
 .../devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml     | 2 ++
 Documentation/devicetree/bindings/media/renesas,ceu.yaml       | 2 ++
 Documentation/devicetree/bindings/mfd/max77650.yaml            | 2 ++
 Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml   | 2 ++
 .../bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml          | 2 ++
 Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml      | 2 ++
 .../devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml  | 2 ++
 .../bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml  | 2 ++
 .../devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml         | 2 ++
 Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml    | 2 ++
 .../devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml          | 2 ++
 .../devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-poweroff.yaml       | 2 ++
 .../devicetree/bindings/power/reset/syscon-reboot.yaml         | 2 ++
 .../devicetree/bindings/power/supply/max77650-charger.yaml     | 3 +++
 Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml           | 2 ++
 .../devicetree/bindings/regulator/max77650-regulator.yaml      | 3 +++
 .../devicetree/bindings/reset/amlogic,meson-reset.yaml         | 2 ++
 .../bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml          | 2 ++
 Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml   | 2 ++
 Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml        | 2 ++
 Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml      | 2 ++
 Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml        | 2 ++
 .../devicetree/bindings/serial/amlogic,meson-uart.yaml         | 2 ++
 .../devicetree/bindings/soc/amlogic/amlogic,canvas.yaml        | 2 ++
 Documentation/devicetree/bindings/sound/adi,adau7118.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml      | 2 ++
 Documentation/devicetree/bindings/sound/renesas,fsi.yaml       | 2 ++
 Documentation/devicetree/bindings/sound/samsung,odroid.yaml    | 2 ++
 Documentation/devicetree/bindings/sound/samsung-i2s.yaml       | 2 ++
 Documentation/devicetree/bindings/sram/qcom,ocmem.yaml         | 2 ++
 Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml | 2 ++
 Documentation/devicetree/bindings/timer/arm,arch_timer.yaml    | 2 ++
 .../devicetree/bindings/timer/arm,arch_timer_mmio.yaml         | 2 ++
 Documentation/devicetree/bindings/timer/arm,global_timer.yaml  | 2 ++
 .../devicetree/bindings/timer/intel,ixp4xx-timer.yaml          | 2 ++
 .../devicetree/bindings/timer/samsung,exynos4210-mct.yaml      | 2 ++
 Documentation/devicetree/bindings/trivial-devices.yaml         | 2 ++
 117 files changed, 240 insertions(+)
Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml +2 -0
index e4131fa42b26..572381306681 100644
@@ -21,6 +21,8 @@ properties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     clkmgr@ffd04000 {
Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml +2 -0
index 853d7d2b56f5..66213bd95e6e 100644
@@ -43,6 +43,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     ao-secure@140 {
Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml +2 -0
index 79902f470e4b..c3a8604dfa80 100644
@@ -43,6 +43,8 @@ required:
   - reg-names
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
Documentation/devicetree/bindings/arm/renesas,prr.yaml +2 -0
index 7f8d17f33983..dd087643a9f8 100644
@@ -27,6 +27,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     prr: chipid@ff000044 {
Documentation/devicetree/bindings/arm/samsung/exynos-chipid.yaml +2 -0
index afcd70803c12..0425d333b50d 100644
@@ -30,6 +30,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     chipid@10000000 {
Documentation/devicetree/bindings/arm/samsung/pmu.yaml +2 -0
index 73b56fc5bf58..c9651892710e 100644
@@ -89,6 +89,8 @@ required:
   - clock-names
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos5250.h>
Documentation/devicetree/bindings/arm/samsung/samsung-secure-firmware.yaml +2 -0
index 51d23b6f8a94..3d9abad3c749 100644
@@ -23,6 +23,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     firmware@203f000 {
Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml +2 -0
index 0dedf94c8578..baff80197d5a 100644
@@ -29,6 +29,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/stm32mp1-clks.h>
Documentation/devicetree/bindings/clock/fsl,plldig.yaml +2 -0
index d1c040228cf7..a203d5d498db 100644
@@ -44,6 +44,8 @@ required:
   - clocks
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Display PIXEL Clock node:
   - |
Documentation/devicetree/bindings/clock/imx8mn-clock.yaml +2 -0
index cd0b8a341321..03386b0861a2 100644
@@ -52,6 +52,8 @@ required:
   - clock-names
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock Control Module node:
   - |
Documentation/devicetree/bindings/clock/imx8mp-clock.yaml +2 -0
index 89aee63c9019..4351a1dbb4f7 100644
@@ -52,6 +52,8 @@ required:
   - clock-names
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock Control Module node:
   - |
Documentation/devicetree/bindings/clock/milbeaut-clock.yaml +2 -0
index f0b804a7f096..0e8b07710451 100644
@@ -35,6 +35,8 @@ required:
   - clocks
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Clock controller node:
   - |
Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml +2 -0
index 3647007f82ca..eacccc88bbf6 100644
@@ -68,6 +68,8 @@ required:
   - nvmem-cell-names
   - '#thermal-sensor-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@900000 {
Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml +2 -0
index 89c6e070e7ac..98572b4a9b60 100644
@@ -40,6 +40,8 @@ required:
   - '#clock-cells'
   - '#reset-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@1800000 {
Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml +2 -0
index 18e4e77b8cfa..5a5b2214f0ca 100644
@@ -56,6 +56,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@300000 {
Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml +2 -0
index 1d3cae980471..a0bb713929b0 100644
@@ -66,6 +66,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmcc.h>
Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml +2 -0
index 8cdece395eba..ce06f3f8c3e3 100644
@@ -40,6 +40,8 @@ required:
   - '#clock-cells'
   - '#reset-cells'
 
+additionalProperties: false
+
 examples:
   - |
     clock-controller@1800000 {
Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml +2 -0
index ee4f968e2909..a345320e0e49 100644
@@ -58,6 +58,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml +2 -0
index 888e9a708390..36f3b3668ced 100644
@@ -56,6 +56,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,gcc.yaml +2 -0
index d18f8ab9eeee..e533bb0cfd2b 100644
@@ -74,6 +74,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   # Example for GCC for MSM8960:
   - |
Documentation/devicetree/bindings/clock/qcom,mmcc.yaml +2 -0
index 85518494ce43..f684fe67db84 100644
@@ -74,6 +74,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 if:
   properties:
     compatible:
Documentation/devicetree/bindings/clock/qcom,msm8998-gpucc.yaml +2 -0
index 7d853c1a85e5..d747bb58f0a7 100644
@@ -50,6 +50,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-msm8998.h>
Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml +2 -0
index 2cd158f13bab..c9fd748b4d7c 100644
@@ -35,6 +35,8 @@ required:
   - compatible
   - '#clock-cells'
 
+additionalProperties: false
+
 examples:
   # Example for GCC for SDM845: The below node should be defined inside
   # &apps_rsc node.
Documentation/devicetree/bindings/clock/qcom,sc7180-dispcc.yaml +2 -0
index 0429062f1585..58cdfd5924d3 100644
@@ -58,6 +58,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sc7180.h>
Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml +2 -0
index 5785192cc4be..8635e35fd3f0 100644
@@ -52,6 +52,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sc7180.h>
Documentation/devicetree/bindings/clock/qcom,sc7180-videocc.yaml +2 -0
index 31df901884ac..0071b9701960 100644
@@ -48,6 +48,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-dispcc.yaml +2 -0
index 89269ddfbdcd..ad47d747a3e4 100644
@@ -67,6 +67,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml +2 -0
index bac04f1c5d79..7a052ac5dc00 100644
@@ -52,6 +52,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,gcc-sdm845.h>
Documentation/devicetree/bindings/clock/qcom,sdm845-videocc.yaml +2 -0
index 9d216c0f11d4..2a6a81ab0318 100644
@@ -48,6 +48,8 @@ required:
   - '#reset-cells'
   - '#power-domain-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/qcom,rpmh.h>
Documentation/devicetree/bindings/display/amlogic,meson-vpu.yaml +2 -0
index d1205a6697a0..d8e573eeb5ec 100644
@@ -107,6 +107,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 examples:
   - |
     vpu: vpu@d0100000 {
Documentation/devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml +2 -0
index 2ca3ddbe1ff4..e7f2ad7dab5e 100644
@@ -43,6 +43,8 @@ required:
   - interrupts
   - '#dma-cells'
 
+additionalProperties: false
+
 examples:
   - |
     dma@3000000 {
Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +2 -0
index f04870d84542..a5dc070d0ca7 100644
@@ -68,6 +68,8 @@ required:
   - mbox-names
   - memory-region
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/firmware/imx/rsrc.h>
Documentation/devicetree/bindings/eeprom/at24.yaml +2 -0
index 0f6d8db18d6c..a15787e504f0 100644
@@ -172,6 +172,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml +3 -0
index 878a2079ebb6..1bd2870c3a9c 100644
@@ -34,9 +34,12 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     npe@c8006000 {
          compatible = "intel,ixp4xx-network-processing-engine";
          reg = <0xc8006000 0x1000>, <0xc8007000 0x1000>, <0xc8008000 0x1000>;
     };
+...
Documentation/devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml +2 -0
index 64e279a4bc10..5f1ed20e43ee 100644
@@ -47,6 +47,8 @@ required:
   - "#gpio-cells"
   - gpio-controller
 
+additionalProperties: false
+
 dependencies:
   interrupt-controller: [ interrupts ]
 
Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml +2 -0
index c58ff9a94f45..1a54db04f29d 100644
@@ -64,6 +64,8 @@ required:
   - gpio-ranges
   - socionext,interrupt-ranges
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml +2 -0
index d102888c1be7..a36aec27069c 100644
@@ -49,6 +49,8 @@ required:
   - "#gpio-cells"
   - gpio-controller
 
+additionalProperties: false
+
 examples:
   - |
     logicvc: logicvc@43c00000 {
Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +2 -0
index 05fd9a404ff7..0b229a7d4a98 100644
@@ -53,6 +53,8 @@ required:
   - interrupt-names
   - clocks
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +2 -0
index 6819cde050df..0407e45eb8c4 100644
@@ -94,6 +94,8 @@ required:
   - interrupt-names
   - clocks
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml +2 -0
index afde81be3c29..f5401cc8de4a 100644
@@ -115,6 +115,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 allOf:
   - if:
       properties:
Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +2 -0
index f4dfa6fc724c..665c6e3b31d3 100644
@@ -36,6 +36,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     rotator@12810000 {
Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml +2 -0
index 2a9822075b36..154bee851139 100644
@@ -47,6 +47,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml +2 -0
index 6a742a51e2f9..44a63fffb4be 100644
@@ -87,6 +87,8 @@ required:
   - reg
 
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml +2 -0
index 5d42e1304202..e8feee38c76c 100644
@@ -32,6 +32,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml +2 -0
index 168235ad5d81..3f043e943668 100644
@@ -76,6 +76,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml +2 -0
index c1c6d6f223cf..8723a336229e 100644
@@ -36,6 +36,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml +2 -0
index 9acde6d2e2d9..a67ba67dab51 100644
@@ -67,6 +67,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml +2 -0
index 91ab9c842273..77605f17901c 100644
@@ -53,6 +53,8 @@ required:
   - dout-gpios
   - avdd-supply
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml +2 -0
index 59009997dca0..118809a03279 100644
@@ -32,6 +32,8 @@ required:
   - vref-supply
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml +2 -0
index 0ce290473fb0..8ffeceb6abae 100644
@@ -52,6 +52,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +2 -0
index acf36eef728b..b1627441a0b2 100644
@@ -69,6 +69,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 patternProperties:
   "^filter@[0-9]+$":
     type: object
Documentation/devicetree/bindings/iio/chemical/plantower,pms7003.yaml +2 -0
index 19e53930ebf6..1fe561574019 100644
@@ -38,6 +38,8 @@ required:
   - compatible
   - vcc-supply
 
+additionalProperties: false
+
 examples:
   - |
     serial {
Documentation/devicetree/bindings/iio/chemical/sensirion,sps30.yaml +2 -0
index 50a50a0d7070..a93d1972a5c2 100644
@@ -24,6 +24,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml +2 -0
index a285eaba7125..e51a585bd5a3 100644
@@ -34,6 +34,8 @@ required:
   - reg
   - vref-supply
 
+additionalProperties: false
+
 examples:
   - |
     spi {
Documentation/devicetree/bindings/iio/light/adux1020.yaml +2 -0
index 69bd5c06319d..d7d14f2f1c20 100644
@@ -28,6 +28,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/bh1750.yaml +2 -0
index 1cc60d7ecfa0..1a88b3c253d5 100644
@@ -28,6 +28,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/isl29018.yaml +2 -0
index cbb00be8f359..0ea278b07d1c 100644
@@ -38,6 +38,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/noa1305.yaml +2 -0
index 17e7f140b69b..fe7bfe1adbda 100644
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/stk33xx.yaml +2 -0
index aae8a6d627c9..f92bf7b2b7f0 100644
@@ -30,6 +30,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/tsl2583.yaml +2 -0
index e86ef64ecf03..7b92ba8cbb9f 100644
@@ -32,6 +32,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/iio/light/tsl2772.yaml +2 -0
index ed2c3d5eadf5..e8f7d1ada57b 100644
@@ -62,6 +62,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/light/veml6030.yaml +2 -0
index 0ff9b11f9d18..fb19a2d7a849 100644
@@ -45,6 +45,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/pressure/asc,dlhl60d.yaml +2 -0
index 9f5ca9c42025..64c18f1693f0 100644
@@ -33,6 +33,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/iio/pressure/bmp085.yaml +2 -0
index 5d4aec0e0d24..49257f9251e8 100644
@@ -52,6 +52,8 @@ required:
   - vddd-supply
   - vdda-supply
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml +2 -0
index 4e80ea7c1475..7ac5eb7560e0 100644
@@ -56,6 +56,8 @@ required:
   - trig-gpios
   - echo-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/proximity/parallax-ping.yaml +2 -0
index a079c9921af6..ada55f186f3c 100644
@@ -42,6 +42,8 @@ required:
   - compatible
   - ping-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml +2 -0
index d4922f9f0376..acc030c1b20e 100644
@@ -398,6 +398,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/input/gpio-vibrator.yaml +2 -0
index b98bf9363c8f..2384465eaa19 100644
@@ -26,6 +26,8 @@ required:
   - compatible
   - enable-gpios
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/input/max77650-onkey.yaml +3 -0
index 2f2e0b6ebbbd..3a2ad6ec64db 100644
@@ -33,3 +33,6 @@ properties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml +2 -0
index 507c141ea760..ccc507f384d2 100644
@@ -44,6 +44,8 @@ required:
   - interrupt-controller
   - '#interrupt-cells'
 
+additionalProperties: false
+
 examples:
   - |
     intcon: interrupt-controller@c8003000 {
Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml +2 -0
index 7cdd3aaa2ba4..0e33cd9e010e 100644
@@ -80,6 +80,8 @@ required:
   - clock-names
   - "#iommu-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos5250.h>
Documentation/devicetree/bindings/leds/leds-max77650.yaml +3 -0
index 8c43f1e1bf7d..c6f96cabd4d1 100644
@@ -49,3 +49,6 @@ required:
   - compatible
   - "#address-cells"
   - "#size-cells"
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml +3 -0
index b50f4bcc98f1..90edf9d33b33 100644
@@ -50,3 +50,6 @@ patternProperties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml +2 -0
index 319280563648..aa2b3bf56b57 100644
@@ -41,6 +41,8 @@ required:
   - interrupts
   - "#mbox-cells"
 
+additionalProperties: false
+
 examples:
   - |
     mailbox@c883c404 {
Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml +2 -0
index 335717e15970..37d77e065491 100644
@@ -124,6 +124,8 @@ required:
   - amlogic,ao-sysctrl
   - amlogic,canvas
 
+additionalProperties: false
+
 examples:
   - |
     vdec: video-decoder@c8820000 {
Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml +2 -0
index e8ce37fcbfec..95ffa8bc0533 100644
@@ -82,6 +82,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     cec_AO: cec@100 {
Documentation/devicetree/bindings/media/renesas,ceu.yaml +2 -0
index 8e9251a0f9ef..fcb5f13704a5 100644
@@ -59,6 +59,8 @@ required:
   - interrupts
   - port
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/arm-gic.h>
Documentation/devicetree/bindings/mfd/max77650.yaml +2 -0
index 480385789394..b0a0f0d3d9d4 100644
@@ -73,6 +73,8 @@ required:
   - gpio-controller
   - "#gpio-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml +2 -0
index 38dc4f8b0ceb..3a6a1a26e2b3 100644
@@ -77,6 +77,8 @@ required:
   - gpio-controller
   - "#gpio-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml +2 -0
index 0ea21a6f70b4..38ab0499102d 100644
@@ -38,6 +38,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +2 -0
index b9e9696da5be..976f139bb66e 100644
@@ -167,6 +167,8 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml +2 -0
index e5922b427342..c03b83103e87 100644
@@ -34,6 +34,8 @@ required:
   - resets
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/sun50i-h6-ccu.h>
Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml +2 -0
index 346f9c35427c..453c083cf44c 100644
@@ -44,6 +44,8 @@ required:
   - reset-names
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     phy@46000 {
Documentation/devicetree/bindings/phy/intel,lgm-emmc-phy.yaml +2 -0
index 0ccee64c6962..9a346d6290d9 100644
@@ -40,6 +40,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     sysconf: chiptop@e0200000 {
Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +2 -0
index 5ab436189f3b..00609ace677c 100644
@@ -31,6 +31,8 @@ required:
   - reset-gpios
   - "#phy-cells"
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/gpio/gpio.h>
Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml +2 -0
index 452cee1aed32..fd1982c56104 100644
@@ -159,6 +159,8 @@ required:
   - "#reset-cells"
   - ranges
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/soc/ti,sci_pm_domain.h>
Documentation/devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml +2 -0
index 135c7dfbc180..7651a675ab2d 100644
@@ -57,6 +57,8 @@ patternProperties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     syscon: scu@1e6e2000 {
Documentation/devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml +2 -0
index 824f7fd1d51b..36feaf5e2dff 100644
@@ -70,6 +70,8 @@ required:
   - compatible
   - aspeed,external-nodes
 
+additionalProperties: false
+
 examples:
   - |
     apb {
Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml +2 -0
index ac8d1c30a8ed..45af29bc3202 100644
@@ -92,6 +92,8 @@ patternProperties:
 required:
   - compatible
 
+additionalProperties: false
+
 examples:
   - |
     syscon: scu@1e6e2000 {
Documentation/devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml +2 -0
index ef4de32cb17c..46a0478cb924 100644
@@ -194,6 +194,8 @@ required:
   - ranges
   - pins-are-numbered
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/pinctrl/stm32-pinfunc.h>
Documentation/devicetree/bindings/power/amlogic,meson-ee-pwrc.yaml +2 -0
index d3098c924b25..6c6079fe1351 100644
@@ -68,6 +68,8 @@ required:
   - "#power-domain-cells"
   - amlogic,ao-sysctrl
 
+additionalProperties: false
+
 examples:
   - |
     pwrc: power-controller {
Documentation/devicetree/bindings/power/reset/syscon-poweroff.yaml +2 -0
index 520e07e6f21b..3412fe7e1e80 100644
@@ -41,6 +41,8 @@ required:
   - regmap
   - offset
 
+additionalProperties: false
+
 allOf:
   - if:
       not:
Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +2 -0
index d38006b1f1f4..b80772cb9f06 100644
@@ -41,6 +41,8 @@ required:
   - regmap
   - offset
 
+additionalProperties: false
+
 allOf:
   - if:
       not:
Documentation/devicetree/bindings/power/supply/max77650-charger.yaml +3 -0
index deef010ec535..62eeddb65aed 100644
@@ -32,3 +32,6 @@ properties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/ptp/ptp-idtcm.yaml +2 -0
index 9e21b83d717e..239b49fad805 100644
@@ -55,6 +55,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     i2c@1 {
Documentation/devicetree/bindings/regulator/max77650-regulator.yaml +3 -0
index 50690487edc8..ce0a4021ae7f 100644
@@ -29,3 +29,6 @@ patternProperties:
 
 required:
   - compatible
+additionalProperties: false
+
+...
Documentation/devicetree/bindings/reset/amlogic,meson-reset.yaml +2 -0
index b3f57d81f007..92922d3afd14 100644
@@ -29,6 +29,8 @@ required:
   - reg
   - "#reset-cells"
 
+additionalProperties: false
+
 examples:
   - |
     reset-controller@c884404 {
Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml +2 -0
index 411bd76f1b64..512a33bdb208 100644
@@ -28,6 +28,8 @@ required:
   - reg
   - "#reset-cells"
 
+additionalProperties: false
+
 examples:
   - |
     reset-controller@8b2c800 {
Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml +2 -0
index a9ff3cb35c5e..444be32a8a29 100644
@@ -29,6 +29,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     rng@c8834000 {
Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml +2 -0
index 42d9a38e4e1a..89ab67f20a7f 100644
@@ -35,6 +35,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     rng {
Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml +2 -0
index dcff573cbdb1..b95cb017f469 100644
@@ -51,6 +51,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/r7s72100-clock.h>
Documentation/devicetree/bindings/rtc/st,stm32-rtc.yaml +2 -0
index 0a54296d7218..48c6cafca90c 100644
@@ -111,6 +111,8 @@ required:
   - clocks
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/mfd/stm32f4-rcc.h>
Documentation/devicetree/bindings/serial/amlogic,meson-uart.yaml +2 -0
index 214fe8beddc3..d4178ab0d675 100644
@@ -62,6 +62,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     serial@84c0 {
Documentation/devicetree/bindings/soc/amlogic/amlogic,canvas.yaml +2 -0
index f548594d020b..cb008fd188d8 100644
@@ -40,6 +40,8 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     canvas: video-lut@48 {
Documentation/devicetree/bindings/sound/adi,adau7118.yaml +2 -0
index 75e0cbe6be70..76ee695097bf 100644
@@ -59,6 +59,8 @@ required:
   - iovdd-supply
   - dvdd-supply
 
+additionalProperties: false
+
 examples:
   - |
     i2c {
Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml +2 -0
index 38eaf0c028f9..a495d5fc0d23 100644
@@ -139,6 +139,8 @@ required:
   - "#address-cells"
   - "#size-cells"
 
+additionalProperties: false
+
 examples:
   - |
     codec@1,0{
Documentation/devicetree/bindings/sound/renesas,fsi.yaml +2 -0
index 140a37fc3c0b..d1b65554e681 100644
@@ -63,6 +63,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     sh_fsi2: sound@ec230000 {
Documentation/devicetree/bindings/sound/samsung,odroid.yaml +2 -0
index c6b244352d05..8ff2d39e7d17 100644
@@ -69,6 +69,8 @@ required:
   - cpu
   - codec
 
+additionalProperties: false
+
 examples:
   - |
     sound {
Documentation/devicetree/bindings/sound/samsung-i2s.yaml +2 -0
index 53e3bad4178c..b2ad093d94df 100644
@@ -115,6 +115,8 @@ required:
   - clocks
   - clock-names
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/clock/exynos-audss-clk.h>
Documentation/devicetree/bindings/sram/qcom,ocmem.yaml +2 -0
index 469cec133647..930188bc5e6a 100644
@@ -56,6 +56,8 @@ required:
   - '#size-cells'
   - ranges
 
+additionalProperties: false
+
 patternProperties:
   "-sram@[0-9a-f]+$":
     type: object
Documentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2 -0
index 93fe7b10a82e..e43ec50bda37 100644
@@ -42,6 +42,8 @@ required:
   - clocks
   - amlogic,ao-secure
 
+additionalProperties: false
+
 examples:
   - |
         cpu_temp: temperature-sensor@ff634800 {
Documentation/devicetree/bindings/timer/arm,arch_timer.yaml +2 -0
index 6deead07728e..fa255672e8e5 100644
@@ -82,6 +82,8 @@ properties:
 required:
   - compatible
 
+additionalProperties: false
+
 oneOf:
   - required:
       - interrupts
Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2 -0
index f7ef6646bade..582bbef62b95 100644
@@ -95,6 +95,8 @@ required:
   - '#address-cells'
   - '#size-cells'
 
+additionalProperties: false
+
 examples:
   - |
     timer@f0000000 {
Documentation/devicetree/bindings/timer/arm,global_timer.yaml +2 -0
index 21c24a8e28fd..4956c8f409d2 100644
@@ -35,6 +35,8 @@ required:
   - reg
   - clocks
 
+additionalProperties: false
+
 examples:
   - |
     timer@2c000600 {
Documentation/devicetree/bindings/timer/intel,ixp4xx-timer.yaml +2 -0
index 2807225db902..1a721d8af67a 100644
@@ -32,6 +32,8 @@ required:
   - reg
   - interrupts
 
+additionalProperties: false
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yaml +2 -0
index 273e359854dd..37bd01a62c52 100644
@@ -52,6 +52,8 @@ required:
   - interrupts
   - reg
 
+additionalProperties: false
+
 examples:
   - |
     // In this example, the IP contains two local timers, using separate
Documentation/devicetree/bindings/trivial-devices.yaml +2 -1
index 51d1f6e43c02..bcae5f9b1d7f 100644
@@ -369,4 +369,6 @@ required:
   - compatible
   - reg
 
+additionalProperties: false
+
...
--

2.20.1

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

maintainer Acked-by on v1 4/4
#22 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

In reply to Rob Herring

Quoting Rob Herring (2020-03-25 15:05:41)

Quoted 30 lines

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

---
 Documentation/devicetree/bindings/clock/fsl,plldig.yaml        | 2 ++
 Documentation/devicetree/bindings/clock/imx8mn-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/imx8mp-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/milbeaut-clock.yaml    | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,mmcc.yaml         | 2 ++
Quoted 9 lines
 .../devicetree/bindings/clock/qcom,msm8998-gpucc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml       | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-videocc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-videocc.yaml         | 2 ++

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Acked-by on v1 4/4
#23 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

In reply to Rob Herring
Quoted 95 lines · Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

---
 .../devicetree/bindings/arm/altera/socfpga-clk-manager.yaml    | 2 ++
 .../bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml       | 2 ++
 Documentation/devicetree/bindings/arm/renesas,prr.yaml         | 2 ++
 .../devicetree/bindings/arm/samsung/exynos-chipid.yaml         | 2 ++
 Documentation/devicetree/bindings/arm/samsung/pmu.yaml         | 2 ++
 .../bindings/arm/samsung/samsung-secure-firmware.yaml          | 2 ++
 .../devicetree/bindings/arm/stm32/st,stm32-syscon.yaml         | 2 ++
 Documentation/devicetree/bindings/clock/fsl,plldig.yaml        | 2 ++
 Documentation/devicetree/bindings/clock/imx8mn-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/imx8mp-clock.yaml      | 2 ++
 Documentation/devicetree/bindings/clock/milbeaut-clock.yaml    | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-ipq8074.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8996.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-msm8998.yaml  | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-qcs404.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sc7180.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml   | 2 ++
 Documentation/devicetree/bindings/clock/qcom,gcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,mmcc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,msm8998-gpucc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,rpmhcc.yaml       | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sc7180-videocc.yaml         | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-dispcc.yaml          | 2 ++
 Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml | 2 ++
 .../devicetree/bindings/clock/qcom,sdm845-videocc.yaml         | 2 ++
 .../devicetree/bindings/display/amlogic,meson-vpu.yaml         | 2 ++
 .../devicetree/bindings/dma/sifive,fu540-c000-pdma.yaml        | 2 ++
 Documentation/devicetree/bindings/dsp/fsl,dsp.yaml             | 2 ++
 Documentation/devicetree/bindings/eeprom/at24.yaml             | 2 ++
 .../firmware/intel,ixp4xx-network-processing-engine.yaml       | 3 +++
 .../devicetree/bindings/gpio/brcm,xgs-iproc-gpio.yaml          | 2 ++
 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++
 Documentation/devicetree/bindings/gpio/xylon,logicvc-gpio.yaml | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml    | 2 ++
 Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml     | 2 ++
 Documentation/devicetree/bindings/gpu/samsung-rotator.yaml     | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/adi,ltc2947.yaml       | 2 ++
 Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml | 2 ++
 Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml         | 2 ++
 Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml  | 2 ++
 Documentation/devicetree/bindings/iio/adc/adi,ad7780.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml    | 2 ++
 .../devicetree/bindings/iio/adc/microchip,mcp3911.yaml         | 2 ++
 .../devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml        | 2 ++
 .../devicetree/bindings/iio/chemical/plantower,pms7003.yaml    | 2 ++
 .../devicetree/bindings/iio/chemical/sensirion,sps30.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/dac/lltc,ltc1660.yaml    | 2 ++
 Documentation/devicetree/bindings/iio/light/adux1020.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/bh1750.yaml        | 2 ++
 Documentation/devicetree/bindings/iio/light/isl29018.yaml      | 2 ++
 Documentation/devicetree/bindings/iio/light/noa1305.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/stk33xx.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2583.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/tsl2772.yaml       | 2 ++
 Documentation/devicetree/bindings/iio/light/veml6030.yaml      | 2 ++
 .../devicetree/bindings/iio/pressure/asc,dlhl60d.yaml          | 2 ++
 Documentation/devicetree/bindings/iio/pressure/bmp085.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/devantech-srf04.yaml     | 2 ++
 .../devicetree/bindings/iio/proximity/parallax-ping.yaml       | 2 ++
 .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml       | 2 ++
 Documentation/devicetree/bindings/input/gpio-vibrator.yaml     | 2 ++
 Documentation/devicetree/bindings/input/max77650-onkey.yaml    | 3 +++
 .../bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml  | 2 ++
 Documentation/devicetree/bindings/iommu/samsung,sysmmu.yaml    | 2 ++
 Documentation/devicetree/bindings/leds/leds-max77650.yaml      | 3 +++
 Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml  | 3 +++
 .../devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml    | 2 ++
 Documentation/devicetree/bindings/media/amlogic,gx-vdec.yaml   | 2 ++
 .../devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml     | 2 ++
 Documentation/devicetree/bindings/media/renesas,ceu.yaml       | 2 ++
Quoted 2 lines
 Documentation/devicetree/bindings/mfd/max77650.yaml            | 2 ++
 Documentation/devicetree/bindings/mfd/rohm,bd71828-pmic.yaml   | 2 ++
Signature · 8 lines
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Acked-by on v1 4/4
#24 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

In reply to Rob Herring
Quoted 16 lines · Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

Quoted 5 lines

---

 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++

You may have already queue this up, but just in case.

Signature · 2 lines
Best Regards
Masahiro Yamada
#25 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

Hi Rob,

Quoted 22 lines · Masahiro Yamada
Quoted 18 lines · Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

---

Quoted 6 lines
Quoted 1 line
 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++

You may have already queue this up, but just in case.

I take back Ack for socionext,uniphier-gpio.yaml

Now "make dt_binding_check" produces a new warning.

gpio@55000000: 'interrupt-parent' does not match any of the regexes:
'pinctrl-[0-9]+'

This binding uses 'interrupt-parent'
without 'interrupts'.

Instead, the mapping of the interrupt numbers
is specified by the vendor-specific property
socionext,interrupt-ranges

I cannot add "interrupt-parent: true" because
dt-schema/meta-schemas/interrupts.yaml
has "interrupt-parent: false".

Is there any solution?

Signature · 2 lines
Best Regards
Masahiro Yamada
author maintainer
#26 lore raw

Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'

Quoted 54 lines · Masahiro Yamada

Hi Rob,

Quoted 22 lines · Masahiro Yamada
Quoted 18 lines · Rob Herring

Setting 'additionalProperties: false' is frequently omitted, but is important in order to check that there aren't extra undocumented properties in a binding.

Ideally, we'd just add this automatically and make this the default, but there's some cases where it doesn't work. For example, if a common schema is referenced, then properties in the common schema aren't part of what's considered for 'additionalProperties'. Also, sometimes there are bus specific properties such as 'spi-max-frequency' that go into bus child nodes, but aren't defined in the child node's schema.

So let's stick with the json-schema defined default and add 'additionalProperties: false' where needed. This will be a continual review comment and game of wack-a-mole.

---

Quoted 6 lines
Quoted 1 line
 .../devicetree/bindings/gpio/socionext,uniphier-gpio.yaml      | 2 ++

You may have already queue this up, but just in case.

I take back Ack for socionext,uniphier-gpio.yaml

Now "make dt_binding_check" produces a new warning.

gpio@55000000: 'interrupt-parent' does not match any of the regexes:
'pinctrl-[0-9]+'

This binding uses 'interrupt-parent'
without 'interrupts'.

Instead, the mapping of the interrupt numbers
is specified by the vendor-specific property
socionext,interrupt-ranges

Quoted 6 lines

I cannot add "interrupt-parent: true" because
dt-schema/meta-schemas/interrupts.yaml
has "interrupt-parent: false".

Is there any solution?

I'd meant to just drop socionext,uniphier-gpio.yaml.

Rob