Commit 5be478f9 authored by Rob Herring's avatar Rob Herring

dt-bindings: Another round of adding missing 'additionalProperties'

Another round of wack-a-mole. The json-schema default is additional
unknown properties are allowed, but for DT all properties should be
defined.

Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Stephen Boyd <sboyd@kernel.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Baolin Wang <baolin.wang7@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Bjorn Helgaas <bhelgaas@google.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: default avatarGuenter Roeck <linux@roeck-us.net>
Reviewed-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Acked-By: default avatarVinod Koul <vkoul@kernel.org>
Acked-by: default avatarLee Jones <lee.jones@linaro.org>
Acked-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> # for iio
Acked-by: default avatarThierry Reding <treding@nvidia.com>
Acked-by: default avatarMark Brown <broonie@kernel.org>
Reviewd-by: default avatarCorey Minyard <cminyard@mvista.com>
Acked-by: default avatarPavel Machek <pavel@ucw.cz>
Acked-by: default avatarSebastian Reichel <sre@kernel.org>
Link: https://lore.kernel.org/r/20201002234143.3570746-1-robh@kernel.orgSigned-off-by: default avatarRob Herring <robh@kernel.org>
parent 891adc13
......@@ -54,6 +54,8 @@ required:
- compatible
- mboxes
additionalProperties: false
examples:
- |
firmware {
......
......@@ -47,6 +47,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
pericfg@10003000 {
......
......@@ -93,4 +93,6 @@ properties:
required:
- compatible
additionalProperties: false
...
......@@ -33,4 +33,7 @@ properties:
contains:
const: apb_pclk
additionalItems: true
additionalProperties: true
...
......@@ -308,6 +308,8 @@ required:
- clocks
- '#clock-cells'
additionalProperties: false
dependencies:
"nvidia,suspend-mode": ["nvidia,core-pwr-off-time", "nvidia,cpu-pwr-off-time"]
"nvidia,core-pwr-off-time": ["nvidia,core-pwr-good-time"]
......
......@@ -26,6 +26,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
cdmm@1bde8000 {
......
......@@ -57,6 +57,11 @@ properties:
"ranges" property should provide a "reasonable" default that is known to
work. The software should initialize the bus controller according to it.
patternProperties:
"^.*@[1-5],[1-9a-f][0-9a-f]+$":
description: Devices attached to chip selects
type: object
required:
- compatible
- reg
......@@ -64,6 +69,8 @@ required:
- "#size-cells"
- ranges
additionalProperties: false
examples:
- |
// In this example,
......
......@@ -89,6 +89,8 @@ required:
- compatible
- clocks
additionalProperties: false
examples:
- |
vco1: clock {
......
......@@ -50,6 +50,15 @@ properties:
'#clock-cells':
const: 1
clock-names:
minItems: 1
maxItems: 2
items:
enum: [ xin, clkin ]
clocks:
minItems: 1
maxItems: 2
patternProperties:
"^OUT[1-4]$":
type: object
......@@ -93,19 +102,12 @@ allOf:
maxItems: 1
else:
# Devices without builtin crystal
properties:
clock-names:
minItems: 1
maxItems: 2
items:
enum: [ xin, clkin ]
clocks:
minItems: 1
maxItems: 2
required:
- clock-names
- clocks
additionalProperties: false
examples:
- |
#include <dt-bindings/clk/versaclock.h>
......
......@@ -57,6 +57,8 @@ required:
- interrupts
- '#clock-cells'
additionalProperties: false
examples:
# Clock Control Module node:
- |
......
......@@ -33,6 +33,8 @@ required:
- interrupts
- '#clock-cells'
additionalProperties: false
examples:
# Clock Control Module node:
- |
......
......@@ -49,6 +49,8 @@ required:
- clocks
- clock-names
additionalProperties: false
examples:
# Clock Control Module node:
- |
......
......@@ -53,6 +53,8 @@ required:
- clocks
- clock-names
additionalProperties: false
examples:
# Clock Control Module node:
- |
......
......@@ -49,6 +49,8 @@ required:
- clocks
- clock-names
additionalProperties: false
examples:
# Clock Control Module node:
- |
......
......@@ -33,6 +33,8 @@ required:
- reg
- '#clock-cells'
additionalProperties: false
examples:
- |
cgu: clock-controller@e0200000 {
......
......@@ -56,6 +56,8 @@ required:
- '#reset-cells'
- '#power-domain-cells'
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/qcom,rpmh.h>
......
......@@ -73,6 +73,8 @@ else:
The 'reg' property for the clock node is also required if there is a sub
range of registers for the clocks.
additionalProperties: false
examples:
- |
ap_clk: clock-controller@21500000 {
......
......@@ -26,6 +26,8 @@ required:
- "#clock-cells"
- reg
additionalProperties: false
examples:
- |
ehrpwm_tbclk: syscon@4140 {
......
......@@ -31,6 +31,9 @@ properties:
compatible:
const: ite,it6505
reg:
maxItems: 1
ovdd-supply:
maxItems: 1
description: I/O voltage
......@@ -63,6 +66,8 @@ required:
- reset-gpios
- extcon
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -83,6 +83,9 @@ required:
- compatible
- ports
additionalProperties: false
examples:
- |
lvds-encoder {
......
......@@ -89,6 +89,8 @@ required:
- iommus
- operating-points-v2
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/qcom,gpucc-sdm845.h>
......
......@@ -49,6 +49,8 @@ required:
- interrupts
- interrupt-names
additionalProperties: false
examples:
- |
dmc0: dmc@200000 {
......
......@@ -28,6 +28,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
fsi2spi@1c00 {
......
......@@ -64,6 +64,8 @@ required:
- gpio-ranges
- socionext,interrupt-ranges
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -44,6 +44,8 @@ required:
- interrupts
- pulses-per-revolution
additionalProperties: false
examples:
- |
fpga_axi: fpga-axi {
......
......@@ -65,6 +65,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
i2c {
......
......@@ -29,10 +29,14 @@ properties:
mount-matrix:
description: an optional 3x3 mounting rotation matrix.
spi-max-frequency: true
required:
- compatible
- reg
additionalProperties: false
examples:
- |
# include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -62,6 +62,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
i2c {
......
......@@ -36,6 +36,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
i2c {
......
......@@ -48,6 +48,8 @@ properties:
description:
End of conversion interrupt.
io-channel-ranges: true
required:
- compatible
- reg
......@@ -232,6 +234,8 @@ allOf:
enum: [ 1, 2, 4, 8, 16 ]
default: 1
additionalProperties: false
examples:
- |
spmi_bus {
......
......@@ -25,10 +25,14 @@ properties:
description: Optional external reference. If not supplied, assume
REFSEL input tied low to enable the internal reference.
spi-max-frequency: true
required:
- compatible
- reg
additionalProperties: false
examples:
- |
spi {
......
......@@ -33,6 +33,8 @@ required:
- compatible
- ctrl-gpios
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -47,11 +47,17 @@ properties:
vddio-supply:
description: Regulator that provides power to the bus
spi-max-frequency: true
spi-cpha: true
spi-cpol: true
required:
- compatible
- reg
- interrupts
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -32,6 +32,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
i2c {
......
......@@ -26,6 +26,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -26,6 +26,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -61,6 +61,8 @@ required:
- sharp,proximity-far-hysteresis
- sharp,proximity-close-hysteresis
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -55,6 +55,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -47,6 +47,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
i2c {
......
......@@ -49,6 +49,8 @@ required:
- "#interrupt-cells"
- interrupt-controller
additionalProperties: false
examples:
- |
intc: interrupt-controller@10001000 {
......
......@@ -46,6 +46,8 @@ required:
- loongson,msi-base-vec
- loongson,msi-num-vecs
additionalProperties: true #fixme
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -41,6 +41,8 @@ required:
- interrupt-controller
- '#interrupt-cells'
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -49,6 +49,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
smic@fff3a000 {
......
......@@ -58,6 +58,12 @@ properties:
- 2 # D1~6 with VOUT, D7~9 with VDD
- 3 # D1~9 are connected to VOUT
'#address-cells':
const: 1
'#size-cells':
const: 0
patternProperties:
"(^led@[0-9a-f]$|led)":
type: object
......@@ -98,6 +104,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
#include <dt-bindings/leds/common.h>
......
......@@ -49,6 +49,8 @@ required:
- reg
- interrupts
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -119,6 +119,8 @@ required:
- reg
- port
additionalProperties: false
examples:
- |
i2c@e66d8000 {
......
......@@ -55,6 +55,8 @@ required:
- clocks
- clock-names
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/imx8mq-clock.h>
......
......@@ -92,6 +92,8 @@ required:
- video-decoder
- video-encoder
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -119,6 +119,8 @@ required:
- video-decoder
- video-encoder
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -108,6 +108,8 @@ required:
- video-decoder
- video-encoder
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -103,6 +103,8 @@ required:
- video-core0
- video-core1
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -120,6 +120,8 @@ required:
- video-core0
- video-core1
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -33,6 +33,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/imx6qdl-clock.h>
......
......@@ -194,6 +194,8 @@ required:
- clocks
- ranges
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -144,6 +144,8 @@ required:
- "#address-cells"
- "#size-cells"
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -26,6 +26,12 @@ properties:
reg:
maxItems: 1
'#address-cells':
const: 1
'#size-cells':
const: 1
select:
properties:
compatible:
......@@ -36,15 +42,19 @@ select:
required:
- compatible
patternProperties:
"^gpio@[0-9a-f]+$":
$ref: /schemas/gpio/xylon,logicvc-gpio.yaml#
required:
- compatible
- reg
additionalProperties: false
examples:
- |
logicvc: logicvc@43c00000 {
compatible = "xylon,logicvc-3.02.a", "syscon", "simple-mfd";
reg = <0x43c00000 0x6000>;
#address-cells = <1>;
#size-cells = <1>;
};
......@@ -32,12 +32,16 @@ properties:
clocks:
maxItems: 1
device_type: true
required:
- device_type
- compatible
- reg
- clocks
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/jz4780-cgu.h>
......@@ -52,7 +56,6 @@ examples:
reg = <0>;
clocks = <&cgu JZ4780_CLK_CPU>;
clock-names = "cpu";
};
cpu1: cpu@1 {
......@@ -61,7 +64,6 @@ examples:
reg = <1>;
clocks = <&cgu JZ4780_CLK_CORE1>;
clock-names = "cpu";
};
};
...
......@@ -23,6 +23,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
isa@0 {
......
......@@ -36,6 +36,8 @@ required:
- compatible
- reset-gpios
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -28,6 +28,8 @@ required:
- powerdown-gpios
- reset-gpios
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -50,6 +50,8 @@ properties:
required:
- compatible
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -144,6 +144,8 @@ oneOf:
- required:
- memory-region
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/irq.h>
......
......@@ -37,6 +37,8 @@ properties:
required:
- compatible
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......@@ -49,6 +51,6 @@ examples:
bluetooth {
compatible = "realtek,rtl8723bs-bt";
device-wake-gpios = <&r_pio 0 5 GPIO_ACTIVE_HIGH>; /* PL5 */
host-wakeup-gpios = <&r_pio 0 6 GPIO_ACTIVE_HIGH>; /* PL6 */
host-wake-gpios = <&r_pio 0 6 GPIO_ACTIVE_HIGH>; /* PL6 */
};
};
......@@ -18,6 +18,8 @@ properties:
compatible:
const: microchip,wilc1000
reg: true
spi-max-frequency: true
interrupts:
......@@ -34,6 +36,8 @@ required:
- compatible
- interrupts
additionalProperties: false
examples:
- |
spi {
......
......@@ -55,6 +55,8 @@ required:
- clock-names
- max-functions
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/r8a774c0-cpg-mssr.h>
......
......@@ -63,6 +63,8 @@ then:
required:
- power-domains
additionalProperties: false
examples:
- |
phy@36000 {
......
......@@ -42,6 +42,8 @@ required:
- clocks
- clock-names
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/qcom,gcc-ipq806x.h>
......
......@@ -60,6 +60,8 @@ required:
- clocks
- clock-names
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/qcom,gcc-ipq806x.h>
......
......@@ -158,6 +158,7 @@ required:
- vdda-phy-dpdm-supply
- resets
additionalProperties: false
examples:
- |
......
......@@ -36,6 +36,8 @@ required:
- reset-names
- "#phy-cells"
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/qcom,gcc-ipq4019.h>
......
......@@ -188,3 +188,5 @@ required:
- gpio-ranges
- pinctrl-0
- pinctrl-names
additionalProperties: false
......@@ -28,6 +28,8 @@ properties:
required:
- compatible
additionalProperties: false
examples:
- |
// The UniPhier pinctrl should be a subnode of a "syscon" compatible node.
......
......@@ -27,6 +27,8 @@ required:
- compatible
- "#power-domain-cells"
additionalProperties: false
examples:
- |
secure-monitor {
......
......@@ -50,6 +50,8 @@ patternProperties:
- exit-latency-us
- min-residency-us
additionalProperties: false
examples:
- |
......
......@@ -26,6 +26,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
cpc@1bde0000 {
......
......@@ -52,6 +52,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
i2c {
......
......@@ -112,6 +112,12 @@ properties:
# threshold, and the current is below this setting (7 in above chart)
# See also Documentation/devicetree/bindings/power/supply/battery.txt
reg:
maxItems: 1
interrupts:
maxItems: 1
monitored-battery:
description:
phandle of battery characteristics devicetree node
......@@ -137,6 +143,8 @@ properties:
required:
- compatible
additionalProperties: false
examples:
- |
i2c {
......
......@@ -31,6 +31,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
i2c {
......
......@@ -25,6 +25,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
imem@146bf000 {
......
......@@ -68,6 +68,8 @@ required:
- interrupts
- reg
additionalProperties: false
allOf:
- if:
properties:
......
......@@ -32,6 +32,8 @@ required:
- interrupts
- clocks
additionalProperties: false
examples:
- |
aliases {
......
......@@ -56,6 +56,8 @@ required:
- reg
- interrupts
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -173,6 +173,7 @@ patternProperties:
- compatible
- interrupts
additionalProperties: false
examples:
- |
......
......@@ -37,6 +37,8 @@ required:
- reg
- resets
additionalProperties: false
examples:
- |
#include <dt-bindings/reset/amlogic,meson-g12a-audio-reset.h>
......
......@@ -84,6 +84,8 @@ required:
- model
- dai-link-0
additionalProperties: false
examples:
- |
sound {
......
......@@ -42,6 +42,8 @@ required:
- clock-names
- resets
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/g12a-clkc.h>
......
......@@ -46,6 +46,8 @@ required:
- reg
- "#sound-dai-cells"
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -74,6 +74,8 @@ required:
- fsl,asrc-rate
- fsl,asrc-format
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/imx8mn-clock.h>
......
......@@ -52,6 +52,8 @@ required:
- clock-names
- interrupts
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -64,6 +64,8 @@ required:
- assigned-clock-parents
- sound-name-prefix
additionalProperties: false
examples:
- |
#include<dt-bindings/clock/tegra186-clock.h>
......
......@@ -81,6 +81,8 @@ required:
- dmas
- dma-names
additionalProperties: false
examples:
- |
admaif@702d0000 {
......
......@@ -64,6 +64,8 @@ required:
- assigned-clocks
- assigned-clock-parents
additionalProperties: false
examples:
- |
#include<dt-bindings/clock/tegra210-car.h>
......
......@@ -82,6 +82,8 @@ required:
- assigned-clocks
- assigned-clock-parents
additionalProperties: false
examples:
- |
#include<dt-bindings/clock/tegra210-car.h>
......
......@@ -53,6 +53,8 @@ required:
- rockchip,grf
- "#sound-dai-cells"
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -138,6 +138,8 @@ required:
- compatible
- reg
additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
......
......@@ -59,6 +59,9 @@ properties:
resets:
maxItems: 1
"#thermal-sensor-cells":
const: 0
if:
properties:
compatible:
......@@ -79,6 +82,8 @@ else:
- power-domains
- resets
additionalProperties: false
examples:
# Example (non interrupt support)
- |
......
......@@ -68,6 +68,8 @@ patternProperties:
- nvmem-cells
- nvmem-cell-names
additionalProperties: false
required:
- compatible
- reg
......@@ -79,6 +81,8 @@ required:
- "#address-cells"
- "#size-cells"
additionalProperties: false
examples:
- |
ap_thm0: thermal@32200000 {
......
......@@ -44,6 +44,8 @@ properties:
required:
- '#cooling-cells'
additionalProperties: false
examples:
- |
#include <dt-bindings/thermal/thermal.h>
......
......@@ -218,6 +218,8 @@ patternProperties:
- trips
additionalProperties: false
additionalProperties: false
examples:
- |
#include <dt-bindings/interrupt-controller/arm-gic.h>
......
......@@ -36,6 +36,8 @@ required:
- interrupts
- clocks
additionalProperties: false
examples:
- |
ttc0: ttc0@f8001000 {
......
......@@ -155,6 +155,8 @@ allOf:
clock-names:
maxItems: 4
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/tegra210-car.h>
......
......@@ -121,6 +121,8 @@ required:
- interrupts
- interrupt-names
additionalProperties: false
examples:
- |
#include <dt-bindings/clock/qcom,gcc-sdm845.h>
......
......@@ -46,6 +46,22 @@ properties:
VBUS pin of the SoC via a 1/3 voltage divider.
type: boolean
assigned-clocks:
maxItems: 1
assigned-clock-parents:
maxItems: 1
'#address-cells':
const: 2
'#size-cells':
const: 2
patternProperties:
"^usb@":
type: object
required:
- compatible
- reg
......@@ -53,6 +69,8 @@ required:
- clocks
- clock-names
additionalProperties: false
examples:
- |
#include <dt-bindings/soc/ti,sci_pm_domain.h>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment