Re: [PATCH 1/1] dt-bindings: iio: pressure: Remove vdd-supply and vddio-supply from required list

[Jonathan Cameron <jonathan.cameron@huawei.com>]

On Wed, 22 Oct 2025 12:03:27 -0500
David Lechner <dlechner@baylibre.com> wrote:

> On 10/22/25 11:41 AM, Frank Li wrote:
> > Some board designs connect vdd and vddio to the system power supply. Remove
> > these properties from the required list and make them optional, since
> > drivers/iio/pressure/mpl3115.c does not use them.
> > 
> > Fix below CHECK_DTBS warnings:
> > arch/arm64/boot/dts/freescale/imx8qm-mek.dtb: pressure-sensor@60 (fsl,mpl3115): 'vdd-supply' is a required property
> >         from schema $id: http://devicetree.org/schemas/iio/pressure/fsl,mpl3115.yaml#
> >   
> 
> Why not just add the required properties to the .dts file?

That would be the ideal. 

We had a long discussion a few years back around whether supplies like this
should be optional or not in dt-bindings. Conclusion (I think at Mark Brown's
suggestion) was that the rule should be if power is needed for the chip to
function they aren't optional in DT.

The driver doesn't necessarily do anything with them, though it probably
should and adding simple support is trivial. 

In a given downstream dts if people are happy to have the checks fail
then fallback regulators can be used but the binding shouldn't reflect
that bit of papering over missing supplies.

So no to this change. 

Jonathan

> 
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/iio/pressure/fsl,mpl3115.yaml | 2 --
> >  1 file changed, 2 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/iio/pressure/fsl,mpl3115.yaml b/Documentation/devicetree/bindings/iio/pressure/fsl,mpl3115.yaml
> > index 2933c2e10695e..04703a01cf7af 100644
> > --- a/Documentation/devicetree/bindings/iio/pressure/fsl,mpl3115.yaml
> > +++ b/Documentation/devicetree/bindings/iio/pressure/fsl,mpl3115.yaml
> > @@ -47,8 +47,6 @@ properties:
> >  required:
> >    - compatible
> >    - reg
> > -  - vdd-supply
> > -  - vddio-supply
> >  
> >  additionalProperties: false
> >    
> 
>