Re: [PATCH v3 01/10] dt-bindings: iio: adc: Add AD4170

[Marcelo Schmitt <marcelo.schmitt1@gmail.com>]

Hi Jonathan,

I'm still working on the changes suggested for this set.
Just to mention, I've updated the dt-binding to declare only channel nodes under
the ADC node.

... 
> > +      adi,sensor-type:
> > +        description: |
> > +          Type of sensor connected to the device. Depending on the sensor type
> > +          (weigh scale, RTD, or thermocouple) the values of sensor-node
> > +          properties have slightly different constraints. This property
> > +          specifies which particular external sensor is connected to the ADC so
> > +          the sensor-node properties can be properly parsed and verified. The
> > +          possible sensor types are:
> > +          0: weigh scale;
> > +          1: RTD;
> > +          2: thermocouple.
> > +        $ref: /schemas/types.yaml#/definitions/uint8
> 
> I think this should be an enum of strings.  That will give us a clean
> way to extend it for other sensor types in future.
Ack

...
> > +
> > +  interrupts:
> > +    maxItems: 1
> 
> What if they are both wired?  At that point shouldn't software
> make up it's mind which to use?

Not sure I understand the concern here. Is 'interrupt-names' property expected
to precede 'interrupts' in dt-doc? For AD4170 and similar parts, the data
ready signal (/RDY) is by default provided on the SDO line after the completion
of a conversion. Alternatively, it can be provided on the DIG_AUX1 pin in
which case the chip disables the RDY function on SDO. So, there can only be one
data ready interrupt enabled at a time. Guess I'll add some description to make
clear only one interrupt can be enabled.

> > +
> > +  interrupt-names:
> > +    description:
> > +      Specify which pin should be configured as Data Ready interrupt.
> > +    enum:
> > +      - sdo
> > +      - dig_aux1
> > +    default: sdo
> > +

Thanks,
Marcelo