Re: [PATCH] dt-bindings: Drop unnecessary *-supply schemas properties
On Mon, 21 Dec 2020 16:46:59 -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 > Cc: Dmitry Torokhov > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: Sakari Ailus > Cc: Lee Jones > Cc: Mark Brown > Cc: Maxime Ripard > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-in...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Signed-off-by: Rob Herring Acked-by: Jonathan Cameron #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 >
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 signature.asc Description: PGP signature
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 > Cc: Dmitry Torokhov > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: Sakari Ailus > Cc: Lee Jones > Cc: Mark Brown > Cc: Maxime Ripard > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-in...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Signed-off-by: Rob Herring > --- > 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 > .../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
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 > Cc: Dmitry Torokhov > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: Sakari Ailus > Cc: Lee Jones > Cc: Mark Brown > Cc: Maxime Ripard > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-in...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Signed-off-by: Rob Herring On media bits: Acked-by: Sakari Ailus -- Sakari Ailus
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 > Cc: Dmitry Torokhov > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: Sakari Ailus > Cc: Lee Jones > Cc: Mark Brown > Cc: Maxime Ripard > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-in...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Signed-off-by: Rob Herring > --- > 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 >
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 > Cc: Dmitry Torokhov > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: Sakari Ailus > Cc: Lee Jones > Cc: Mark Brown > Cc: Maxime Ripard > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-in...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Signed-off-by: Rob Herring For the Documentation/devicetree/bindings/display/* parts: Acked-by: Sam Ravnborg Sam