2020-12-21 23:49:45

by Rob Herring (Arm)

[permalink] [raw]
Subject: [PATCH] dt-bindings: Drop unnecessary *-supply schemas properties

*-supply properties are always a single phandle, so binding schemas
don't need a type $ref nor 'maxItems'.

A meta-schema check for this is pending once these existing cases are
fixed.

Cc: Jonathan Cameron <[email protected]>
Cc: Dmitry Torokhov <[email protected]>
Cc: Laurent Pinchart <[email protected]>
Cc: Mauro Carvalho Chehab <[email protected]>
Cc: Sakari Ailus <[email protected]>
Cc: Lee Jones <[email protected]>
Cc: Mark Brown <[email protected]>
Cc: Maxime Ripard <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Signed-off-by: Rob Herring <[email protected]>
---
Documentation/devicetree/bindings/display/bridge/anx6345.yaml | 2 --
.../devicetree/bindings/display/bridge/ite,it6505.yaml | 2 --
.../devicetree/bindings/display/bridge/lvds-codec.yaml | 3 +--
Documentation/devicetree/bindings/display/bridge/ps8640.yaml | 2 --
.../devicetree/bindings/display/bridge/simple-bridge.yaml | 1 -
.../bindings/display/bridge/thine,thc63lvd1024.yaml | 1 -
.../devicetree/bindings/display/bridge/toshiba,tc358775.yaml | 2 --
Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml | 4 +---
.../devicetree/bindings/iio/humidity/ti,hdc2010.yaml | 3 +--
.../devicetree/bindings/input/fsl,mpr121-touchkey.yaml | 3 +--
.../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml | 3 +--
.../devicetree/bindings/media/i2c/maxim,max9286.yaml | 1 -
Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml | 3 ---
Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml | 3 ---
Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml | 3 ---
Documentation/devicetree/bindings/mfd/st,stmfx.yaml | 3 +--
.../devicetree/bindings/regulator/anatop-regulator.yaml | 1 -
17 files changed, 6 insertions(+), 34 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/bridge/anx6345.yaml b/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
index 8c0e4f285fbc..fccd63521a8c 100644
--- a/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
@@ -26,11 +26,9 @@ properties:
description: GPIO connected to active low reset

dvdd12-supply:
- maxItems: 1
description: Regulator for 1.2V digital core power.

dvdd25-supply:
- maxItems: 1
description: Regulator for 2.5V digital core power.

ports:
diff --git a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
index efbb3d0117dc..02cfc0a3b550 100644
--- a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
@@ -35,11 +35,9 @@ properties:
maxItems: 1

ovdd-supply:
- maxItems: 1
description: I/O voltage

pwr18-supply:
- maxItems: 1
description: core voltage

interrupts:
diff --git a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
index e5e3c72630cf..66a14d60ce1d 100644
--- a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
@@ -79,8 +79,7 @@ properties:
The GPIO used to control the power down line of this device.
maxItems: 1

- power-supply:
- maxItems: 1
+ power-supply: true

required:
- compatible
diff --git a/Documentation/devicetree/bindings/display/bridge/ps8640.yaml b/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
index 7e27cfcf770d..763c7909473e 100644
--- a/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
@@ -35,11 +35,9 @@ properties:
description: GPIO connected to active low reset.

vdd12-supply:
- maxItems: 1
description: Regulator for 1.2V digital core power.

vdd33-supply:
- maxItems: 1
description: Regulator for 3.3V digital core power.

ports:
diff --git a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
index 3ddb35fcf0a2..64e8a1c24b40 100644
--- a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
@@ -60,7 +60,6 @@ properties:
description: GPIO controlling bridge enable

vdd-supply:
- maxItems: 1
description: Power supply for the bridge

required:
diff --git a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
index 469ac4a34273..3d5ce08a5792 100644
--- a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
@@ -74,7 +74,6 @@ properties:
description: Power down GPIO signal, pin name "/PDWN", active low.

vcc-supply:
- maxItems: 1
description:
Power supply for the TTL output, TTL CLOCKOUT signal, LVDS input, PLL and
digital circuitry.
diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
index fd3113aa9ccd..b5959cc78b8d 100644
--- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
@@ -28,11 +28,9 @@ properties:
description: i2c address of the bridge, 0x0f

vdd-supply:
- maxItems: 1
description: 1.2V LVDS Power Supply

vddio-supply:
- maxItems: 1
description: 1.8V IO Power Supply

stby-gpios:
diff --git a/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml b/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
index 6a991e9f78e2..f04084fae5e8 100644
--- a/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
+++ b/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
@@ -16,9 +16,7 @@ properties:
enum:
- lltc,ltc2496

- vref-supply:
- description: phandle to an external regulator providing the reference voltage
- $ref: /schemas/types.yaml#/definitions/phandle
+ vref-supply: true

reg:
description: spi chipselect number according to the usual spi bindings
diff --git a/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml b/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
index 7037f82ec753..88384b69f917 100644
--- a/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
+++ b/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
@@ -22,8 +22,7 @@ properties:
- ti,hdc2010
- ti,hdc2080

- vdd-supply:
- maxItems: 1
+ vdd-supply: true

reg:
maxItems: 1
diff --git a/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml b/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
index 378a85c09d34..878464f128dc 100644
--- a/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
+++ b/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
@@ -31,8 +31,7 @@ properties:
interrupts:
maxItems: 1

- vdd-supply:
- maxItems: 1
+ vdd-supply: true

linux,keycodes:
minItems: 1
diff --git a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
index 4ce109476a0e..bfc3a8b5e118 100644
--- a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
+++ b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
@@ -55,8 +55,7 @@ properties:

wakeup-source: true

- vcc-supply:
- maxItems: 1
+ vcc-supply: true

gain:
description: Allows setting the sensitivity in the range from 0 to 31.
diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
index 9ea827092fdd..68ee8c7d9e79 100644
--- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
@@ -40,7 +40,6 @@ properties:

poc-supply:
description: Regulator providing Power over Coax to the cameras
- maxItems: 1

enable-gpios:
description: GPIO connected to the \#PWDN pin with inverted polarity
diff --git a/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml b/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
index 0df0334d2d0d..bb3528315f20 100644
--- a/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
@@ -39,15 +39,12 @@ properties:

vana-supply:
description: Analogue voltage supply (VANA), sensor dependent.
- maxItems: 1

vcore-supply:
description: Core voltage supply (VCore), sensor dependent.
- maxItems: 1

vio-supply:
description: I/O voltage supply (VIO), sensor dependent.
- maxItems: 1

clocks:
description: External clock to the sensor.
diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
index 1a3590dd0e98..eb12526a462f 100644
--- a/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
@@ -37,15 +37,12 @@ properties:

vdddo-supply:
description: Chip digital IO regulator (1.8V).
- maxItems: 1

vdda-supply:
description: Chip analog regulator (2.7V).
- maxItems: 1

vddd-supply:
description: Chip digital core regulator (1.12V).
- maxItems: 1

flash-leds:
description: See ../video-interfaces.txt
diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
index f697e1a20beb..a66acb20d59b 100644
--- a/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
+++ b/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
@@ -33,15 +33,12 @@ properties:

vana-supply:
description: Sensor 2.8 V analog supply.
- maxItems: 1

vdig-supply:
description: Sensor 1.8 V digital core supply.
- maxItems: 1

vddl-supply:
description: Sensor digital IO 1.2 V supply.
- maxItems: 1

port:
type: object
diff --git a/Documentation/devicetree/bindings/mfd/st,stmfx.yaml b/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
index 888ab4b5df45..19e9afb385ac 100644
--- a/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
+++ b/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
@@ -26,8 +26,7 @@ properties:

drive-open-drain: true

- vdd-supply:
- maxItems: 1
+ vdd-supply: true

pinctrl:
type: object
diff --git a/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml b/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
index e7b3abe30363..0a66338c7e5a 100644
--- a/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
+++ b/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
@@ -59,7 +59,6 @@ properties:
description: u32 value representing regulator enable bit offset.

vin-supply:
- $ref: '/schemas/types.yaml#/definitions/phandle'
description: input supply phandle.

required:
--
2.27.0


2020-12-22 06:36:56

by Sam Ravnborg

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Drop unnecessary *-supply schemas properties

Hi Rob

On Mon, Dec 21, 2020 at 04:46:59PM -0700, Rob Herring wrote:
> *-supply properties are always a single phandle, so binding schemas
> don't need a type $ref nor 'maxItems'.
>
> A meta-schema check for this is pending once these existing cases are
> fixed.
>
> Cc: Jonathan Cameron <[email protected]>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Laurent Pinchart <[email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>
> Cc: Sakari Ailus <[email protected]>
> Cc: Lee Jones <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>

For the Documentation/devicetree/bindings/display/* parts:
Acked-by: Sam Ravnborg <[email protected]>

Sam

2020-12-22 07:05:07

by Laurent Pinchart

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Drop unnecessary *-supply schemas properties

Hi Rob,

Thank you for the patch.

On Mon, Dec 21, 2020 at 04:46:59PM -0700, Rob Herring wrote:
> *-supply properties are always a single phandle, so binding schemas
> don't need a type $ref nor 'maxItems'.
>
> A meta-schema check for this is pending once these existing cases are
> fixed.
>
> Cc: Jonathan Cameron <[email protected]>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Laurent Pinchart <[email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>
> Cc: Sakari Ailus <[email protected]>
> Cc: Lee Jones <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>
> ---
> Documentation/devicetree/bindings/display/bridge/anx6345.yaml | 2 --
> .../devicetree/bindings/display/bridge/ite,it6505.yaml | 2 --
> .../devicetree/bindings/display/bridge/lvds-codec.yaml | 3 +--
> Documentation/devicetree/bindings/display/bridge/ps8640.yaml | 2 --
> .../devicetree/bindings/display/bridge/simple-bridge.yaml | 1 -
> .../bindings/display/bridge/thine,thc63lvd1024.yaml | 1 -
> .../devicetree/bindings/display/bridge/toshiba,tc358775.yaml | 2 --
> Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml | 4 +---
> .../devicetree/bindings/iio/humidity/ti,hdc2010.yaml | 3 +--
> .../devicetree/bindings/input/fsl,mpr121-touchkey.yaml | 3 +--
> .../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml | 3 +--
> .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 1 -
> Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml | 3 ---
> Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml | 3 ---
> Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml | 3 ---
> Documentation/devicetree/bindings/mfd/st,stmfx.yaml | 3 +--
> .../devicetree/bindings/regulator/anatop-regulator.yaml | 1 -
> 17 files changed, 6 insertions(+), 34 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/anx6345.yaml b/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
> index 8c0e4f285fbc..fccd63521a8c 100644
> --- a/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
> @@ -26,11 +26,9 @@ properties:
> description: GPIO connected to active low reset
>
> dvdd12-supply:
> - maxItems: 1
> description: Regulator for 1.2V digital core power.
>
> dvdd25-supply:
> - maxItems: 1
> description: Regulator for 2.5V digital core power.
>
> ports:
> diff --git a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> index efbb3d0117dc..02cfc0a3b550 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> @@ -35,11 +35,9 @@ properties:
> maxItems: 1
>
> ovdd-supply:
> - maxItems: 1
> description: I/O voltage
>
> pwr18-supply:
> - maxItems: 1
> description: core voltage
>
> interrupts:
> diff --git a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
> index e5e3c72630cf..66a14d60ce1d 100644
> --- a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
> @@ -79,8 +79,7 @@ properties:
> The GPIO used to control the power down line of this device.
> maxItems: 1
>
> - power-supply:
> - maxItems: 1
> + power-supply: true
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/display/bridge/ps8640.yaml b/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
> index 7e27cfcf770d..763c7909473e 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
> @@ -35,11 +35,9 @@ properties:
> description: GPIO connected to active low reset.
>
> vdd12-supply:
> - maxItems: 1
> description: Regulator for 1.2V digital core power.
>
> vdd33-supply:
> - maxItems: 1
> description: Regulator for 3.3V digital core power.
>
> ports:
> diff --git a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> index 3ddb35fcf0a2..64e8a1c24b40 100644
> --- a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> @@ -60,7 +60,6 @@ properties:
> description: GPIO controlling bridge enable
>
> vdd-supply:
> - maxItems: 1
> description: Power supply for the bridge
>
> required:
> diff --git a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
> index 469ac4a34273..3d5ce08a5792 100644
> --- a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
> @@ -74,7 +74,6 @@ properties:
> description: Power down GPIO signal, pin name "/PDWN", active low.
>
> vcc-supply:
> - maxItems: 1
> description:
> Power supply for the TTL output, TTL CLOCKOUT signal, LVDS input, PLL and
> digital circuitry.
> diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
> index fd3113aa9ccd..b5959cc78b8d 100644
> --- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
> @@ -28,11 +28,9 @@ properties:
> description: i2c address of the bridge, 0x0f
>
> vdd-supply:
> - maxItems: 1
> description: 1.2V LVDS Power Supply
>
> vddio-supply:
> - maxItems: 1
> description: 1.8V IO Power Supply
>
> stby-gpios:
> diff --git a/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml b/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
> index 6a991e9f78e2..f04084fae5e8 100644
> --- a/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
> @@ -16,9 +16,7 @@ properties:
> enum:
> - lltc,ltc2496
>
> - vref-supply:
> - description: phandle to an external regulator providing the reference voltage
> - $ref: /schemas/types.yaml#/definitions/phandle
> + vref-supply: true

I'd keep part of the description as it's not just the device's main
power supply (at least if we can trust the current description).

vref-supply:
description: Power supply for the reference voltage

Reviewed-by: Laurent Pinchart <[email protected]>

> reg:
> description: spi chipselect number according to the usual spi bindings
> diff --git a/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml b/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
> index 7037f82ec753..88384b69f917 100644
> --- a/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
> +++ b/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
> @@ -22,8 +22,7 @@ properties:
> - ti,hdc2010
> - ti,hdc2080
>
> - vdd-supply:
> - maxItems: 1
> + vdd-supply: true
>
> reg:
> maxItems: 1
> diff --git a/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml b/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
> index 378a85c09d34..878464f128dc 100644
> --- a/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
> +++ b/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
> @@ -31,8 +31,7 @@ properties:
> interrupts:
> maxItems: 1
>
> - vdd-supply:
> - maxItems: 1
> + vdd-supply: true
>
> linux,keycodes:
> minItems: 1
> diff --git a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> index 4ce109476a0e..bfc3a8b5e118 100644
> --- a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> +++ b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> @@ -55,8 +55,7 @@ properties:
>
> wakeup-source: true
>
> - vcc-supply:
> - maxItems: 1
> + vcc-supply: true
>
> gain:
> description: Allows setting the sensitivity in the range from 0 to 31.
> diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> index 9ea827092fdd..68ee8c7d9e79 100644
> --- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> @@ -40,7 +40,6 @@ properties:
>
> poc-supply:
> description: Regulator providing Power over Coax to the cameras
> - maxItems: 1
>
> enable-gpios:
> description: GPIO connected to the \#PWDN pin with inverted polarity
> diff --git a/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml b/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> index 0df0334d2d0d..bb3528315f20 100644
> --- a/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> @@ -39,15 +39,12 @@ properties:
>
> vana-supply:
> description: Analogue voltage supply (VANA), sensor dependent.
> - maxItems: 1
>
> vcore-supply:
> description: Core voltage supply (VCore), sensor dependent.
> - maxItems: 1
>
> vio-supply:
> description: I/O voltage supply (VIO), sensor dependent.
> - maxItems: 1
>
> clocks:
> description: External clock to the sensor.
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> index 1a3590dd0e98..eb12526a462f 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> @@ -37,15 +37,12 @@ properties:
>
> vdddo-supply:
> description: Chip digital IO regulator (1.8V).
> - maxItems: 1
>
> vdda-supply:
> description: Chip analog regulator (2.7V).
> - maxItems: 1
>
> vddd-supply:
> description: Chip digital core regulator (1.12V).
> - maxItems: 1
>
> flash-leds:
> description: See ../video-interfaces.txt
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
> index f697e1a20beb..a66acb20d59b 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
> @@ -33,15 +33,12 @@ properties:
>
> vana-supply:
> description: Sensor 2.8 V analog supply.
> - maxItems: 1
>
> vdig-supply:
> description: Sensor 1.8 V digital core supply.
> - maxItems: 1
>
> vddl-supply:
> description: Sensor digital IO 1.2 V supply.
> - maxItems: 1
>
> port:
> type: object
> diff --git a/Documentation/devicetree/bindings/mfd/st,stmfx.yaml b/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
> index 888ab4b5df45..19e9afb385ac 100644
> --- a/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
> +++ b/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
> @@ -26,8 +26,7 @@ properties:
>
> drive-open-drain: true
>
> - vdd-supply:
> - maxItems: 1
> + vdd-supply: true
>
> pinctrl:
> type: object
> diff --git a/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml b/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
> index e7b3abe30363..0a66338c7e5a 100644
> --- a/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
> @@ -59,7 +59,6 @@ properties:
> description: u32 value representing regulator enable bit offset.
>
> vin-supply:
> - $ref: '/schemas/types.yaml#/definitions/phandle'
> description: input supply phandle.
>
> required:

--
Regards,

Laurent Pinchart

2020-12-22 09:27:29

by Sakari Ailus

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Drop unnecessary *-supply schemas properties

Hi Rob,

On Mon, Dec 21, 2020 at 04:46:59PM -0700, Rob Herring wrote:
> *-supply properties are always a single phandle, so binding schemas
> don't need a type $ref nor 'maxItems'.
>
> A meta-schema check for this is pending once these existing cases are
> fixed.
>
> Cc: Jonathan Cameron <[email protected]>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Laurent Pinchart <[email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>
> Cc: Sakari Ailus <[email protected]>
> Cc: Lee Jones <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>

On media bits:

Acked-by: Sakari Ailus <[email protected]>

--
Sakari Ailus

2020-12-22 10:02:05

by Lee Jones

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Drop unnecessary *-supply schemas properties

On Mon, 21 Dec 2020, Rob Herring wrote:

> *-supply properties are always a single phandle, so binding schemas
> don't need a type $ref nor 'maxItems'.
>
> A meta-schema check for this is pending once these existing cases are
> fixed.
>
> Cc: Jonathan Cameron <[email protected]>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Laurent Pinchart <[email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>
> Cc: Sakari Ailus <[email protected]>
> Cc: Lee Jones <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>
> ---
> Documentation/devicetree/bindings/display/bridge/anx6345.yaml | 2 --
> .../devicetree/bindings/display/bridge/ite,it6505.yaml | 2 --
> .../devicetree/bindings/display/bridge/lvds-codec.yaml | 3 +--
> Documentation/devicetree/bindings/display/bridge/ps8640.yaml | 2 --
> .../devicetree/bindings/display/bridge/simple-bridge.yaml | 1 -
> .../bindings/display/bridge/thine,thc63lvd1024.yaml | 1 -
> .../devicetree/bindings/display/bridge/toshiba,tc358775.yaml | 2 --
> Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml | 4 +---
> .../devicetree/bindings/iio/humidity/ti,hdc2010.yaml | 3 +--
> .../devicetree/bindings/input/fsl,mpr121-touchkey.yaml | 3 +--
> .../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml | 3 +--
> .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 1 -
> Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml | 3 ---
> Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml | 3 ---
> Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml | 3 ---
> Documentation/devicetree/bindings/mfd/st,stmfx.yaml | 3 +--

Acked-by: Lee Jones <[email protected]>

> .../devicetree/bindings/regulator/anatop-regulator.yaml | 1 -
> 17 files changed, 6 insertions(+), 34 deletions(-)

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog

2020-12-22 17:17:09

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Drop unnecessary *-supply schemas properties

On Mon, Dec 21, 2020 at 04:46:59PM -0700, Rob Herring wrote:
> *-supply properties are always a single phandle, so binding schemas
> don't need a type $ref nor 'maxItems'.

Acked-by: Mark Brown <[email protected]>


Attachments:
(No filename) (220.00 B)
signature.asc (499.00 B)
Download all attachments

2020-12-29 16:11:59

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Drop unnecessary *-supply schemas properties

On Mon, 21 Dec 2020 16:46:59 -0700
Rob Herring <[email protected]> wrote:

> *-supply properties are always a single phandle, so binding schemas
> don't need a type $ref nor 'maxItems'.
>
> A meta-schema check for this is pending once these existing cases are
> fixed.
>
> Cc: Jonathan Cameron <[email protected]>
> Cc: Dmitry Torokhov <[email protected]>
> Cc: Laurent Pinchart <[email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>
> Cc: Sakari Ailus <[email protected]>
> Cc: Lee Jones <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: Rob Herring <[email protected]>

Acked-by: Jonathan Cameron <[email protected]> #for-iio

> ---
> Documentation/devicetree/bindings/display/bridge/anx6345.yaml | 2 --
> .../devicetree/bindings/display/bridge/ite,it6505.yaml | 2 --
> .../devicetree/bindings/display/bridge/lvds-codec.yaml | 3 +--
> Documentation/devicetree/bindings/display/bridge/ps8640.yaml | 2 --
> .../devicetree/bindings/display/bridge/simple-bridge.yaml | 1 -
> .../bindings/display/bridge/thine,thc63lvd1024.yaml | 1 -
> .../devicetree/bindings/display/bridge/toshiba,tc358775.yaml | 2 --
> Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml | 4 +---
> .../devicetree/bindings/iio/humidity/ti,hdc2010.yaml | 3 +--
> .../devicetree/bindings/input/fsl,mpr121-touchkey.yaml | 3 +--
> .../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml | 3 +--
> .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 1 -
> Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml | 3 ---
> Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml | 3 ---
> Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml | 3 ---
> Documentation/devicetree/bindings/mfd/st,stmfx.yaml | 3 +--
> .../devicetree/bindings/regulator/anatop-regulator.yaml | 1 -
> 17 files changed, 6 insertions(+), 34 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/anx6345.yaml b/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
> index 8c0e4f285fbc..fccd63521a8c 100644
> --- a/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/anx6345.yaml
> @@ -26,11 +26,9 @@ properties:
> description: GPIO connected to active low reset
>
> dvdd12-supply:
> - maxItems: 1
> description: Regulator for 1.2V digital core power.
>
> dvdd25-supply:
> - maxItems: 1
> description: Regulator for 2.5V digital core power.
>
> ports:
> diff --git a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> index efbb3d0117dc..02cfc0a3b550 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> @@ -35,11 +35,9 @@ properties:
> maxItems: 1
>
> ovdd-supply:
> - maxItems: 1
> description: I/O voltage
>
> pwr18-supply:
> - maxItems: 1
> description: core voltage
>
> interrupts:
> diff --git a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
> index e5e3c72630cf..66a14d60ce1d 100644
> --- a/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/lvds-codec.yaml
> @@ -79,8 +79,7 @@ properties:
> The GPIO used to control the power down line of this device.
> maxItems: 1
>
> - power-supply:
> - maxItems: 1
> + power-supply: true
>
> required:
> - compatible
> diff --git a/Documentation/devicetree/bindings/display/bridge/ps8640.yaml b/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
> index 7e27cfcf770d..763c7909473e 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/ps8640.yaml
> @@ -35,11 +35,9 @@ properties:
> description: GPIO connected to active low reset.
>
> vdd12-supply:
> - maxItems: 1
> description: Regulator for 1.2V digital core power.
>
> vdd33-supply:
> - maxItems: 1
> description: Regulator for 3.3V digital core power.
>
> ports:
> diff --git a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> index 3ddb35fcf0a2..64e8a1c24b40 100644
> --- a/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/simple-bridge.yaml
> @@ -60,7 +60,6 @@ properties:
> description: GPIO controlling bridge enable
>
> vdd-supply:
> - maxItems: 1
> description: Power supply for the bridge
>
> required:
> diff --git a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
> index 469ac4a34273..3d5ce08a5792 100644
> --- a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvd1024.yaml
> @@ -74,7 +74,6 @@ properties:
> description: Power down GPIO signal, pin name "/PDWN", active low.
>
> vcc-supply:
> - maxItems: 1
> description:
> Power supply for the TTL output, TTL CLOCKOUT signal, LVDS input, PLL and
> digital circuitry.
> diff --git a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
> index fd3113aa9ccd..b5959cc78b8d 100644
> --- a/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/toshiba,tc358775.yaml
> @@ -28,11 +28,9 @@ properties:
> description: i2c address of the bridge, 0x0f
>
> vdd-supply:
> - maxItems: 1
> description: 1.2V LVDS Power Supply
>
> vddio-supply:
> - maxItems: 1
> description: 1.8V IO Power Supply
>
> stby-gpios:
> diff --git a/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml b/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
> index 6a991e9f78e2..f04084fae5e8 100644
> --- a/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
> +++ b/Documentation/devicetree/bindings/iio/adc/lltc,ltc2496.yaml
> @@ -16,9 +16,7 @@ properties:
> enum:
> - lltc,ltc2496
>
> - vref-supply:
> - description: phandle to an external regulator providing the reference voltage
> - $ref: /schemas/types.yaml#/definitions/phandle
> + vref-supply: true
>
> reg:
> description: spi chipselect number according to the usual spi bindings
> diff --git a/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml b/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
> index 7037f82ec753..88384b69f917 100644
> --- a/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
> +++ b/Documentation/devicetree/bindings/iio/humidity/ti,hdc2010.yaml
> @@ -22,8 +22,7 @@ properties:
> - ti,hdc2010
> - ti,hdc2080
>
> - vdd-supply:
> - maxItems: 1
> + vdd-supply: true
>
> reg:
> maxItems: 1
> diff --git a/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml b/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
> index 378a85c09d34..878464f128dc 100644
> --- a/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
> +++ b/Documentation/devicetree/bindings/input/fsl,mpr121-touchkey.yaml
> @@ -31,8 +31,7 @@ properties:
> interrupts:
> maxItems: 1
>
> - vdd-supply:
> - maxItems: 1
> + vdd-supply: true
>
> linux,keycodes:
> minItems: 1
> diff --git a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> index 4ce109476a0e..bfc3a8b5e118 100644
> --- a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> +++ b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
> @@ -55,8 +55,7 @@ properties:
>
> wakeup-source: true
>
> - vcc-supply:
> - maxItems: 1
> + vcc-supply: true
>
> gain:
> description: Allows setting the sensitivity in the range from 0 to 31.
> diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> index 9ea827092fdd..68ee8c7d9e79 100644
> --- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml
> @@ -40,7 +40,6 @@ properties:
>
> poc-supply:
> description: Regulator providing Power over Coax to the cameras
> - maxItems: 1
>
> enable-gpios:
> description: GPIO connected to the \#PWDN pin with inverted polarity
> diff --git a/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml b/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> index 0df0334d2d0d..bb3528315f20 100644
> --- a/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
> @@ -39,15 +39,12 @@ properties:
>
> vana-supply:
> description: Analogue voltage supply (VANA), sensor dependent.
> - maxItems: 1
>
> vcore-supply:
> description: Core voltage supply (VCore), sensor dependent.
> - maxItems: 1
>
> vio-supply:
> description: I/O voltage supply (VIO), sensor dependent.
> - maxItems: 1
>
> clocks:
> description: External clock to the sensor.
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> index 1a3590dd0e98..eb12526a462f 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml
> @@ -37,15 +37,12 @@ properties:
>
> vdddo-supply:
> description: Chip digital IO regulator (1.8V).
> - maxItems: 1
>
> vdda-supply:
> description: Chip analog regulator (2.7V).
> - maxItems: 1
>
> vddd-supply:
> description: Chip digital core regulator (1.12V).
> - maxItems: 1
>
> flash-leds:
> description: See ../video-interfaces.txt
> diff --git a/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml b/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
> index f697e1a20beb..a66acb20d59b 100644
> --- a/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml
> @@ -33,15 +33,12 @@ properties:
>
> vana-supply:
> description: Sensor 2.8 V analog supply.
> - maxItems: 1
>
> vdig-supply:
> description: Sensor 1.8 V digital core supply.
> - maxItems: 1
>
> vddl-supply:
> description: Sensor digital IO 1.2 V supply.
> - maxItems: 1
>
> port:
> type: object
> diff --git a/Documentation/devicetree/bindings/mfd/st,stmfx.yaml b/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
> index 888ab4b5df45..19e9afb385ac 100644
> --- a/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
> +++ b/Documentation/devicetree/bindings/mfd/st,stmfx.yaml
> @@ -26,8 +26,7 @@ properties:
>
> drive-open-drain: true
>
> - vdd-supply:
> - maxItems: 1
> + vdd-supply: true
>
> pinctrl:
> type: object
> diff --git a/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml b/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
> index e7b3abe30363..0a66338c7e5a 100644
> --- a/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/anatop-regulator.yaml
> @@ -59,7 +59,6 @@ properties:
> description: u32 value representing regulator enable bit offset.
>
> vin-supply:
> - $ref: '/schemas/types.yaml#/definitions/phandle'
> description: input supply phandle.
>
> required: