Messages in this thread | | | Date | Mon, 31 May 2021 09:41:24 -0400 | From | "Liam Beguin" <> | Subject | Re: [PATCH v1 8/9] dt-bindings: iio: afe: add binding for temperature-sense-current |
| |
Hi Peter,
On Mon May 31, 2021 at 4:58 AM EDT, Peter Rosin wrote: > > > On 2021-05-31 09:28, Peter Rosin wrote: > >> + use-kelvin-scale: > >> + type: boolean > >> + description: | > >> + Boolean indicating if alpha uses Kelvin degrees instead of Celsius. > > > > It's "kelvin", not "Kelvin degrees", and it's "degrees Celsius".
I'll rephrase the description base on your comment.
I also thought of using alpha-micro-amps-per-degree-celsius and alpha-micro-amps-kelvin instead. I don't know if that would be better.
Thanks, Liam
> > > > But what exactly is this property for? We always want degrees Celsius, don't we, > > and any offset can be handled...with an offset. No? > > Ahh, I hit send too soon. I now see that you calculate the offset in the > driver > instead of requiring the devicetree author to do it "by hand". > > Cheers, > Peter
| |