Re: [PATCH v1 8/9] dt-bindings: iio: afe: add binding for temperature-sense-current
From: Liam Beguin <hidden>
Date: 2021-05-31 15:17:18
Also in:
linux-devicetree, lkml
From: Liam Beguin <hidden>
Date: 2021-05-31 15:17:18
Also in:
linux-devicetree, lkml
Hi Peter, On Mon May 31, 2021 at 4:58 AM EDT, Peter Rosin wrote:
On 2021-05-31 09:28, Peter Rosin wrote:quoted
quoted
+ 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
quoted
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