From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Rob Herring <robh@kernel.org>
Cc: <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
Albert Ou <aou@eecs.berkeley.edu>,
Alessandro Zummo <a.zummo@towertech.it>,
"Andrew Lunn" <andrew@lunn.ch>,
Bjorn Helgaas <bhelgaas@google.com>,
Chanwoo Choi <cw00.choi@samsung.com>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
"David S. Miller" <davem@davemloft.net>,
<dmaengine@vger.kernel.org>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
<dri-devel@lists.freedesktop.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Heiner Kallweit <hkallweit1@gmail.com>,
Jakub Kicinski <kuba@kernel.org>,
Jason Cooper <jason@lakedaemon.net>, Jens Axboe <axboe@kernel.dk>,
Jonathan Cameron <jic23@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
"Lars-Peter Clausen" <lars@metafoo.de>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Lee Jones <lee.jones@linaro.org>,
Liam Girdwood <lgirdwood@gmail.com>,
Linus Walleij <linus.walleij@linaro.org>,
Marc Kleine-Budde <mkl@pengutronix.de>,
"Marc Zyngier" <maz@kernel.org>, Mark Brown <broonie@kernel.org>,
"Mathieu Poirier" <mathieu.poirier@linaro.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Miquel Raynal <miquel.raynal@bootlin.com>,
"MyungJoo Ham" <myungjoo.ham@samsung.com>,
Palmer Dabbelt <palmer@dabbelt.com>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
Pavel Machek <pavel@ucw.cz>,
"Peter Meerwald-Stadler" <pmeerw@pmeerw.net>,
Richard Weinberger <richard@nod.at>,
Sam Ravnborg <sam@ravnborg.org>,
Sebastian Reichel <sre@kernel.org>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
Stephen Boyd <sboyd@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>,
"Thomas Bogendoerfer" <tsbogend@alpha.franken.de>,
Thomas Gleixner <tglx@linutronix.de>,
Ulf Hansson <ulf.hansson@linaro.org>,
"Vignesh Raghavendra" <vigneshr@ti.com>,
Vinod Koul <vkoul@kernel.org>, Viresh Kumar <vireshk@kernel.org>,
Vivien Didelot <vivien.didelot@gmail.com>,
"Wim Van Sebroeck" <wim@linux-watchdog.org>,
Wolfgang Grandegger <wg@grandegger.com>,
Zhang Rui <rui.zhang@intel.com>,
<linux-arm-kernel@lists.infradead.org>,
<linux-can@vger.kernel.org>, <linux-clk@vger.kernel.org>,
<linux-gpio@vger.kernel.org>, <linux-hwmon@vger.kernel.org>,
<linux-i2c@vger.kernel.org>, <linux-ide@vger.kernel.org>,
<linux-iio@vger.kernel.org>, <linux-input@vger.kernel.org>,
<linux-leds@vger.kernel.org>, <linux-media@vger.kernel.org>,
<linux-mips@vger.kernel.org>, <linux-mmc@vger.kernel.org>,
<linux-mtd@lists.infradead.org>, <linux-pci@vger.kernel.org>,
<linux-pm@vger.kernel.org>, <linux-pwm@vger.kernel.org>,
<linux-riscv@lists.infradead.org>, <linux-rtc@vger.kernel.org>,
<linux-serial@vger.kernel.org>, <linux-spi@vger.kernel.org>,
<linux-usb@vger.kernel.org>, <linux-watchdog@vger.kernel.org>
Subject: Re: [PATCH 2/4] dt-bindings: Use 'additionalProperties' instead of 'unevaluatedProperties'
Date: Tue, 6 Oct 2020 10:58:22 +0100 [thread overview]
Message-ID: <20201006095822.00005f32@Huawei.com> (raw)
In-Reply-To: <20201005183830.486085-3-robh@kernel.org>
On Mon, 5 Oct 2020 13:38:28 -0500
Rob Herring <robh@kernel.org> wrote:
> In cases where we don't reference another schema, 'additionalProperties'
> can be used instead. This is preferred for now as 'unevaluatedProperties'
> support isn't implemented yet.
>
> In a few cases, this means adding some missing property definitions of
> which most are for SPI bus properties. 'unevaluatedProperties' is not going
> to work for the SPI bus properties anyways as they are evaluated from the
> parent node, not the SPI child node.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> .../devicetree/bindings/clock/baikal,bt1-ccu-div.yaml | 6 +++++-
> .../devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml | 2 +-
> .../devicetree/bindings/clock/imx8m-clock.yaml | 2 +-
> .../devicetree/bindings/hwmon/baikal,bt1-pvt.yaml | 4 ++--
> .../devicetree/bindings/iio/accel/adi,adis16240.yaml | 4 +++-
> .../devicetree/bindings/iio/accel/adi,adxl345.yaml | 4 +++-
> .../devicetree/bindings/iio/accel/adi,adxl372.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7124.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7192.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7292.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7606.yaml | 6 +++++-
> .../devicetree/bindings/iio/adc/adi,ad7923.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/maxim,max1241.yaml | 4 +++-
> .../devicetree/bindings/iio/dac/adi,ad5770r.yaml | 10 +++++++++-
> .../devicetree/bindings/iio/frequency/adf4371.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/adi,adis16460.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/adi,adis16475.yaml | 2 +-
> .../devicetree/bindings/iio/imu/bosch,bmi160.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 4 +++-
> .../bindings/interrupt-controller/mti,gic.yaml | 2 +-
> .../devicetree/bindings/mfd/cirrus,lochnagar.yaml | 10 +++++++++-
> .../bindings/mfd/ti,j721e-system-controller.yaml | 5 ++++-
> .../devicetree/bindings/misc/olpc,xo1.75-ec.yaml | 4 +++-
> .../opp/allwinner,sun50i-h6-operating-points.yaml | 4 +++-
> .../bindings/regulator/qcom-labibb-regulator.yaml | 2 +-
> .../devicetree/bindings/timer/snps,dw-apb-timer.yaml | 2 +-
> 26 files changed, 82 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> index 2821425ee445..bd4cefbb1244 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> @@ -134,7 +134,11 @@ properties:
> "#reset-cells":
> const: 1
>
> -unevaluatedProperties: false
> + clocks: true
> +
> + clock-names: true
> +
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> index 97131bfa6f87..624984d51c10 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> @@ -101,7 +101,7 @@ properties:
> clock-names:
> const: ref_clk
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> index 31e7cc9693c3..625f573a7b90 100644
> --- a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> @@ -96,7 +96,7 @@ allOf:
> - const: clk_ext3
> - const: clk_ext4
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> # Clock Control Module node:
> diff --git a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> index 84ae4cdd08ed..00a6511354e6 100644
> --- a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> @@ -79,7 +79,7 @@ properties:
> minimum: 0
> maximum: 7130
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> @@ -99,7 +99,7 @@ examples:
>
> interrupts = <GIC_SHARED 31 IRQ_TYPE_LEVEL_HIGH>;
>
> - baikal,pvt-temp-trim-millicelsius = <1000>;
> + baikal,pvt-temp-offset-millicelsius = <1000>;
>
> clocks = <&ccu_sys>, <&ccu_sys>;
> clock-names = "ref", "pclk";
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> index 8589b722028d..4fcbfd93e218 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> @@ -25,12 +25,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> index 591ca32181b0..11d32a288535 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> @@ -32,6 +32,8 @@ properties:
>
> spi-cpol: true
>
> + spi-max-frequency: true
> +
> interrupts:
> maxItems: 1
>
> @@ -40,7 +42,7 @@ required:
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> index 64f275c8e2d9..38b59b6454ce 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> @@ -25,12 +25,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> index d0d2880626c2..f1c574c896cb 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> @@ -52,6 +52,8 @@ properties:
> avdd-supply:
> description: avdd supply can be used as reference for conversion.
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> @@ -108,7 +110,7 @@ patternProperties:
> - reg
> - diff-channels
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> index ed363a796e50..e0cc3b2e8957 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> @@ -30,6 +30,8 @@ properties:
>
> spi-cpha: true
>
> + spi-max-frequency: true
> +
> clocks:
> maxItems: 1
> description: phandle to the master clock (mclk)
> @@ -92,7 +94,7 @@ required:
> - spi-cpol
> - spi-cpha
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> index 55e973c6449c..108d202b288f 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> @@ -30,6 +30,8 @@ properties:
>
> spi-cpha: true
>
> + spi-max-frequency: true
> +
> '#address-cells':
> const: 1
>
> @@ -63,7 +65,7 @@ patternProperties:
> required:
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> index 014b020ed0c2..73775174cf57 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> @@ -31,6 +31,10 @@ properties:
>
> spi-cpha: true
>
> + spi-cpol: true
> +
> + spi-max-frequency: true
> +
> avcc-supply: true
>
> interrupts:
> @@ -102,7 +106,7 @@ required:
> - interrupts
> - adi,conversion-start-gpios
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> index 2a17641faed5..e82194974eea 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> @@ -43,11 +43,13 @@ properties:
> '#size-cells':
> const: 0
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml b/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> index 181213b862db..4c7e0d94bff1 100644
> --- a/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> @@ -39,13 +39,15 @@ properties:
> thus enabling power-down mode.
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - vdd-supply
> - vref-supply
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> index faef288b7148..fb2c48fc7ce4 100644
> --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> @@ -49,6 +49,14 @@ properties:
> asserted during driver probe.
> maxItems: 1
>
> + spi-max-frequency: true
> +
> + '#address-cells':
> + const: 1
> +
> + '#size-cells':
> + const: 0
> +
> channel@0:
> description: Represents an external channel which are
> connected to the DAC. Channel 0 can act both as a current
> @@ -130,7 +138,7 @@ required:
> - channel@4
> - channel@5
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> index 11d445f7010e..6b3a611e1cf1 100644
> --- a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> +++ b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> @@ -40,13 +40,15 @@ properties:
> output stage will shut down until the ADF4371/ADF4372 achieves lock as
> measured by the digital lock detect circuitry.
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - clocks
> - clock-names
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> index 07c8ed4ee0f1..340be256f283 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> @@ -25,6 +25,8 @@ properties:
>
> spi-cpol: true
>
> + spi-max-frequency: true
> +
> interrupts:
> maxItems: 1
>
> @@ -33,7 +35,7 @@ required:
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> index c29385697bbf..79fba1508e89 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> @@ -116,7 +116,7 @@ allOf:
> dependencies:
> adi,sync-mode: [ clocks ]
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> index 4f215399c8df..6e73cd889b5c 100644
> --- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> @@ -46,11 +46,13 @@ properties:
> mount-matrix:
> description: an optional 3x3 mounting rotation matrix
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> index 716731c2b794..479e7065d4eb 100644
> --- a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> @@ -36,11 +36,13 @@ properties:
> drive-open-drain:
> type: boolean
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml b/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> index ce6aaff15214..039e08af98bb 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> @@ -95,7 +95,7 @@ properties:
>
> additionalProperties: false
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> index 7a616577ac63..c00ad3e21c21 100644
> --- a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> +++ b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> @@ -130,6 +130,14 @@ properties:
> type: object
> $ref: /schemas/pinctrl/cirrus,lochnagar.yaml#
>
> + lochnagar-hwmon:
> + type: object
> + $ref: /schemas/hwmon/cirrus,lochnagar.yaml#
> +
> + lochnagar-sc:
> + type: object
> + $ref: /schemas/sound/cirrus,lochnagar.yaml#
> +
> VDDCORE:
> description:
> Initialisation data for the VDDCORE regulator, which supplies the
> @@ -249,7 +257,7 @@ required:
> - lochnagar-clk
> - lochnagar-pinctrl
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> index da3d9ab758b9..19fcf59fd2fe 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> @@ -30,6 +30,9 @@ properties:
> - const: syscon
> - const: simple-mfd
>
> + reg:
> + maxItems: 1
> +
> "#address-cells":
> const: 1
>
> @@ -54,7 +57,7 @@ required:
> - "#size-cells"
> - ranges
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml b/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> index e75d77beec6a..ade733cd60f7 100644
> --- a/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> +++ b/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> @@ -28,11 +28,13 @@ properties:
> description: GPIO uspecifier of the CMD pin
> maxItems: 1
>
> + spi-cpha: true
> +
> required:
> - compatible
> - cmd-gpios
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> index aef87a33a7c9..aeff2bd774dd 100644
> --- a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> +++ b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> @@ -31,6 +31,8 @@ properties:
> Documentation/devicetree/bindings/nvmem/nvmem.txt and also
> examples below.
>
> + opp-shared: true
> +
> required:
> - compatible
> - nvmem-cells
> @@ -53,7 +55,7 @@ patternProperties:
>
> unevaluatedProperties: false
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> index fb111e2d5b99..53853ec20fe2 100644
> --- a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> @@ -47,7 +47,7 @@ properties:
> required:
> - compatible
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml b/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> index 7b39e3204fb3..2fc617377e2c 100644
> --- a/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> +++ b/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> @@ -45,7 +45,7 @@ properties:
> frequency in HZ, but is defined only for the backwards compatibility
> with the picoxcell platform.
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Rob Herring <robh@kernel.org>
Cc: Andrew Lunn <andrew@lunn.ch>,
Ulf Hansson <ulf.hansson@linaro.org>,
linux-usb@vger.kernel.org,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
Sam Ravnborg <sam@ravnborg.org>,
linux-pwm@vger.kernel.org, linux-iio@vger.kernel.org,
Viresh Kumar <vireshk@kernel.org>,
Linus Walleij <linus.walleij@linaro.org>,
linux-pci@vger.kernel.org, dri-devel@lists.freedesktop.org,
Liam Girdwood <lgirdwood@gmail.com>,
linux-ide@vger.kernel.org,
Thierry Reding <thierry.reding@gmail.com>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Pavel Machek <pavel@ucw.cz>,
Miquel Raynal <miquel.raynal@bootlin.com>,
linux-riscv@lists.infradead.org, linux-hwmon@vger.kernel.org,
Lee Jones <lee.jones@linaro.org>,
linux-clk@vger.kernel.org, linux-leds@vger.kernel.org,
Vignesh Raghavendra <vigneshr@ti.com>,
linux-rtc@vger.kernel.org, Lars-Peter Clausen <lars@metafoo.de>,
Marc Zyngier <maz@kernel.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Chanwoo Choi <cw00.choi@samsung.com>,
MyungJoo Ham <myungjoo.ham@samsung.com>,
linux-serial@vger.kernel.org, linux-input@vger.kernel.org,
Jakub Kicinski <kuba@kernel.org>, Zhang Rui <rui.zhang@intel.com>,
linux-mips@vger.kernel.org,
Vivien Didelot <vivien.didelot@gmail.com>,
Wolfgang Grandegger <wg@grandegger.com>,
linux-media@vger.kernel.org, devicetree@vger.kernel.org,
Albert Ou <aou@eecs.berkeley.edu>,
Jason Cooper <jason@lakedaemon.net>,
linux-pm@vger.kernel.org, linux-can@vger.kernel.org,
linux-gpio@vger.kernel.org, Mark Brown <broonie@kernel.org>,
Marc Kleine-Budde <mkl@pengutronix.de>,
Sebastian Reichel <sre@kernel.org>,
Paul Walmsley <paul.walmsley@sifive.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Thomas Gleixner <tglx@linutronix.de>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
linux-arm-kernel@lists.infradead.org,
Jens Axboe <axboe@kernel.dk>,
Alessandro Zummo <a.zummo@towertech.it>,
linux-watchdog@vger.kernel.org,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
Mathieu Poirier <mathieu.poirier@linaro.org>,
Stephen Boyd <sboyd@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-spi@vger.kernel.org, linux-i2c@vger.kernel.org,
Vinod Koul <vkoul@kernel.org>,
Palmer Dabbelt <palmer@dabbelt.com>,
Wim Van Sebroeck <wim@linux-watchdog.org>,
Richard Weinberger <richard@nod.at>,
dmaengine@vger.kernel.org, linux-mtd@lists.infradead.org,
"David S. Miller" <davem@davemloft.net>,
Jonathan Cameron <jic23@kernel.org>,
Heiner Kallweit <hkallweit1@gmail.com>
Subject: Re: [PATCH 2/4] dt-bindings: Use 'additionalProperties' instead of 'unevaluatedProperties'
Date: Tue, 6 Oct 2020 10:58:22 +0100 [thread overview]
Message-ID: <20201006095822.00005f32@Huawei.com> (raw)
In-Reply-To: <20201005183830.486085-3-robh@kernel.org>
On Mon, 5 Oct 2020 13:38:28 -0500
Rob Herring <robh@kernel.org> wrote:
> In cases where we don't reference another schema, 'additionalProperties'
> can be used instead. This is preferred for now as 'unevaluatedProperties'
> support isn't implemented yet.
>
> In a few cases, this means adding some missing property definitions of
> which most are for SPI bus properties. 'unevaluatedProperties' is not going
> to work for the SPI bus properties anyways as they are evaluated from the
> parent node, not the SPI child node.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> .../devicetree/bindings/clock/baikal,bt1-ccu-div.yaml | 6 +++++-
> .../devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml | 2 +-
> .../devicetree/bindings/clock/imx8m-clock.yaml | 2 +-
> .../devicetree/bindings/hwmon/baikal,bt1-pvt.yaml | 4 ++--
> .../devicetree/bindings/iio/accel/adi,adis16240.yaml | 4 +++-
> .../devicetree/bindings/iio/accel/adi,adxl345.yaml | 4 +++-
> .../devicetree/bindings/iio/accel/adi,adxl372.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7124.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7192.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7292.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7606.yaml | 6 +++++-
> .../devicetree/bindings/iio/adc/adi,ad7923.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/maxim,max1241.yaml | 4 +++-
> .../devicetree/bindings/iio/dac/adi,ad5770r.yaml | 10 +++++++++-
> .../devicetree/bindings/iio/frequency/adf4371.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/adi,adis16460.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/adi,adis16475.yaml | 2 +-
> .../devicetree/bindings/iio/imu/bosch,bmi160.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 4 +++-
> .../bindings/interrupt-controller/mti,gic.yaml | 2 +-
> .../devicetree/bindings/mfd/cirrus,lochnagar.yaml | 10 +++++++++-
> .../bindings/mfd/ti,j721e-system-controller.yaml | 5 ++++-
> .../devicetree/bindings/misc/olpc,xo1.75-ec.yaml | 4 +++-
> .../opp/allwinner,sun50i-h6-operating-points.yaml | 4 +++-
> .../bindings/regulator/qcom-labibb-regulator.yaml | 2 +-
> .../devicetree/bindings/timer/snps,dw-apb-timer.yaml | 2 +-
> 26 files changed, 82 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> index 2821425ee445..bd4cefbb1244 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> @@ -134,7 +134,11 @@ properties:
> "#reset-cells":
> const: 1
>
> -unevaluatedProperties: false
> + clocks: true
> +
> + clock-names: true
> +
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> index 97131bfa6f87..624984d51c10 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> @@ -101,7 +101,7 @@ properties:
> clock-names:
> const: ref_clk
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> index 31e7cc9693c3..625f573a7b90 100644
> --- a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> @@ -96,7 +96,7 @@ allOf:
> - const: clk_ext3
> - const: clk_ext4
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> # Clock Control Module node:
> diff --git a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> index 84ae4cdd08ed..00a6511354e6 100644
> --- a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> @@ -79,7 +79,7 @@ properties:
> minimum: 0
> maximum: 7130
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> @@ -99,7 +99,7 @@ examples:
>
> interrupts = <GIC_SHARED 31 IRQ_TYPE_LEVEL_HIGH>;
>
> - baikal,pvt-temp-trim-millicelsius = <1000>;
> + baikal,pvt-temp-offset-millicelsius = <1000>;
>
> clocks = <&ccu_sys>, <&ccu_sys>;
> clock-names = "ref", "pclk";
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> index 8589b722028d..4fcbfd93e218 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> @@ -25,12 +25,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> index 591ca32181b0..11d32a288535 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> @@ -32,6 +32,8 @@ properties:
>
> spi-cpol: true
>
> + spi-max-frequency: true
> +
> interrupts:
> maxItems: 1
>
> @@ -40,7 +42,7 @@ required:
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> index 64f275c8e2d9..38b59b6454ce 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> @@ -25,12 +25,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> index d0d2880626c2..f1c574c896cb 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> @@ -52,6 +52,8 @@ properties:
> avdd-supply:
> description: avdd supply can be used as reference for conversion.
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> @@ -108,7 +110,7 @@ patternProperties:
> - reg
> - diff-channels
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> index ed363a796e50..e0cc3b2e8957 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> @@ -30,6 +30,8 @@ properties:
>
> spi-cpha: true
>
> + spi-max-frequency: true
> +
> clocks:
> maxItems: 1
> description: phandle to the master clock (mclk)
> @@ -92,7 +94,7 @@ required:
> - spi-cpol
> - spi-cpha
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> index 55e973c6449c..108d202b288f 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> @@ -30,6 +30,8 @@ properties:
>
> spi-cpha: true
>
> + spi-max-frequency: true
> +
> '#address-cells':
> const: 1
>
> @@ -63,7 +65,7 @@ patternProperties:
> required:
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> index 014b020ed0c2..73775174cf57 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> @@ -31,6 +31,10 @@ properties:
>
> spi-cpha: true
>
> + spi-cpol: true
> +
> + spi-max-frequency: true
> +
> avcc-supply: true
>
> interrupts:
> @@ -102,7 +106,7 @@ required:
> - interrupts
> - adi,conversion-start-gpios
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> index 2a17641faed5..e82194974eea 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> @@ -43,11 +43,13 @@ properties:
> '#size-cells':
> const: 0
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml b/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> index 181213b862db..4c7e0d94bff1 100644
> --- a/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> @@ -39,13 +39,15 @@ properties:
> thus enabling power-down mode.
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - vdd-supply
> - vref-supply
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> index faef288b7148..fb2c48fc7ce4 100644
> --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> @@ -49,6 +49,14 @@ properties:
> asserted during driver probe.
> maxItems: 1
>
> + spi-max-frequency: true
> +
> + '#address-cells':
> + const: 1
> +
> + '#size-cells':
> + const: 0
> +
> channel@0:
> description: Represents an external channel which are
> connected to the DAC. Channel 0 can act both as a current
> @@ -130,7 +138,7 @@ required:
> - channel@4
> - channel@5
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> index 11d445f7010e..6b3a611e1cf1 100644
> --- a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> +++ b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> @@ -40,13 +40,15 @@ properties:
> output stage will shut down until the ADF4371/ADF4372 achieves lock as
> measured by the digital lock detect circuitry.
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - clocks
> - clock-names
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> index 07c8ed4ee0f1..340be256f283 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> @@ -25,6 +25,8 @@ properties:
>
> spi-cpol: true
>
> + spi-max-frequency: true
> +
> interrupts:
> maxItems: 1
>
> @@ -33,7 +35,7 @@ required:
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> index c29385697bbf..79fba1508e89 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> @@ -116,7 +116,7 @@ allOf:
> dependencies:
> adi,sync-mode: [ clocks ]
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> index 4f215399c8df..6e73cd889b5c 100644
> --- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> @@ -46,11 +46,13 @@ properties:
> mount-matrix:
> description: an optional 3x3 mounting rotation matrix
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> index 716731c2b794..479e7065d4eb 100644
> --- a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> @@ -36,11 +36,13 @@ properties:
> drive-open-drain:
> type: boolean
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml b/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> index ce6aaff15214..039e08af98bb 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> @@ -95,7 +95,7 @@ properties:
>
> additionalProperties: false
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> index 7a616577ac63..c00ad3e21c21 100644
> --- a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> +++ b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> @@ -130,6 +130,14 @@ properties:
> type: object
> $ref: /schemas/pinctrl/cirrus,lochnagar.yaml#
>
> + lochnagar-hwmon:
> + type: object
> + $ref: /schemas/hwmon/cirrus,lochnagar.yaml#
> +
> + lochnagar-sc:
> + type: object
> + $ref: /schemas/sound/cirrus,lochnagar.yaml#
> +
> VDDCORE:
> description:
> Initialisation data for the VDDCORE regulator, which supplies the
> @@ -249,7 +257,7 @@ required:
> - lochnagar-clk
> - lochnagar-pinctrl
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> index da3d9ab758b9..19fcf59fd2fe 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> @@ -30,6 +30,9 @@ properties:
> - const: syscon
> - const: simple-mfd
>
> + reg:
> + maxItems: 1
> +
> "#address-cells":
> const: 1
>
> @@ -54,7 +57,7 @@ required:
> - "#size-cells"
> - ranges
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml b/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> index e75d77beec6a..ade733cd60f7 100644
> --- a/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> +++ b/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> @@ -28,11 +28,13 @@ properties:
> description: GPIO uspecifier of the CMD pin
> maxItems: 1
>
> + spi-cpha: true
> +
> required:
> - compatible
> - cmd-gpios
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> index aef87a33a7c9..aeff2bd774dd 100644
> --- a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> +++ b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> @@ -31,6 +31,8 @@ properties:
> Documentation/devicetree/bindings/nvmem/nvmem.txt and also
> examples below.
>
> + opp-shared: true
> +
> required:
> - compatible
> - nvmem-cells
> @@ -53,7 +55,7 @@ patternProperties:
>
> unevaluatedProperties: false
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> index fb111e2d5b99..53853ec20fe2 100644
> --- a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> @@ -47,7 +47,7 @@ properties:
> required:
> - compatible
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml b/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> index 7b39e3204fb3..2fc617377e2c 100644
> --- a/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> +++ b/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> @@ -45,7 +45,7 @@ properties:
> frequency in HZ, but is defined only for the backwards compatibility
> with the picoxcell platform.
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Rob Herring <robh@kernel.org>
Cc: Andrew Lunn <andrew@lunn.ch>,
Ulf Hansson <ulf.hansson@linaro.org>,
linux-usb@vger.kernel.org,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
Sam Ravnborg <sam@ravnborg.org>,
linux-pwm@vger.kernel.org, linux-iio@vger.kernel.org,
Viresh Kumar <vireshk@kernel.org>,
Linus Walleij <linus.walleij@linaro.org>,
linux-pci@vger.kernel.org, dri-devel@lists.freedesktop.org,
Liam Girdwood <lgirdwood@gmail.com>,
linux-ide@vger.kernel.org,
Thierry Reding <thierry.reding@gmail.com>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Pavel Machek <pavel@ucw.cz>,
Miquel Raynal <miquel.raynal@bootlin.com>,
linux-riscv@lists.infradead.org, linux-hwmon@vger.kernel.org,
Lee Jones <lee.jones@linaro.org>,
linux-clk@vger.kernel.org, linux-leds@vger.kernel.org,
Vignesh Raghavendra <vigneshr@ti.com>,
linux-rtc@vger.kernel.org, Lars-Peter Clausen <lars@metafoo.de>,
Marc Zyngier <maz@kernel.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Chanwoo Choi <cw00.choi@samsung.com>,
MyungJoo Ham <myungjoo.ham@samsung.com>,
linux-serial@vger.kernel.org, linux-input@vger.kernel.org,
Jakub Kicinski <kuba@kernel.org>, Zhang Rui <rui.zhang@intel.com>,
linux-mips@vger.kernel.org,
Vivien Didelot <vivien.didelot@gmail.com>,
Wolfgang Grandegger <wg@grandegger.com>,
linux-media@vger.kernel.org, devicetree@vger.kernel.org,
Albert Ou <aou@eecs.berkeley.edu>,
Jason Cooper <jason@lakedaemon.net>,
linux-pm@vger.kernel.org, linux-can@vger.kernel.org,
linux-gpio@vger.kernel.org, Mark Brown <broonie@kernel.org>,
Marc Kleine-Budde <mkl@pengutronix.de>,
Sebastian Reichel <sre@kernel.org>,
Paul Walmsley <paul.walmsley@sifive.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Thomas Gleixner <tglx@linutronix.de>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
linux-arm-kernel@lists.infradead.org,
Jens Axboe <axboe@kernel.dk>,
Alessandro Zummo <a.zummo@towertech.it>,
linux-watchdog@vger.kernel.org,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
Mathieu Poirier <mathieu.poirier@linaro.org>,
Stephen Boyd <sboyd@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-spi@vger.kernel.org, linux-i2c@vger.kernel.org,
Vinod Koul <vkoul@kernel.org>,
Palmer Dabbelt <palmer@dabbelt.com>,
Wim Van Sebroeck <wim@linux-watchdog.org>,
Richard Weinberger <richard@nod.at>,
dmaengine@vger.kernel.org, linux-mtd@lists.infradead.org,
"David S. Miller" <davem@davemloft.net>,
Jonathan Cameron <jic23@kernel.org>,
Heiner Kallweit <hkallweit1@gmail.com>
Subject: Re: [PATCH 2/4] dt-bindings: Use 'additionalProperties' instead of 'unevaluatedProperties'
Date: Tue, 6 Oct 2020 10:58:22 +0100 [thread overview]
Message-ID: <20201006095822.00005f32@Huawei.com> (raw)
In-Reply-To: <20201005183830.486085-3-robh@kernel.org>
On Mon, 5 Oct 2020 13:38:28 -0500
Rob Herring <robh@kernel.org> wrote:
> In cases where we don't reference another schema, 'additionalProperties'
> can be used instead. This is preferred for now as 'unevaluatedProperties'
> support isn't implemented yet.
>
> In a few cases, this means adding some missing property definitions of
> which most are for SPI bus properties. 'unevaluatedProperties' is not going
> to work for the SPI bus properties anyways as they are evaluated from the
> parent node, not the SPI child node.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> .../devicetree/bindings/clock/baikal,bt1-ccu-div.yaml | 6 +++++-
> .../devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml | 2 +-
> .../devicetree/bindings/clock/imx8m-clock.yaml | 2 +-
> .../devicetree/bindings/hwmon/baikal,bt1-pvt.yaml | 4 ++--
> .../devicetree/bindings/iio/accel/adi,adis16240.yaml | 4 +++-
> .../devicetree/bindings/iio/accel/adi,adxl345.yaml | 4 +++-
> .../devicetree/bindings/iio/accel/adi,adxl372.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7124.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7192.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7292.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7606.yaml | 6 +++++-
> .../devicetree/bindings/iio/adc/adi,ad7923.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/maxim,max1241.yaml | 4 +++-
> .../devicetree/bindings/iio/dac/adi,ad5770r.yaml | 10 +++++++++-
> .../devicetree/bindings/iio/frequency/adf4371.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/adi,adis16460.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/adi,adis16475.yaml | 2 +-
> .../devicetree/bindings/iio/imu/bosch,bmi160.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 4 +++-
> .../bindings/interrupt-controller/mti,gic.yaml | 2 +-
> .../devicetree/bindings/mfd/cirrus,lochnagar.yaml | 10 +++++++++-
> .../bindings/mfd/ti,j721e-system-controller.yaml | 5 ++++-
> .../devicetree/bindings/misc/olpc,xo1.75-ec.yaml | 4 +++-
> .../opp/allwinner,sun50i-h6-operating-points.yaml | 4 +++-
> .../bindings/regulator/qcom-labibb-regulator.yaml | 2 +-
> .../devicetree/bindings/timer/snps,dw-apb-timer.yaml | 2 +-
> 26 files changed, 82 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> index 2821425ee445..bd4cefbb1244 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> @@ -134,7 +134,11 @@ properties:
> "#reset-cells":
> const: 1
>
> -unevaluatedProperties: false
> + clocks: true
> +
> + clock-names: true
> +
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> index 97131bfa6f87..624984d51c10 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> @@ -101,7 +101,7 @@ properties:
> clock-names:
> const: ref_clk
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> index 31e7cc9693c3..625f573a7b90 100644
> --- a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> @@ -96,7 +96,7 @@ allOf:
> - const: clk_ext3
> - const: clk_ext4
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> # Clock Control Module node:
> diff --git a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> index 84ae4cdd08ed..00a6511354e6 100644
> --- a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> @@ -79,7 +79,7 @@ properties:
> minimum: 0
> maximum: 7130
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> @@ -99,7 +99,7 @@ examples:
>
> interrupts = <GIC_SHARED 31 IRQ_TYPE_LEVEL_HIGH>;
>
> - baikal,pvt-temp-trim-millicelsius = <1000>;
> + baikal,pvt-temp-offset-millicelsius = <1000>;
>
> clocks = <&ccu_sys>, <&ccu_sys>;
> clock-names = "ref", "pclk";
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> index 8589b722028d..4fcbfd93e218 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> @@ -25,12 +25,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> index 591ca32181b0..11d32a288535 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> @@ -32,6 +32,8 @@ properties:
>
> spi-cpol: true
>
> + spi-max-frequency: true
> +
> interrupts:
> maxItems: 1
>
> @@ -40,7 +42,7 @@ required:
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> index 64f275c8e2d9..38b59b6454ce 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> @@ -25,12 +25,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> index d0d2880626c2..f1c574c896cb 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> @@ -52,6 +52,8 @@ properties:
> avdd-supply:
> description: avdd supply can be used as reference for conversion.
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> @@ -108,7 +110,7 @@ patternProperties:
> - reg
> - diff-channels
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> index ed363a796e50..e0cc3b2e8957 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> @@ -30,6 +30,8 @@ properties:
>
> spi-cpha: true
>
> + spi-max-frequency: true
> +
> clocks:
> maxItems: 1
> description: phandle to the master clock (mclk)
> @@ -92,7 +94,7 @@ required:
> - spi-cpol
> - spi-cpha
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> index 55e973c6449c..108d202b288f 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> @@ -30,6 +30,8 @@ properties:
>
> spi-cpha: true
>
> + spi-max-frequency: true
> +
> '#address-cells':
> const: 1
>
> @@ -63,7 +65,7 @@ patternProperties:
> required:
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> index 014b020ed0c2..73775174cf57 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> @@ -31,6 +31,10 @@ properties:
>
> spi-cpha: true
>
> + spi-cpol: true
> +
> + spi-max-frequency: true
> +
> avcc-supply: true
>
> interrupts:
> @@ -102,7 +106,7 @@ required:
> - interrupts
> - adi,conversion-start-gpios
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> index 2a17641faed5..e82194974eea 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> @@ -43,11 +43,13 @@ properties:
> '#size-cells':
> const: 0
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml b/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> index 181213b862db..4c7e0d94bff1 100644
> --- a/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> @@ -39,13 +39,15 @@ properties:
> thus enabling power-down mode.
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - vdd-supply
> - vref-supply
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> index faef288b7148..fb2c48fc7ce4 100644
> --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> @@ -49,6 +49,14 @@ properties:
> asserted during driver probe.
> maxItems: 1
>
> + spi-max-frequency: true
> +
> + '#address-cells':
> + const: 1
> +
> + '#size-cells':
> + const: 0
> +
> channel@0:
> description: Represents an external channel which are
> connected to the DAC. Channel 0 can act both as a current
> @@ -130,7 +138,7 @@ required:
> - channel@4
> - channel@5
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> index 11d445f7010e..6b3a611e1cf1 100644
> --- a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> +++ b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> @@ -40,13 +40,15 @@ properties:
> output stage will shut down until the ADF4371/ADF4372 achieves lock as
> measured by the digital lock detect circuitry.
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - clocks
> - clock-names
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> index 07c8ed4ee0f1..340be256f283 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> @@ -25,6 +25,8 @@ properties:
>
> spi-cpol: true
>
> + spi-max-frequency: true
> +
> interrupts:
> maxItems: 1
>
> @@ -33,7 +35,7 @@ required:
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> index c29385697bbf..79fba1508e89 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> @@ -116,7 +116,7 @@ allOf:
> dependencies:
> adi,sync-mode: [ clocks ]
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> index 4f215399c8df..6e73cd889b5c 100644
> --- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> @@ -46,11 +46,13 @@ properties:
> mount-matrix:
> description: an optional 3x3 mounting rotation matrix
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> index 716731c2b794..479e7065d4eb 100644
> --- a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> @@ -36,11 +36,13 @@ properties:
> drive-open-drain:
> type: boolean
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml b/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> index ce6aaff15214..039e08af98bb 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> @@ -95,7 +95,7 @@ properties:
>
> additionalProperties: false
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> index 7a616577ac63..c00ad3e21c21 100644
> --- a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> +++ b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> @@ -130,6 +130,14 @@ properties:
> type: object
> $ref: /schemas/pinctrl/cirrus,lochnagar.yaml#
>
> + lochnagar-hwmon:
> + type: object
> + $ref: /schemas/hwmon/cirrus,lochnagar.yaml#
> +
> + lochnagar-sc:
> + type: object
> + $ref: /schemas/sound/cirrus,lochnagar.yaml#
> +
> VDDCORE:
> description:
> Initialisation data for the VDDCORE regulator, which supplies the
> @@ -249,7 +257,7 @@ required:
> - lochnagar-clk
> - lochnagar-pinctrl
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> index da3d9ab758b9..19fcf59fd2fe 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> @@ -30,6 +30,9 @@ properties:
> - const: syscon
> - const: simple-mfd
>
> + reg:
> + maxItems: 1
> +
> "#address-cells":
> const: 1
>
> @@ -54,7 +57,7 @@ required:
> - "#size-cells"
> - ranges
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml b/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> index e75d77beec6a..ade733cd60f7 100644
> --- a/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> +++ b/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> @@ -28,11 +28,13 @@ properties:
> description: GPIO uspecifier of the CMD pin
> maxItems: 1
>
> + spi-cpha: true
> +
> required:
> - compatible
> - cmd-gpios
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> index aef87a33a7c9..aeff2bd774dd 100644
> --- a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> +++ b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> @@ -31,6 +31,8 @@ properties:
> Documentation/devicetree/bindings/nvmem/nvmem.txt and also
> examples below.
>
> + opp-shared: true
> +
> required:
> - compatible
> - nvmem-cells
> @@ -53,7 +55,7 @@ patternProperties:
>
> unevaluatedProperties: false
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> index fb111e2d5b99..53853ec20fe2 100644
> --- a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> @@ -47,7 +47,7 @@ properties:
> required:
> - compatible
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml b/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> index 7b39e3204fb3..2fc617377e2c 100644
> --- a/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> +++ b/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> @@ -45,7 +45,7 @@ properties:
> frequency in HZ, but is defined only for the backwards compatibility
> with the picoxcell platform.
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <Jonathan.Cameron@Huawei.com>
To: Rob Herring <robh@kernel.org>
Cc: Andrew Lunn <andrew@lunn.ch>,
Ulf Hansson <ulf.hansson@linaro.org>,
linux-usb@vger.kernel.org,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
Sam Ravnborg <sam@ravnborg.org>,
linux-pwm@vger.kernel.org, linux-iio@vger.kernel.org,
Viresh Kumar <vireshk@kernel.org>,
linux-pci@vger.kernel.org, dri-devel@lists.freedesktop.org,
Liam Girdwood <lgirdwood@gmail.com>,
linux-ide@vger.kernel.org,
Thierry Reding <thierry.reding@gmail.com>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Pavel Machek <pavel@ucw.cz>,
Miquel Raynal <miquel.raynal@bootlin.com>,
linux-riscv@lists.infradead.org, linux-hwmon@vger.kernel.org,
Lee Jones <lee.jones@linaro.org>,
linux-clk@vger.kernel.org, linux-leds@vger.kernel.org,
Vignesh Raghavendra <vigneshr@ti.com>,
linux-rtc@vger.kernel.org, Marc Zyngier <maz@kernel.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Chanwoo Choi <cw00.choi@samsung.com>,
MyungJoo Ham <myungjoo.ham@samsung.com>,
linux-serial@vger.kernel.org, linux-input@vger.kernel.org,
Jakub Kicinski <kuba@kernel.org>, Zhang Rui <rui.zhang@intel.com>,
linux-mips@vger.kernel.org,
Vivien Didelot <vivien.didelot@gmail.com>,
Wolfgang Grandegger <wg@grandegger.com>,
linux-media@vger.kernel.org, devicetree@vger.kernel.org,
Albert Ou <aou@eecs.berkeley.edu>,
Jason Cooper <jason@lakedaemon.net>,
linux-pm@vger.kernel.org, linux-can@vger.kernel.org,
linux-gpio@vger.kernel.org, Mark Brown <broonie@kernel.org>,
Marc Kleine-Budde <mkl@pengutronix.de>,
Sebastian Reichel <sre@kernel.org>,
Paul Walmsley <paul.walmsley@sifive.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Thomas Gleixner <tglx@linutronix.de>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
linux-arm-kernel@lists.infradead.org,
Jens Axboe <axboe@kernel.dk>,
Alessandro Zummo <a.zummo@towertech.it>,
linux-watchdog@vger.kernel.org,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
Mathieu Poirier <mathieu.poirier@linaro.org>,
Stephen Boyd <sboyd@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-spi@vger.kernel.org, linux-i2c@vger.kernel.org,
Vinod Koul <vkoul@kernel.org>,
Palmer Dabbelt <palmer@dabbelt.com>,
Wim Van Sebroeck <wim@linux-watchdog.org>,
Richard Weinberger <richard@nod.at>,
dmaengine@vger.kernel.org, linux-mtd@lists.infradead.org,
"David S. Miller" <davem@davemloft.net>,
Jonathan Cameron <jic23@kernel.org>,
Heiner Kallweit <hkallweit1@gmail.com>
Subject: Re: [PATCH 2/4] dt-bindings: Use 'additionalProperties' instead of 'unevaluatedProperties'
Date: Tue, 6 Oct 2020 10:58:22 +0100 [thread overview]
Message-ID: <20201006095822.00005f32@Huawei.com> (raw)
In-Reply-To: <20201005183830.486085-3-robh@kernel.org>
On Mon, 5 Oct 2020 13:38:28 -0500
Rob Herring <robh@kernel.org> wrote:
> In cases where we don't reference another schema, 'additionalProperties'
> can be used instead. This is preferred for now as 'unevaluatedProperties'
> support isn't implemented yet.
>
> In a few cases, this means adding some missing property definitions of
> which most are for SPI bus properties. 'unevaluatedProperties' is not going
> to work for the SPI bus properties anyways as they are evaluated from the
> parent node, not the SPI child node.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> ---
> .../devicetree/bindings/clock/baikal,bt1-ccu-div.yaml | 6 +++++-
> .../devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml | 2 +-
> .../devicetree/bindings/clock/imx8m-clock.yaml | 2 +-
> .../devicetree/bindings/hwmon/baikal,bt1-pvt.yaml | 4 ++--
> .../devicetree/bindings/iio/accel/adi,adis16240.yaml | 4 +++-
> .../devicetree/bindings/iio/accel/adi,adxl345.yaml | 4 +++-
> .../devicetree/bindings/iio/accel/adi,adxl372.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7124.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7192.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7292.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/adi,ad7606.yaml | 6 +++++-
> .../devicetree/bindings/iio/adc/adi,ad7923.yaml | 4 +++-
> .../devicetree/bindings/iio/adc/maxim,max1241.yaml | 4 +++-
> .../devicetree/bindings/iio/dac/adi,ad5770r.yaml | 10 +++++++++-
> .../devicetree/bindings/iio/frequency/adf4371.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/adi,adis16460.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/adi,adis16475.yaml | 2 +-
> .../devicetree/bindings/iio/imu/bosch,bmi160.yaml | 4 +++-
> .../devicetree/bindings/iio/imu/nxp,fxos8700.yaml | 4 +++-
> .../bindings/interrupt-controller/mti,gic.yaml | 2 +-
> .../devicetree/bindings/mfd/cirrus,lochnagar.yaml | 10 +++++++++-
> .../bindings/mfd/ti,j721e-system-controller.yaml | 5 ++++-
> .../devicetree/bindings/misc/olpc,xo1.75-ec.yaml | 4 +++-
> .../opp/allwinner,sun50i-h6-operating-points.yaml | 4 +++-
> .../bindings/regulator/qcom-labibb-regulator.yaml | 2 +-
> .../devicetree/bindings/timer/snps,dw-apb-timer.yaml | 2 +-
> 26 files changed, 82 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> index 2821425ee445..bd4cefbb1244 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-div.yaml
> @@ -134,7 +134,11 @@ properties:
> "#reset-cells":
> const: 1
>
> -unevaluatedProperties: false
> + clocks: true
> +
> + clock-names: true
> +
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> index 97131bfa6f87..624984d51c10 100644
> --- a/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> +++ b/Documentation/devicetree/bindings/clock/baikal,bt1-ccu-pll.yaml
> @@ -101,7 +101,7 @@ properties:
> clock-names:
> const: ref_clk
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> index 31e7cc9693c3..625f573a7b90 100644
> --- a/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx8m-clock.yaml
> @@ -96,7 +96,7 @@ allOf:
> - const: clk_ext3
> - const: clk_ext4
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> # Clock Control Module node:
> diff --git a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> index 84ae4cdd08ed..00a6511354e6 100644
> --- a/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/baikal,bt1-pvt.yaml
> @@ -79,7 +79,7 @@ properties:
> minimum: 0
> maximum: 7130
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> @@ -99,7 +99,7 @@ examples:
>
> interrupts = <GIC_SHARED 31 IRQ_TYPE_LEVEL_HIGH>;
>
> - baikal,pvt-temp-trim-millicelsius = <1000>;
> + baikal,pvt-temp-offset-millicelsius = <1000>;
>
> clocks = <&ccu_sys>, <&ccu_sys>;
> clock-names = "ref", "pclk";
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> index 8589b722028d..4fcbfd93e218 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adis16240.yaml
> @@ -25,12 +25,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> index 591ca32181b0..11d32a288535 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl345.yaml
> @@ -32,6 +32,8 @@ properties:
>
> spi-cpol: true
>
> + spi-max-frequency: true
> +
> interrupts:
> maxItems: 1
>
> @@ -40,7 +42,7 @@ required:
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> index 64f275c8e2d9..38b59b6454ce 100644
> --- a/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> +++ b/Documentation/devicetree/bindings/iio/accel/adi,adxl372.yaml
> @@ -25,12 +25,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> index d0d2880626c2..f1c574c896cb 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7124.yaml
> @@ -52,6 +52,8 @@ properties:
> avdd-supply:
> description: avdd supply can be used as reference for conversion.
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> @@ -108,7 +110,7 @@ patternProperties:
> - reg
> - diff-channels
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> index ed363a796e50..e0cc3b2e8957 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml
> @@ -30,6 +30,8 @@ properties:
>
> spi-cpha: true
>
> + spi-max-frequency: true
> +
> clocks:
> maxItems: 1
> description: phandle to the master clock (mclk)
> @@ -92,7 +94,7 @@ required:
> - spi-cpol
> - spi-cpha
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> index 55e973c6449c..108d202b288f 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7292.yaml
> @@ -30,6 +30,8 @@ properties:
>
> spi-cpha: true
>
> + spi-max-frequency: true
> +
> '#address-cells':
> const: 1
>
> @@ -63,7 +65,7 @@ patternProperties:
> required:
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> index 014b020ed0c2..73775174cf57 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7606.yaml
> @@ -31,6 +31,10 @@ properties:
>
> spi-cpha: true
>
> + spi-cpol: true
> +
> + spi-max-frequency: true
> +
> avcc-supply: true
>
> interrupts:
> @@ -102,7 +106,7 @@ required:
> - interrupts
> - adi,conversion-start-gpios
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml b/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> index 2a17641faed5..e82194974eea 100644
> --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7923.yaml
> @@ -43,11 +43,13 @@ properties:
> '#size-cells':
> const: 0
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml b/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> index 181213b862db..4c7e0d94bff1 100644
> --- a/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/maxim,max1241.yaml
> @@ -39,13 +39,15 @@ properties:
> thus enabling power-down mode.
> maxItems: 1
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - vdd-supply
> - vref-supply
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> index faef288b7148..fb2c48fc7ce4 100644
> --- a/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> +++ b/Documentation/devicetree/bindings/iio/dac/adi,ad5770r.yaml
> @@ -49,6 +49,14 @@ properties:
> asserted during driver probe.
> maxItems: 1
>
> + spi-max-frequency: true
> +
> + '#address-cells':
> + const: 1
> +
> + '#size-cells':
> + const: 0
> +
> channel@0:
> description: Represents an external channel which are
> connected to the DAC. Channel 0 can act both as a current
> @@ -130,7 +138,7 @@ required:
> - channel@4
> - channel@5
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> index 11d445f7010e..6b3a611e1cf1 100644
> --- a/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> +++ b/Documentation/devicetree/bindings/iio/frequency/adf4371.yaml
> @@ -40,13 +40,15 @@ properties:
> output stage will shut down until the ADF4371/ADF4372 achieves lock as
> measured by the digital lock detect circuitry.
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
> - clocks
> - clock-names
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> index 07c8ed4ee0f1..340be256f283 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16460.yaml
> @@ -25,6 +25,8 @@ properties:
>
> spi-cpol: true
>
> + spi-max-frequency: true
> +
> interrupts:
> maxItems: 1
>
> @@ -33,7 +35,7 @@ required:
> - reg
> - interrupts
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> index c29385697bbf..79fba1508e89 100644
> --- a/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/adi,adis16475.yaml
> @@ -116,7 +116,7 @@ allOf:
> dependencies:
> adi,sync-mode: [ clocks ]
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> index 4f215399c8df..6e73cd889b5c 100644
> --- a/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/bosch,bmi160.yaml
> @@ -46,11 +46,13 @@ properties:
> mount-matrix:
> description: an optional 3x3 mounting rotation matrix
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> index 716731c2b794..479e7065d4eb 100644
> --- a/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> +++ b/Documentation/devicetree/bindings/iio/imu/nxp,fxos8700.yaml
> @@ -36,11 +36,13 @@ properties:
> drive-open-drain:
> type: boolean
>
> + spi-max-frequency: true
> +
> required:
> - compatible
> - reg
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml b/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> index ce6aaff15214..039e08af98bb 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/mti,gic.yaml
> @@ -95,7 +95,7 @@ properties:
>
> additionalProperties: false
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> index 7a616577ac63..c00ad3e21c21 100644
> --- a/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> +++ b/Documentation/devicetree/bindings/mfd/cirrus,lochnagar.yaml
> @@ -130,6 +130,14 @@ properties:
> type: object
> $ref: /schemas/pinctrl/cirrus,lochnagar.yaml#
>
> + lochnagar-hwmon:
> + type: object
> + $ref: /schemas/hwmon/cirrus,lochnagar.yaml#
> +
> + lochnagar-sc:
> + type: object
> + $ref: /schemas/sound/cirrus,lochnagar.yaml#
> +
> VDDCORE:
> description:
> Initialisation data for the VDDCORE regulator, which supplies the
> @@ -249,7 +257,7 @@ required:
> - lochnagar-clk
> - lochnagar-pinctrl
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> index da3d9ab758b9..19fcf59fd2fe 100644
> --- a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> +++ b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml
> @@ -30,6 +30,9 @@ properties:
> - const: syscon
> - const: simple-mfd
>
> + reg:
> + maxItems: 1
> +
> "#address-cells":
> const: 1
>
> @@ -54,7 +57,7 @@ required:
> - "#size-cells"
> - ranges
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml b/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> index e75d77beec6a..ade733cd60f7 100644
> --- a/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> +++ b/Documentation/devicetree/bindings/misc/olpc,xo1.75-ec.yaml
> @@ -28,11 +28,13 @@ properties:
> description: GPIO uspecifier of the CMD pin
> maxItems: 1
>
> + spi-cpha: true
> +
> required:
> - compatible
> - cmd-gpios
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> index aef87a33a7c9..aeff2bd774dd 100644
> --- a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> +++ b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> @@ -31,6 +31,8 @@ properties:
> Documentation/devicetree/bindings/nvmem/nvmem.txt and also
> examples below.
>
> + opp-shared: true
> +
> required:
> - compatible
> - nvmem-cells
> @@ -53,7 +55,7 @@ patternProperties:
>
> unevaluatedProperties: false
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> index fb111e2d5b99..53853ec20fe2 100644
> --- a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml
> @@ -47,7 +47,7 @@ properties:
> required:
> - compatible
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> examples:
> - |
> diff --git a/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml b/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> index 7b39e3204fb3..2fc617377e2c 100644
> --- a/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> +++ b/Documentation/devicetree/bindings/timer/snps,dw-apb-timer.yaml
> @@ -45,7 +45,7 @@ properties:
> frequency in HZ, but is defined only for the backwards compatibility
> with the picoxcell platform.
>
> -unevaluatedProperties: false
> +additionalProperties: false
>
> required:
> - compatible
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-10-06 10:00 UTC|newest]
Thread overview: 148+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-05 18:38 [PATCH 0/4] dt-bindings: additional/unevaluatedProperties clean-ups Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` [PATCH 1/4] dt-bindings: Add missing 'unevaluatedProperties' Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 20:01 ` Mark Brown
2020-10-05 20:01 ` Mark Brown
2020-10-05 20:01 ` Mark Brown
2020-10-05 20:01 ` Mark Brown
2020-10-05 20:29 ` Wolfram Sang
2020-10-05 20:29 ` Wolfram Sang
2020-10-05 20:29 ` Wolfram Sang
2020-10-05 20:29 ` Wolfram Sang
2020-10-05 20:31 ` Krzysztof Kozlowski
2020-10-05 20:31 ` Krzysztof Kozlowski
2020-10-05 20:31 ` Krzysztof Kozlowski
2020-10-05 20:31 ` Krzysztof Kozlowski
2020-10-06 5:29 ` Vinod Koul
2020-10-06 5:29 ` Vinod Koul
2020-10-06 5:29 ` Vinod Koul
2020-10-06 5:29 ` Vinod Koul
2020-10-06 7:37 ` Geert Uytterhoeven
2020-10-06 7:37 ` Geert Uytterhoeven
2020-10-06 7:37 ` Geert Uytterhoeven
2020-10-06 7:37 ` Geert Uytterhoeven
2020-10-06 9:12 ` Ulf Hansson
2020-10-06 9:12 ` Ulf Hansson
2020-10-06 9:12 ` Ulf Hansson
2020-10-06 9:12 ` Ulf Hansson
2020-10-06 14:33 ` Guenter Roeck
2020-10-06 14:33 ` Guenter Roeck
2020-10-06 14:33 ` Guenter Roeck
2020-10-06 14:33 ` Guenter Roeck
2020-10-06 16:57 ` Mathieu Poirier
2020-10-06 16:57 ` Mathieu Poirier
2020-10-06 16:57 ` Mathieu Poirier
2020-10-06 16:57 ` Mathieu Poirier
2020-10-07 1:03 ` Dmitry Torokhov
2020-10-07 1:03 ` Dmitry Torokhov
2020-10-07 1:03 ` Dmitry Torokhov
2020-10-07 1:03 ` Dmitry Torokhov
2020-10-07 22:48 ` Alexandre Belloni
2020-10-07 22:48 ` Alexandre Belloni
2020-10-07 22:48 ` Alexandre Belloni
2020-10-07 22:48 ` Alexandre Belloni
2020-10-05 18:38 ` [PATCH 2/4] dt-bindings: Use 'additionalProperties' instead of 'unevaluatedProperties' Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 20:02 ` Mark Brown
2020-10-05 20:02 ` Mark Brown
2020-10-05 20:02 ` Mark Brown
2020-10-05 20:02 ` Mark Brown
2020-10-05 20:33 ` Krzysztof Kozlowski
2020-10-05 20:33 ` Krzysztof Kozlowski
2020-10-05 20:33 ` Krzysztof Kozlowski
2020-10-05 20:33 ` Krzysztof Kozlowski
2020-10-06 6:59 ` Lee Jones
2020-10-06 6:59 ` Lee Jones
2020-10-06 6:59 ` Lee Jones
2020-10-06 6:59 ` Lee Jones
2020-10-06 9:58 ` Jonathan Cameron [this message]
2020-10-06 9:58 ` Jonathan Cameron
2020-10-06 9:58 ` Jonathan Cameron
2020-10-06 9:58 ` Jonathan Cameron
2020-10-06 14:32 ` Guenter Roeck
2020-10-06 14:32 ` Guenter Roeck
2020-10-06 14:32 ` Guenter Roeck
2020-10-06 14:32 ` Guenter Roeck
2020-10-05 18:38 ` [PATCH 3/4] dt-bindings: Explicitly allow additional properties in board/SoC schemas Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 20:34 ` Krzysztof Kozlowski
2020-10-05 20:34 ` Krzysztof Kozlowski
2020-10-05 20:34 ` Krzysztof Kozlowski
2020-10-05 20:34 ` Krzysztof Kozlowski
2020-10-06 5:42 ` Viresh Kumar
2020-10-06 5:42 ` Viresh Kumar
2020-10-06 5:42 ` Viresh Kumar
2020-10-06 5:42 ` Viresh Kumar
2020-10-06 7:42 ` Geert Uytterhoeven
2020-10-06 7:42 ` Geert Uytterhoeven
2020-10-06 7:42 ` Geert Uytterhoeven
2020-10-06 7:42 ` Geert Uytterhoeven
2020-10-05 18:38 ` [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 18:38 ` Rob Herring
2020-10-05 20:04 ` Mark Brown
2020-10-05 20:04 ` Mark Brown
2020-10-05 20:04 ` Mark Brown
2020-10-05 20:04 ` Mark Brown
2020-10-05 20:35 ` Krzysztof Kozlowski
2020-10-05 20:35 ` Krzysztof Kozlowski
2020-10-05 20:35 ` Krzysztof Kozlowski
2020-10-05 20:35 ` Krzysztof Kozlowski
2020-10-05 20:55 ` Sebastian Reichel
2020-10-05 20:55 ` Sebastian Reichel
2020-10-05 20:55 ` Sebastian Reichel
2020-10-05 20:55 ` Sebastian Reichel
2020-10-06 1:23 ` Chanwoo Choi
2020-10-06 1:23 ` Chanwoo Choi
2020-10-06 1:23 ` Chanwoo Choi
2020-10-06 1:23 ` Chanwoo Choi
2020-10-06 5:30 ` Vinod Koul
2020-10-06 5:30 ` Vinod Koul
2020-10-06 5:30 ` Vinod Koul
2020-10-06 5:30 ` Vinod Koul
2020-10-06 6:58 ` Lee Jones
2020-10-06 6:58 ` Lee Jones
2020-10-06 6:58 ` Lee Jones
2020-10-06 6:58 ` Lee Jones
2020-10-06 7:43 ` Geert Uytterhoeven
2020-10-06 7:43 ` Geert Uytterhoeven
2020-10-06 7:43 ` Geert Uytterhoeven
2020-10-06 7:43 ` Geert Uytterhoeven
2020-10-06 8:22 ` Srinivas Kandagatla
2020-10-06 8:22 ` Srinivas Kandagatla
2020-10-06 8:22 ` Srinivas Kandagatla
2020-10-06 8:22 ` Srinivas Kandagatla
2020-10-06 9:12 ` Ulf Hansson
2020-10-06 9:12 ` Ulf Hansson
2020-10-06 9:12 ` Ulf Hansson
2020-10-06 9:12 ` Ulf Hansson
2020-10-06 10:00 ` Jonathan Cameron
2020-10-06 10:00 ` Jonathan Cameron
2020-10-06 10:00 ` Jonathan Cameron
2020-10-06 10:00 ` Jonathan Cameron
2020-10-06 14:30 ` Guenter Roeck
2020-10-06 14:30 ` Guenter Roeck
2020-10-06 14:30 ` Guenter Roeck
2020-10-06 14:30 ` Guenter Roeck
2020-10-07 1:04 ` Dmitry Torokhov
2020-10-07 1:04 ` Dmitry Torokhov
2020-10-07 1:04 ` Dmitry Torokhov
2020-10-07 1:04 ` Dmitry Torokhov
2020-10-07 22:40 ` Alexandre Belloni
2020-10-07 22:40 ` Alexandre Belloni
2020-10-07 22:40 ` Alexandre Belloni
2020-10-07 22:40 ` Alexandre Belloni
2020-10-06 8:25 ` [PATCH 0/4] dt-bindings: additional/unevaluatedProperties clean-ups Srinivas Kandagatla
2020-10-06 8:25 ` Srinivas Kandagatla
2020-10-06 8:25 ` Srinivas Kandagatla
2020-10-06 8:25 ` Srinivas Kandagatla
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201006095822.00005f32@Huawei.com \
--to=jonathan.cameron@huawei.com \
--cc=a.zummo@towertech.it \
--cc=andrew@lunn.ch \
--cc=aou@eecs.berkeley.edu \
--cc=axboe@kernel.dk \
--cc=bhelgaas@google.com \
--cc=broonie@kernel.org \
--cc=cw00.choi@samsung.com \
--cc=daniel.lezcano@linaro.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=dmaengine@vger.kernel.org \
--cc=dmitry.torokhov@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=gregkh@linuxfoundation.org \
--cc=hkallweit1@gmail.com \
--cc=jason@lakedaemon.net \
--cc=jic23@kernel.org \
--cc=krzk@kernel.org \
--cc=kuba@kernel.org \
--cc=lars@metafoo.de \
--cc=laurent.pinchart@ideasonboard.com \
--cc=lee.jones@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-can@vger.kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=linux-rtc@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=mathieu.poirier@linaro.org \
--cc=maz@kernel.org \
--cc=mchehab@kernel.org \
--cc=miquel.raynal@bootlin.com \
--cc=mkl@pengutronix.de \
--cc=myungjoo.ham@samsung.com \
--cc=palmer@dabbelt.com \
--cc=paul.walmsley@sifive.com \
--cc=pavel@ucw.cz \
--cc=pmeerw@pmeerw.net \
--cc=richard@nod.at \
--cc=robh@kernel.org \
--cc=rui.zhang@intel.com \
--cc=sam@ravnborg.org \
--cc=sboyd@kernel.org \
--cc=sre@kernel.org \
--cc=srinivas.kandagatla@linaro.org \
--cc=tglx@linutronix.de \
--cc=thierry.reding@gmail.com \
--cc=tsbogend@alpha.franken.de \
--cc=ulf.hansson@linaro.org \
--cc=vigneshr@ti.com \
--cc=vireshk@kernel.org \
--cc=vivien.didelot@gmail.com \
--cc=vkoul@kernel.org \
--cc=wg@grandegger.com \
--cc=wim@linux-watchdog.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.