Re: [PATCH v1 8/9] dt-bindings: iio: afe: add binding for temperature-sense-current

From: Peter Rosin
Date: Mon May 31 2021 - 04:58:33 EST




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".
>
> 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