Re: [PATCH v2 6/7] dt-bindings: iio: temperature: Add ADT7604 support to adi,ltc2983

[Liviu Stan]

On Sat, 16 May 2026 18:03:58 +0100 Jonathan Cameron <jic23@xxxxxxxxxx> wrote:

...

> > +  '^leak-detector@':
> > +    $ref: '#/$defs/sensor-node'
> > +    unevaluatedProperties: false
> > +    description: |
> > +      Leak detector sensor (some parts only). Outputs resistance in ohms and,
> > +      when a custom table is provided, a coverage percentage via
> > +      IIO_COVERAGE_PERCENT (raw/1024 = coverage %).
> > +
> > +    properties:
> > +      reg:
> > +        minimum: 2
> > +        maximum: 20
> > +
> > +      adi,sensor-type:
> > +        description: Sensor type for leak detector sensors.
> > +        $ref: /schemas/types.yaml#/definitions/uint32
> > +        const: 33
> > +
> > +      adi,rsense-handle:
> > +        description: Associated sense resistor sensor.
> > +        $ref: /schemas/types.yaml#/definitions/phandle
> > +
> > +      adi,excitation-current-nanoamp:
> > +        description:
> > +          Excitation current applied to the leak detector. The correct value
> > +          depends on the electrical characteristics of the liquid being sensed.
> > +          For example, 10000 (10µA) is recommended for PG25 (see datasheet
> > +          Table 39).
> > +        enum: [250, 500, 1000, 5000, 10000, 25000, 50000, 100000, 250000,
> > +               500000, 1000000]
> > +
> > +      adi,custom-leak-detector:
> > +        description: |
> > +          Lookup table mapping resistance to coverage data. Entries must be
> > +          in ascending resistance order. The coverage data field encodes the
> > +          coverage percentage P as (P + 273.15) expressed in uK, i.e.
> > +          (P * 1000000 + 273150000).
> 
> I like sashiko's comment on this.  Why are we having the hardware / driver
> offset encoded in DT?  Just put in the value without the offset and apply that
> in the driver.
> 

Will do this in v3.

Thanks,
Liviu