RE: [PATCH 2/2] iio: temperature: ltc2983: Add support for ADT7604

From: Stan, Liviu

Date: Wed May 13 2026 - 12:42:52 EST


On Tue, 12 May 2026, Jonathan Cameron wrote:
> > > I'm perhaps missing something - as far as I understand it there is no
> > > meaningful connection to resistance in what is being measured.
> > > I think what you are proposing is similar to measuring current via voltage
> > > drop over a sense resistor. We don't present that as modified voltage, we
> > > present it as current.
> > >
> > > Here the thing being measured is coverage rather than resistance
> > > so keeping resistance in there is confusing for the user.
> > >
> > > If we wanted a type to modify then we could do this as a modified area
> > > measurement. Channel type IIO_AREA (which is new) and modifier
> > > IIO_MOD_RATIO (also new).
> >
> > On the implementation: you originally suggested IIO_COVERAGE_PERCENT,
> > which would give in_coveragepercent0_raw in sysfs. The IIO_AREA +
> > IIO_MOD_RATIO alternative would give in_area0_ratio_raw, which is
> > more generic but less immediately obvious for a leak detector. Do you
> > have a preference between the two?
>
> I wonder what other reasonable area sensors we'll get in future?
> Maybe more specific is better here - like humidityrelative (we've never had
> an absolute humidity sensor because they are really hard to build!)

I agree. Thanks for the input. Will go with IIO_COVERAGE_PERCENT in v2 then.

Thanks,
Liviu