dt-bindings: additionalProperties clean-up
Subsystems (73)
Rob Herring
maintainercover 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
Rob Herring
maintainerPATCH 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.
--- .../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
@@ -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
@@ -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
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.yamlThe 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]>Signed-off-by Rob Herring <[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@@ -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: 1Documentation/devicetree/bindings/trivial-devices.yaml +0
-4@@ -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
Jonathan Cameron
Acked-byRe: [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.yamlThe 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]>Signed-off-by Rob Herring <[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@@ -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: 1Documentation/devicetree/bindings/trivial-devices.yaml +0
-3@@ -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
Re: [PATCH 1/4] dt-bindings: iio/accel: Drop duplicate adi, adxl345/6 from trivial-devices.yaml
In reply to Rob HerringHi 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.yamlThe 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]>Signed-off-by Rob Herring <[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@@ -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
-0reg: maxItems: 1Documentation/devicetree/bindings/trivial-devices.yaml +0
-4@@ -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
Rob Herring
maintainerRe: [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.yamlThe 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]>Signed-off-by Rob Herring <[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@@ -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,adxl375I 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
Rob Herring
maintainerPATCH 2/4 dt-bindings: sram: qcom: Clean-up 'ranges' and child node names
In reply to Rob HerringThe 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.
--- .../devicetree/bindings/sram/qcom,ocmem.yaml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
Documentation/devicetree/bindings/sram/qcom,ocmem.yaml +6 -6
@@ -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
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]>Signed-off-by Rob Herring <[email protected]>
Rob Herring
maintainerPATCH 3/4 dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'
In reply to Rob HerringNumerous 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.
--- .../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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
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]>Signed-off-by Rob Herring <[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@@ -21,6 +21,9 @@ properties: reg: maxItems: 1 + clocks: + maxItems: 1 + '#clock-cells': const: 0(quoted diff) +0
-0
gpio.yamlb/Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml
index580a39e09d39..c58ff9a94f45100644Documentation/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-arrayDocumentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +3
-3@@ -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: - resetsDocumentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +3
-0@@ -75,6 +75,9 @@ properties: mali-supply: true + power-domains: + maxItems: 1 + resets: minItems: 1 maxItems: 2Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml +0
-1@@ -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@@ -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.yamlb/Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml
index41197578f19a..e8ce37fcbfec100644Documentation/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@@ -41,6 +41,9 @@ properties: "#clock-cells": const: 0 + clock-output-names: + const: bd71828-32k-out + rohm,charger-sense-resistor-ohms: minimum: 10000000 maximum: 50000000Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +15
-8@@ -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.yamlb/Documentation/devicetree/bindings/regulator/max77650-regulator.yaml
index7d724159f890..50690487edc8100644Documentation/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@@ -32,6 +32,8 @@ properties: description: phandle to the ao-secure syscon $ref: '/schemas/types.yaml#/definitions/phandle' + '#thermal-sensor-cells': + const: 0 required: - compatibleDocumentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2
-0@@ -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
Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'
In reply to Rob HerringQuoted 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]>Signed-off-by Rob Herring <[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 ++
Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'
In reply to Rob HerringQuoted 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@@ -21,6 +21,9 @@ properties: reg: maxItems: 1 + clocks: + maxItems: 1 + '#clock-cells': const: 0Documentation/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-arrayDocumentation/devicetree/bindings/gpu/arm,mali-bifrost.yaml +3
-3@@ -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: - resetsDocumentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +3
-0@@ -75,6 +75,9 @@ properties: mali-supply: true + power-domains: + maxItems: 1 + resets: minItems: 1 maxItems: 2Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml +0
-1@@ -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@@ -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@@ -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@@ -41,6 +41,9 @@ properties: "#clock-cells": const: 0 + clock-output-names: + const: bd71828-32k-out + rohm,charger-sense-resistor-ohms: minimum: 10000000 maximum: 50000000Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml +15
-8@@ -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@@ -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@@ -32,6 +32,8 @@ properties: description: phandle to the ao-secure syscon $ref: '/schemas/types.yaml#/definitions/phandle' + '#thermal-sensor-cells': + const: 0 required: - compatibleDocumentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2
-0@@ -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
Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'
In reply to Rob HerringQuoting 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]>Signed-off-by Rob Herring <[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
Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'
In reply to Rob HerringQuoted 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]>Signed-off-by Rob Herring <[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
Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'
In reply to Rob HerringQuoted 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
Re: [PATCH 3/4] dt-bindings: Clean-up schema errors due to missing 'addtionalProperties: false'
In reply to Rob HerringQuoted 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]>Signed-off-by Rob Herring <[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
Rob Herring
maintainerPATCH 4/4 dt-bindings: Add missing 'additionalProperties: false'
In reply to Rob HerringSetting '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
@@ -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
@@ -43,6 +43,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | ao-secure@140 {
Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml +2 -0
@@ -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
@@ -27,6 +27,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | prr: chipid@ff000044 {
Documentation/devicetree/bindings/arm/samsung/exynos-chipid.yaml +2 -0
@@ -30,6 +30,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | chipid@10000000 {
Documentation/devicetree/bindings/arm/samsung/pmu.yaml +2 -0
@@ -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
@@ -23,6 +23,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | firmware@203f000 {
Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml +2 -0
@@ -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
@@ -44,6 +44,8 @@ required: - clocks - '#clock-cells' +additionalProperties: false + examples: # Display PIXEL Clock node: - |
Documentation/devicetree/bindings/clock/imx8mn-clock.yaml +2 -0
@@ -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
@@ -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
@@ -35,6 +35,8 @@ required: - clocks - '#clock-cells' +additionalProperties: false + examples: # Clock controller node: - |
Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml +2 -0
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -43,6 +43,8 @@ required: - interrupts - '#dma-cells' +additionalProperties: false + examples: - | dma@3000000 {
Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +2 -0
@@ -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
@@ -172,6 +172,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {
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
@@ -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
@@ -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
@@ -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
@@ -53,6 +53,8 @@ required: - interrupt-names - clocks +additionalProperties: false + allOf: - if: properties:
Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +2 -0
@@ -94,6 +94,8 @@ required: - interrupt-names - clocks +additionalProperties: false + allOf: - if: properties:
Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml +2 -0
@@ -115,6 +115,8 @@ required: - clocks - clock-names +additionalProperties: false + allOf: - if: properties:
Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +2 -0
@@ -36,6 +36,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | rotator@12810000 {
Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml +2 -0
@@ -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
@@ -87,6 +87,8 @@ required: - reg +additionalProperties: false + examples: - | spi {
Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml +2 -0
@@ -32,6 +32,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {
Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml +2 -0
@@ -76,6 +76,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {
Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml +2 -0
@@ -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
@@ -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
@@ -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
@@ -32,6 +32,8 @@ required: - vref-supply - reg +additionalProperties: false + examples: - | spi {
Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml +2 -0
@@ -52,6 +52,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | spi {
Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +2 -0
@@ -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
@@ -38,6 +38,8 @@ required: - compatible - vcc-supply +additionalProperties: false + examples: - | serial {
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
@@ -34,6 +34,8 @@ required: - reg - vref-supply +additionalProperties: false + examples: - | spi {
Documentation/devicetree/bindings/iio/light/adux1020.yaml +2 -0
@@ -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
@@ -28,6 +28,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {
Documentation/devicetree/bindings/iio/light/isl29018.yaml +2 -0
@@ -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
@@ -29,6 +29,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {
Documentation/devicetree/bindings/iio/light/stk33xx.yaml +2 -0
@@ -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
@@ -32,6 +32,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {
Documentation/devicetree/bindings/iio/light/tsl2772.yaml +2 -0
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -33,3 +33,6 @@ properties: required: - compatible +additionalProperties: false + +...
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
@@ -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
@@ -49,3 +49,6 @@ required: - compatible - "#address-cells" - "#size-cells" +additionalProperties: false + +...
Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml +3 -0
@@ -50,3 +50,6 @@ patternProperties: required: - compatible +additionalProperties: false + +...
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
@@ -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
@@ -82,6 +82,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | cec_AO: cec@100 {
Documentation/devicetree/bindings/media/renesas,ceu.yaml +2 -0
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -40,6 +40,8 @@ required: - reg - clocks +additionalProperties: false + examples: - | sysconf: chiptop@e0200000 {
Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +2 -0
@@ -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
@@ -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
@@ -57,6 +57,8 @@ patternProperties: required: - compatible +additionalProperties: false + examples: - | syscon: scu@1e6e2000 {
Documentation/devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml +2 -0
@@ -70,6 +70,8 @@ required: - compatible - aspeed,external-nodes +additionalProperties: false + examples: - | apb {
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
@@ -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
@@ -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
@@ -41,6 +41,8 @@ required: - regmap - offset +additionalProperties: false + allOf: - if: not:
Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +2 -0
@@ -41,6 +41,8 @@ required: - regmap - offset +additionalProperties: false + allOf: - if: not:
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
@@ -55,6 +55,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c@1 {
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
@@ -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
@@ -28,6 +28,8 @@ required: - reg - "#reset-cells" +additionalProperties: false + examples: - | reset-controller@8b2c800 {
Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml +2 -0
@@ -29,6 +29,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | rng@c8834000 {
Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml +2 -0
@@ -35,6 +35,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | rng {
Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml +2 -0
@@ -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
@@ -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
@@ -62,6 +62,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | serial@84c0 {
Documentation/devicetree/bindings/soc/amlogic/amlogic,canvas.yaml +2 -0
@@ -40,6 +40,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | canvas: video-lut@48 {
Documentation/devicetree/bindings/sound/adi,adau7118.yaml +2 -0
@@ -59,6 +59,8 @@ required: - iovdd-supply - dvdd-supply +additionalProperties: false + examples: - | i2c {
Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml +2 -0
@@ -139,6 +139,8 @@ required: - "#address-cells" - "#size-cells" +additionalProperties: false + examples: - | codec@1,0{
Documentation/devicetree/bindings/sound/renesas,fsi.yaml +2 -0
@@ -63,6 +63,8 @@ required: - reg - interrupts +additionalProperties: false + examples: - | sh_fsi2: sound@ec230000 {
Documentation/devicetree/bindings/sound/samsung,odroid.yaml +2 -0
@@ -69,6 +69,8 @@ required: - cpu - codec +additionalProperties: false + examples: - | sound {
Documentation/devicetree/bindings/sound/samsung-i2s.yaml +2 -0
@@ -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
@@ -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
@@ -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
@@ -82,6 +82,8 @@ properties: required: - compatible +additionalProperties: false + oneOf: - required: - interrupts
Documentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2 -0
@@ -95,6 +95,8 @@ required: - '#address-cells' - '#size-cells' +additionalProperties: false + examples: - | timer@f0000000 {
Documentation/devicetree/bindings/timer/arm,global_timer.yaml +2 -0
@@ -35,6 +35,8 @@ required: - reg - clocks +additionalProperties: false + examples: - | timer@2c000600 {
Documentation/devicetree/bindings/timer/intel,ixp4xx-timer.yaml +2 -0
@@ -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
@@ -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
@@ -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
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
Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'
In reply to Rob HerringQuoted 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
Signed-off-by Rob Herring <[email protected]>--- .../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.yamlb/Documentation/devicetree/bindings/arm/altera/socfpga-clk-manager.yaml
index e4131fa42b26..572381306681 100644Documentation/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.yamlb/Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml
index853d7d2b56f5..66213bd95e6e100644
--- a/Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yaml
+++ b/Documentation/devicetree/bindings/arm/amlogic/amlogic,meson-gx-ao-secure.yamlDocumentation/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@@ -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@@ -27,6 +27,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | prr: chipid@ff000044 {Documentation/devicetree/bindings/arm/samsung/exynos-chipid.yaml +2
-0@@ -30,6 +30,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | chipid@10000000 {Documentation/devicetree/bindings/arm/samsung/pmu.yaml +2
-0@@ -89,6 +89,8 @@ required: - clock-names - clocks +additionalProperties: false + examples: - | #include <dt-bindings/clock/exynos5250.h>(quoted diff) +0
-0
firmware.yamlb/Documentation/devicetree/bindings/arm/samsung/samsung-secure-firmware.yaml
index51d23b6f8a94..3d9abad3c749100644
--- a/Documentation/devicetree/bindings/arm/samsung/samsung-secure-firmware.yaml
+++ b/Documentation/devicetree/bindings/arm/samsung/samsung-secure-firmware.yamlDocumentation/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@@ -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@@ -44,6 +44,8 @@ required: - clocks - '#clock-cells' +additionalProperties: false + examples: # Display PIXEL Clock node: - |Documentation/devicetree/bindings/clock/imx8mn-clock.yaml +2
-0@@ -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@@ -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@@ -35,6 +35,8 @@ required: - clocks - '#clock-cells' +additionalProperties: false + examples: # Clock controller node: - |Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml +2
-0@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -43,6 +43,8 @@ required: - interrupts - '#dma-cells' +additionalProperties: false + examples: - | dma@3000000 {Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +2
-0@@ -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@@ -172,6 +172,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {(quoted diff) +0
-0
processing-engine.yamlb/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
index878a2079ebb6..1bd2870c3a9c100644
--- a/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yaml
+++ b/Documentation/devicetree/bindings/firmware/intel,ixp4xx-network-processing-engine.yamlDocumentation/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@@ -47,6 +47,8 @@ required: - "#gpio-cells" - gpio-controller +additionalProperties: false + dependencies: interrupt-controller: [ interrupts ](quoted diff) +0
-0
gpio.yamlb/Documentation/devicetree/bindings/gpio/socionext,uniphier-gpio.yaml
indexc58ff9a94f45..1a54db04f29d100644Documentation/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@@ -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@@ -53,6 +53,8 @@ required: - interrupt-names - clocks +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +2
-0@@ -94,6 +94,8 @@ required: - interrupt-names - clocks +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml +2
-0@@ -115,6 +115,8 @@ required: - clocks - clock-names +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +2
-0@@ -36,6 +36,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | rotator@12810000 {Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml +2
-0@@ -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@@ -87,6 +87,8 @@ required: - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml +2
-0@@ -32,6 +32,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml +2
-0@@ -76,6 +76,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml +2
-0@@ -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@@ -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@@ -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@@ -32,6 +32,8 @@ required: - vref-supply - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml +2
-0@@ -52,6 +52,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +2
-0@@ -69,6 +69,8 @@ required: - "#address-cells" - "#size-cells" +additionalProperties: false + patternProperties: "^filter@[0-9]+$": type: objectdiff --git
a/Documentation/devicetree/bindings/iio/chemical/plantower,pms7003.yamlb/Documentation/devicetree/bindings/iio/chemical/plantower,pms7003.yamlindex19e53930ebf6..1fe561574019100644Documentation/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.yamlb/Documentation/devicetree/bindings/iio/chemical/sensirion,sps30.yamlindex50a50a0d7070..a93d1972a5c2100644Documentation/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@@ -34,6 +34,8 @@ required: - reg - vref-supply +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/light/adux1020.yaml +2
-0@@ -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@@ -28,6 +28,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/isl29018.yaml +2
-0@@ -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@@ -29,6 +29,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/stk33xx.yaml +2
-0@@ -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@@ -32,6 +32,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/tsl2772.yaml +2
-0@@ -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@@ -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@@ -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@@ -52,6 +52,8 @@ required: - vddd-supply - vdda-supply +additionalProperties: false + examples: - | #include <dt-bindings/gpio/gpio.h>(quoted diff) +0
-0
srf04.yamlb/Documentation/devicetree/bindings/iio/proximity/devantech-srf04.yaml
index4e80ea7c1475..7ac5eb7560e0100644Documentation/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.yamlb/Documentation/devicetree/bindings/iio/proximity/parallax-ping.yamlindexa079c9921af6..ada55f186f3c100644Documentation/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.yamlb/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yamlindexd4922f9f0376..acc030c1b20e100644Documentation/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@@ -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@@ -33,3 +33,6 @@ properties: required: - compatible +additionalProperties: false + +...(quoted diff) +0
-0
controller/intel,ixp4xx-interrupt.yamlb/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml
index507c141ea760..ccc507f384d2100644
--- a/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/intel,ixp4xx-interrupt.yamlDocumentation/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@@ -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@@ -49,3 +49,6 @@ required: - compatible - "#address-cells" - "#size-cells" +additionalProperties: false + +...Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml +3
-0@@ -50,3 +50,6 @@ patternProperties: required: - compatible +additionalProperties: false + +...(quoted diff) +0
-0
mhu.yamlb/Documentation/devicetree/bindings/mailbox/amlogic,meson-gxbb-mhu.yaml
index 319280563648..aa2b3bf56b57 100644Documentation/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@@ -124,6 +124,8 @@ required: - amlogic,ao-sysctrl - amlogic,canvas +additionalProperties: false + examples: - | vdec: video-decoder@c8820000 {(quoted diff) +0
-0
cec.yamlb/Documentation/devicetree/bindings/media/amlogic,meson-gx-ao-cec.yaml
indexe8ce37fcbfec..95ffa8bc0533100644Documentation/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@@ -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@@ -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@@ -77,6 +77,8 @@ required: - gpio-controller - "#gpio-cells" +additionalProperties: false + examples: - | #include <dt-bindings/interrupt-controller/irq.h>(quoted diff) +0
-0
manager.yamlb/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
index0ea21a6f70b4..38ab0499102d100644
--- a/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yaml
+++ b/Documentation/devicetree/bindings/misc/intel,ixp4xx-ahb-queue-manager.yamlDocumentation/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@@ -167,6 +167,8 @@ required: - '#address-cells' - '#size-cells' +additionalProperties: false + examples: - | #include <dt-bindings/interrupt-controller/irq.h>(quoted diff) +0
-0
phy.yamlb/Documentation/devicetree/bindings/phy/allwinner,sun50i-h6-usb3-phy.yaml
indexe5922b427342..c03b83103e87100644Documentation/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.yamlb/Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml
index346f9c35427c..453c083cf44c100644
--- a/Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/amlogic,meson-g12a-usb3-pcie-phy.yamlDocumentation/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@@ -40,6 +40,8 @@ required: - reg - clocks +additionalProperties: false + examples: - | sysconf: chiptop@e0200000 {Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +2
-0@@ -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@@ -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.yamlb/Documentation/devicetree/bindings/pinctrl/aspeed,ast2400-pinctrl.yaml
index135c7dfbc180..7651a675ab2d100644Documentation/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.yamlb/Documentation/devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml
index824f7fd1d51b..36feaf5e2dff100644Documentation/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.yamlb/Documentation/devicetree/bindings/pinctrl/aspeed,ast2600-pinctrl.yaml
indexac8d1c30a8ed..45af29bc3202100644Documentation/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@@ -194,6 +194,8 @@ required: - ranges - pins-are-numbered +additionalProperties: false + examples: - | #include <dt-bindings/pinctrl/stm32-pinfunc.h>(quoted diff) +0
-0
pwrc.yamlb/Documentation/devicetree/bindings/power/amlogic,meson-ee-pwrc.yamlindexd3098c924b25..6c6079fe1351100644Documentation/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.yamlb/Documentation/devicetree/bindings/power/reset/syscon-poweroff.yaml
index520e07e6f21b..3412fe7e1e80100644Documentation/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@@ -41,6 +41,8 @@ required: - regmap - offset +additionalProperties: false + allOf: - if: not:(quoted diff) +0
-0
charger.yamlb/Documentation/devicetree/bindings/power/supply/max77650-charger.yaml
indexdeef010ec535..62eeddb65aed100644Documentation/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@@ -55,6 +55,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c@1 {(quoted diff) +0
-0
regulator.yamlb/Documentation/devicetree/bindings/regulator/max77650-regulator.yaml
index50690487edc8..ce0a4021ae7f100644Documentation/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@@ -29,6 +29,8 @@ required: - reg - "#reset-cells" +additionalProperties: false + examples: - | reset-controller@c884404 {(quoted diff) +0
-0
rescal.yamlb/Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml
index411bd76f1b64..512a33bdb208100644
--- a/Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata-rescal.yaml
+++ b/Documentation/devicetree/bindings/reset/brcm,bcm7216-pcie-sata-rescal.yamlDocumentation/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@@ -29,6 +29,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | rng@c8834000 {Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml +2
-0@@ -35,6 +35,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | rng {Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml +2
-0@@ -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@@ -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@@ -62,6 +62,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | serial@84c0 {Documentation/devicetree/bindings/soc/amlogic/amlogic,canvas.yaml +2
-0@@ -40,6 +40,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | canvas: video-lut@48 {Documentation/devicetree/bindings/sound/adi,adau7118.yaml +2
-0@@ -59,6 +59,8 @@ required: - iovdd-supply - dvdd-supply +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml +2
-0@@ -139,6 +139,8 @@ required: - "#address-cells" - "#size-cells" +additionalProperties: false + examples: - | codec@1,0{Documentation/devicetree/bindings/sound/renesas,fsi.yaml +2
-0@@ -63,6 +63,8 @@ required: - reg - interrupts +additionalProperties: false + examples: - | sh_fsi2: sound@ec230000 {Documentation/devicetree/bindings/sound/samsung,odroid.yaml +2
-0@@ -69,6 +69,8 @@ required: - cpu - codec +additionalProperties: false + examples: - | sound {Documentation/devicetree/bindings/sound/samsung-i2s.yaml +2
-0@@ -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@@ -56,6 +56,8 @@ required: - '#size-cells' - ranges +additionalProperties: false + patternProperties: "-sram@[0-9a-f]+$": type: objectDocumentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2
-0@@ -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@@ -82,6 +82,8 @@ properties: required: - compatible +additionalProperties: false + oneOf: - required: - interruptsDocumentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2
-0@@ -95,6 +95,8 @@ required: - '#address-cells' - '#size-cells' +additionalProperties: false + examples: - | timer@f0000000 {Documentation/devicetree/bindings/timer/arm,global_timer.yaml +2
-0@@ -35,6 +35,8 @@ required: - reg - clocks +additionalProperties: false + examples: - | timer@2c000600 {Documentation/devicetree/bindings/timer/intel,ixp4xx-timer.yaml +2
-0@@ -32,6 +32,8 @@ required: - reg - interrupts +additionalProperties: false + examples: - | #include <dt-bindings/interrupt-controller/irq.h>(quoted diff) +0
-0
mct.yamlb/Documentation/devicetree/bindings/timer/samsung,exynos4210-mct.yamlindex273e359854dd..37bd01a62c52100644Documentation/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 separateDocumentation/devicetree/bindings/trivial-devices.yaml +0
-0
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
Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'
In reply to Rob HerringQuoted 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.
Signed-off-by Rob Herring <[email protected]>--- .../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
Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'
In reply to Rob HerringQuoted 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.
Signed-off-by Rob Herring <[email protected]>
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@@ -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@@ -43,6 +43,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | ao-secure@140 {Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml +2
-0@@ -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@@ -27,6 +27,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | prr: chipid@ff000044 {Documentation/devicetree/bindings/arm/samsung/exynos-chipid.yaml +2
-0@@ -30,6 +30,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | chipid@10000000 {Documentation/devicetree/bindings/arm/samsung/pmu.yaml +2
-0@@ -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@@ -23,6 +23,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | firmware@203f000 {Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml +2
-0@@ -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@@ -44,6 +44,8 @@ required: - clocks - '#clock-cells' +additionalProperties: false + examples: # Display PIXEL Clock node: - |Documentation/devicetree/bindings/clock/imx8mn-clock.yaml +2
-0@@ -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@@ -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@@ -35,6 +35,8 @@ required: - clocks - '#clock-cells' +additionalProperties: false + examples: # Clock controller node: - |Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml +2
-0@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -43,6 +43,8 @@ required: - interrupts - '#dma-cells' +additionalProperties: false + examples: - | dma@3000000 {Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +2
-0@@ -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@@ -172,6 +172,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {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@@ -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@@ -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@@ -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@@ -53,6 +53,8 @@ required: - interrupt-names - clocks +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +2
-0@@ -94,6 +94,8 @@ required: - interrupt-names - clocks +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml +2
-0@@ -115,6 +115,8 @@ required: - clocks - clock-names +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +2
-0@@ -36,6 +36,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | rotator@12810000 {Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml +2
-0@@ -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@@ -87,6 +87,8 @@ required: - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml +2
-0@@ -32,6 +32,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml +2
-0@@ -76,6 +76,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml +2
-0@@ -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@@ -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@@ -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@@ -32,6 +32,8 @@ required: - vref-supply - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml +2
-0@@ -52,6 +52,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +2
-0@@ -69,6 +69,8 @@ required: - "#address-cells" - "#size-cells" +additionalProperties: false + patternProperties: "^filter@[0-9]+$": type: objectDocumentation/devicetree/bindings/iio/chemical/plantower,pms7003.yaml +2
-0@@ -38,6 +38,8 @@ required: - compatible - vcc-supply +additionalProperties: false + examples: - | serial {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@@ -34,6 +34,8 @@ required: - reg - vref-supply +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/light/adux1020.yaml +2
-0@@ -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@@ -28,6 +28,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/isl29018.yaml +2
-0@@ -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@@ -29,6 +29,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/stk33xx.yaml +2
-0@@ -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@@ -32,6 +32,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/tsl2772.yaml +2
-0@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -33,3 +33,6 @@ properties: required: - compatible +additionalProperties: false + +...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@@ -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@@ -49,3 +49,6 @@ required: - compatible - "#address-cells" - "#size-cells" +additionalProperties: false + +...Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml +3
-0@@ -50,3 +50,6 @@ patternProperties: required: - compatible +additionalProperties: false + +...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@@ -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@@ -82,6 +82,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | cec_AO: cec@100 {Documentation/devicetree/bindings/media/renesas,ceu.yaml +2
-0@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -40,6 +40,8 @@ required: - reg - clocks +additionalProperties: false + examples: - | sysconf: chiptop@e0200000 {Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +2
-0@@ -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@@ -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@@ -57,6 +57,8 @@ patternProperties: required: - compatible +additionalProperties: false + examples: - | syscon: scu@1e6e2000 {Documentation/devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml +2
-0@@ -70,6 +70,8 @@ required: - compatible - aspeed,external-nodes +additionalProperties: false + examples: - | apb {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@@ -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@@ -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@@ -41,6 +41,8 @@ required: - regmap - offset +additionalProperties: false + allOf: - if: not:Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +2
-0@@ -41,6 +41,8 @@ required: - regmap - offset +additionalProperties: false + allOf: - if: not: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@@ -55,6 +55,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c@1 {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@@ -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@@ -28,6 +28,8 @@ required: - reg - "#reset-cells" +additionalProperties: false + examples: - | reset-controller@8b2c800 {Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml +2
-0@@ -29,6 +29,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | rng@c8834000 {Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml +2
-0@@ -35,6 +35,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | rng {Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml +2
-0@@ -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@@ -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@@ -62,6 +62,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | serial@84c0 {Documentation/devicetree/bindings/soc/amlogic/amlogic,canvas.yaml +2
-0@@ -40,6 +40,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | canvas: video-lut@48 {Documentation/devicetree/bindings/sound/adi,adau7118.yaml +2
-0@@ -59,6 +59,8 @@ required: - iovdd-supply - dvdd-supply +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml +2
-0@@ -139,6 +139,8 @@ required: - "#address-cells" - "#size-cells" +additionalProperties: false + examples: - | codec@1,0{Documentation/devicetree/bindings/sound/renesas,fsi.yaml +2
-0@@ -63,6 +63,8 @@ required: - reg - interrupts +additionalProperties: false + examples: - | sh_fsi2: sound@ec230000 {Documentation/devicetree/bindings/sound/samsung,odroid.yaml +2
-0@@ -69,6 +69,8 @@ required: - cpu - codec +additionalProperties: false + examples: - | sound {Documentation/devicetree/bindings/sound/samsung-i2s.yaml +2
-0@@ -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@@ -56,6 +56,8 @@ required: - '#size-cells' - ranges +additionalProperties: false + patternProperties: "-sram@[0-9a-f]+$": type: objectDocumentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2
-0@@ -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@@ -82,6 +82,8 @@ properties: required: - compatible +additionalProperties: false + oneOf: - required: - interruptsDocumentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2
-0@@ -95,6 +95,8 @@ required: - '#address-cells' - '#size-cells' +additionalProperties: false + examples: - | timer@f0000000 {Documentation/devicetree/bindings/timer/arm,global_timer.yaml +2
-0@@ -35,6 +35,8 @@ required: - reg - clocks +additionalProperties: false + examples: - | timer@2c000600 {Documentation/devicetree/bindings/timer/intel,ixp4xx-timer.yaml +2
-0@@ -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@@ -52,6 +52,8 @@ required: - interrupts - reg +additionalProperties: false + examples: - | // In this example, the IP contains two local timers, using separateDocumentation/devicetree/bindings/trivial-devices.yaml +2
-0@@ -369,4 +369,6 @@ required: - compatible - reg +additionalProperties: false +
Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'
In reply to Rob HerringQuoted 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
Signed-off-by Rob Herring <[email protected]>--- .../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@@ -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@@ -43,6 +43,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | ao-secure@140 {Documentation/devicetree/bindings/arm/msm/qcom,llcc.yaml +2
-0@@ -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@@ -27,6 +27,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | prr: chipid@ff000044 {Documentation/devicetree/bindings/arm/samsung/exynos-chipid.yaml +2
-0@@ -30,6 +30,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | chipid@10000000 {Documentation/devicetree/bindings/arm/samsung/pmu.yaml +2
-0@@ -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@@ -23,6 +23,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | firmware@203f000 {Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml +2
-0@@ -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@@ -44,6 +44,8 @@ required: - clocks - '#clock-cells' +additionalProperties: false + examples: # Display PIXEL Clock node: - |Documentation/devicetree/bindings/clock/imx8mn-clock.yaml +2
-0@@ -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@@ -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@@ -35,6 +35,8 @@ required: - clocks - '#clock-cells' +additionalProperties: false + examples: # Clock controller node: - |Documentation/devicetree/bindings/clock/qcom,gcc-apq8064.yaml +2
-0@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -43,6 +43,8 @@ required: - interrupts - '#dma-cells' +additionalProperties: false + examples: - | dma@3000000 {Documentation/devicetree/bindings/dsp/fsl,dsp.yaml +2
-0@@ -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@@ -172,6 +172,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {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@@ -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@@ -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@@ -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@@ -53,6 +53,8 @@ required: - interrupt-names - clocks +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/arm,mali-midgard.yaml +2
-0@@ -94,6 +94,8 @@ required: - interrupt-names - clocks +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/arm,mali-utgard.yaml +2
-0@@ -115,6 +115,8 @@ required: - clocks - clock-names +additionalProperties: false + allOf: - if: properties:Documentation/devicetree/bindings/gpu/samsung-rotator.yaml +2
-0@@ -36,6 +36,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | rotator@12810000 {Documentation/devicetree/bindings/hwmon/adi,adm1177.yaml +2
-0@@ -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@@ -87,6 +87,8 @@ required: - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/hwmon/pmbus/ti,ucd90320.yaml +2
-0@@ -32,6 +32,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/hwmon/ti,tmp513.yaml +2
-0@@ -76,6 +76,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/accel/bosch,bma400.yaml +2
-0@@ -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@@ -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@@ -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@@ -32,6 +32,8 @@ required: - vref-supply - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/adc/microchip,mcp3911.yaml +2
-0@@ -52,6 +52,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/adc/st,stm32-dfsdm-adc.yaml +2
-0@@ -69,6 +69,8 @@ required: - "#address-cells" - "#size-cells" +additionalProperties: false + patternProperties: "^filter@[0-9]+$": type: objectDocumentation/devicetree/bindings/iio/chemical/plantower,pms7003.yaml +2
-0@@ -38,6 +38,8 @@ required: - compatible - vcc-supply +additionalProperties: false + examples: - | serial {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@@ -34,6 +34,8 @@ required: - reg - vref-supply +additionalProperties: false + examples: - | spi {Documentation/devicetree/bindings/iio/light/adux1020.yaml +2
-0@@ -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@@ -28,6 +28,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/isl29018.yaml +2
-0@@ -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@@ -29,6 +29,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/stk33xx.yaml +2
-0@@ -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@@ -32,6 +32,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/iio/light/tsl2772.yaml +2
-0@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -33,3 +33,6 @@ properties: required: - compatible +additionalProperties: false + +...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@@ -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@@ -49,3 +49,6 @@ required: - compatible - "#address-cells" - "#size-cells" +additionalProperties: false + +...Documentation/devicetree/bindings/leds/rohm,bd71828-leds.yaml +3
-0@@ -50,3 +50,6 @@ patternProperties: required: - compatible +additionalProperties: false + +...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@@ -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@@ -82,6 +82,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | cec_AO: cec@100 {Documentation/devicetree/bindings/media/renesas,ceu.yaml +2
-0@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -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@@ -40,6 +40,8 @@ required: - reg - clocks +additionalProperties: false + examples: - | sysconf: chiptop@e0200000 {Documentation/devicetree/bindings/phy/marvell,mmp3-hsic-phy.yaml +2
-0@@ -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@@ -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@@ -57,6 +57,8 @@ patternProperties: required: - compatible +additionalProperties: false + examples: - | syscon: scu@1e6e2000 {Documentation/devicetree/bindings/pinctrl/aspeed,ast2500-pinctrl.yaml +2
-0@@ -70,6 +70,8 @@ required: - compatible - aspeed,external-nodes +additionalProperties: false + examples: - | apb {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@@ -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@@ -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@@ -41,6 +41,8 @@ required: - regmap - offset +additionalProperties: false + allOf: - if: not:Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml +2
-0@@ -41,6 +41,8 @@ required: - regmap - offset +additionalProperties: false + allOf: - if: not: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@@ -55,6 +55,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | i2c@1 {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@@ -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@@ -28,6 +28,8 @@ required: - reg - "#reset-cells" +additionalProperties: false + examples: - | reset-controller@8b2c800 {Documentation/devicetree/bindings/rng/amlogic,meson-rng.yaml +2
-0@@ -29,6 +29,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | rng@c8834000 {Documentation/devicetree/bindings/rng/brcm,bcm2835.yaml +2
-0@@ -35,6 +35,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | rng {Documentation/devicetree/bindings/rtc/renesas,sh-rtc.yaml +2
-0@@ -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@@ -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@@ -62,6 +62,8 @@ required: - clocks - clock-names +additionalProperties: false + examples: - | serial@84c0 {Documentation/devicetree/bindings/soc/amlogic/amlogic,canvas.yaml +2
-0@@ -40,6 +40,8 @@ required: - compatible - reg +additionalProperties: false + examples: - | canvas: video-lut@48 {Documentation/devicetree/bindings/sound/adi,adau7118.yaml +2
-0@@ -59,6 +59,8 @@ required: - iovdd-supply - dvdd-supply +additionalProperties: false + examples: - | i2c {Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml +2
-0@@ -139,6 +139,8 @@ required: - "#address-cells" - "#size-cells" +additionalProperties: false + examples: - | codec@1,0{Documentation/devicetree/bindings/sound/renesas,fsi.yaml +2
-0@@ -63,6 +63,8 @@ required: - reg - interrupts +additionalProperties: false + examples: - | sh_fsi2: sound@ec230000 {Documentation/devicetree/bindings/sound/samsung,odroid.yaml +2
-0@@ -69,6 +69,8 @@ required: - cpu - codec +additionalProperties: false + examples: - | sound {Documentation/devicetree/bindings/sound/samsung-i2s.yaml +2
-0@@ -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@@ -56,6 +56,8 @@ required: - '#size-cells' - ranges +additionalProperties: false + patternProperties: "-sram@[0-9a-f]+$": type: objectDocumentation/devicetree/bindings/thermal/amlogic,thermal.yaml +2
-0@@ -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@@ -82,6 +82,8 @@ properties: required: - compatible +additionalProperties: false + oneOf: - required: - interruptsDocumentation/devicetree/bindings/timer/arm,arch_timer_mmio.yaml +2
-0@@ -95,6 +95,8 @@ required: - '#address-cells' - '#size-cells' +additionalProperties: false + examples: - | timer@f0000000 {Documentation/devicetree/bindings/timer/arm,global_timer.yaml +2
-0@@ -35,6 +35,8 @@ required: - reg - clocks +additionalProperties: false + examples: - | timer@2c000600 {Documentation/devicetree/bindings/timer/intel,ixp4xx-timer.yaml +2
-0@@ -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@@ -52,6 +52,8 @@ required: - interrupts - reg +additionalProperties: false + examples: - | // In this example, the IP contains two local timers, using separateDocumentation/devicetree/bindings/trivial-devices.yaml +2
-1@@ -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
Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'
In reply to Rob HerringQuoting 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.
Signed-off-by Rob Herring <[email protected]>--- 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
Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'
In reply to Rob HerringQuoted 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.
Signed-off-by Rob Herring <[email protected]>--- .../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
Re: [PATCH 4/4] dt-bindings: Add missing 'additionalProperties: false'
In reply to Rob HerringQuoted 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
Signed-off-by Rob Herring <[email protected]>---
.../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
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.
Signed-off-by Rob Herring <[email protected]>---
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.
Acked-by Masahiro Yamada <[email protected]>
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" becausedt-schema/meta-schemas/interrupts.yaml
has "interrupt-parent: false".
Is there any solution?
Signature · 2 lines
Best Regards Masahiro Yamada
Rob Herring
maintainerRe: [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.
Signed-off-by Rob Herring <[email protected]>---
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.
Acked-by Masahiro Yamada <[email protected]>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