Re: [PATCH v2 1/2] dt-bindings: iio: temperature: add support for MCP998X

[Victor.Duicu]

On Fri, 2025-05-30 at 16:55 +0100, Conor Dooley wrote:
> On Thu, May 29, 2025 at 01:13:38PM -0500, David Lechner wrote:
> > On 5/29/25 4:36 AM, victor.duicu@xxxxxxxxxxxxx wrote:
> > > From: Victor Duicu <victor.duicu@xxxxxxxxxxxxx>
> > > 
> > > 
Hi Conor,

...

> > > +  microchip,recd12:
> > > +    description:
> > > +      Enable resistance error correction for external channels 1
> > > and 2.
> > > +      Omit this tag to disable REC for channels 1 and 2.
> > > +    type: boolean
> > > +
> > > +  microchip,recd34:
> > > +    description:
> > > +      Enable resistance error correction for external channels 3
> > > and 4.
> > > +      Omit this tag to disable REC for channels 3 and 4.
> 
> Why are these two devicetree properties, rather than runtime
> controls?

The parasitic resistance added to the series resistance is dependent
only on the circuit. 
It is possible for the chip and the transistor to be at some distance
from each other. The manufacturer can approximate the error added and
decide if resistance error correction should be applied.
The user cannot influence the parasitic resistance nor calculate it.

With Kind Regards,
Victor Duicu