[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240214-boat-drapery-3f957d393ccc@spud>
Date: Wed, 14 Feb 2024 17:29:36 +0000
From: Conor Dooley <conor@...nel.org>
To: Jonathan Cameron <Jonathan.Cameron@...wei.com>
Cc: Ondřej Jirman <megi@....cz>,
Jonathan Cameron <jic23@...nel.org>,
Lars-Peter Clausen <lars@...afoo.de>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>,
Andrey Skvortsov <andrej.skvortzov@...il.com>,
Icenowy Zheng <icenowy@...c.io>, linux-iio@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/4] dt-bindings: iio: magnetometer: Add DT binding for
Voltafield AF8133J
On Wed, Feb 14, 2024 at 04:31:16PM +0000, Jonathan Cameron wrote:
>
> > > > +
> > > > + dvdd-supply:
> > > > + description: |
> > > > + an optional regulator that needs to be on to provide DVDD power (Digital
> > > > + IO power, 1.8V~AVDD) to the sensor.
> > > > +
> > > > + mount-matrix:
> > > > + description: an optional 3x3 mounting rotation matrix.
> > > > +
> > > > +required:
> > > > + - compatible
> > > > + - reg
> > >
> > > Any power supply that is required for operation should be listed here (even though
> > > we can rely on the stub supplies if it isn't in the DT).
> > > I used to think this wasn't necessary, so lots of bindings upstream don't yet
> > > have it.
> >
> > By stub supply you mean dummy supply created when the *-supply property is not
> > specified in DT? Or something else?
>
> Ah yes. I got the term wrong.
> >
> > Because DTC_CHK prints a warning if I make the proerty required in bindings and
> > not specify it in DT
> >
> > arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone-1.2b.dtb: magnetometer@1c: 'avdd-supply' is a required property
> > from schema $id: http://devicetree.org/schemas/iio/magnetometer/voltafield,af8133j.yaml#
>
> Provide one, or don't worry about that warning.
For the sake of the platform maintainer, please choose option 1.
Thanks,
Conor.
> Various discussions have taken place on this over time and end
> result is bindings should require them to differentiate from power
> supplies that are actually optional.
>
> Jonathan
>
>
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists