Thread (38 messages) 38 messages, 5 authors, 2021-06-05

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

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
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help