Re: [PATCH v8 v8 02/11] dt-bindings: iio: adc: hx711: add VSUP supply property

[Piyush Patle]

On Tue, May 12, 2026 at 5:38 PM Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>
> On Tue, 12 May 2026 13:06:22 +0100
> Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>
> > On Mon, 11 May 2026 23:13:27 +0530
> > Piyush Patle <piyushpatle228@xxxxxxxxx> wrote:
> >
> > > Document the optional VSUP supply used by the HX711 on-chip regulator.
> > >
> > > Signed-off-by: Piyush Patle <piyushpatle228@xxxxxxxxx>
> > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
> > > Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> > > ---
> >
> > I got curious given the driver doesn't yet turn this on and wanted
> > to just sanity check it via a datasheet.
> >
> > Why are we including this one but not dvdd-supply?  The suggested
> > wiring does connect them to the same supply but it's external to the
> > chip so in theory they might not be.  Curiously dvdd supply is allowed
> > to be 0.1 V lower than vsup-supply.
The HX711 does have a DVDD pin, but it is the output of the internal
LDO and is normally just bypassed to ground. VSUP is the rail supplied by
the board, so that was the one I documented here.
That said, dvdd-supply is already allowed for HX711 nodes by patch 04,
as only vref-supply is forbidden for HX711. So the binding does not block
a board from describing DVDD if needed.
The driver does not use DVDD for HX711 today, but explicit support can be
added later if there is a real user for it.
>
> I should have said, given there is nothing wrong with what you have here
> and the above is a possible additional change - applied to the testing
> branch of iio.git.
>
> Thanks,
>
> Jonathan
>
Thanks for applying patches 01-03.
> >
> > >  Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml | 4 ++++
> > >  1 file changed, 4 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml b/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> > > index 1ea60dff98d5..a8eaa1f18de5 100644
> > > --- a/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> > > +++ b/Documentation/devicetree/bindings/iio/adc/avia-hx711.yaml
> > > @@ -33,6 +33,10 @@ properties:
> > >      description:
> > >        Analog supply voltage (AVDD).
> > >
> > > +  vsup-supply:
> > > +    description:
> > > +      Supply voltage for the on-chip regulator (VSUP).
> > > +
> > >    clock-frequency:
> > >      description:
> > >        Controls the SCK bit-bang timing. The value is used to derive the
> >
>