[PATCH v4 10/13] dt-bindings: iio: temperature: ltc2983: describe broken mux delay property

Cosmin Tanislav posted 13 patches 3 years, 5 months ago
[PATCH v4 10/13] dt-bindings: iio: temperature: ltc2983: describe broken mux delay property
Posted by Cosmin Tanislav 3 years, 5 months ago
From: Cosmin Tanislav <cosmin.tanislav@analog.com>

The 'adi,mux-delay-config-us' property is broken. It was supposed to
be in us, but the value is actually written directly to the register.
Describe the fact that it is broken and how it actually works.

Signed-off-by: Cosmin Tanislav <cosmin.tanislav@analog.com>
---
 .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml       | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
index 676801b036cf..467e165e9b0b 100644
--- a/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
+++ b/Documentation/devicetree/bindings/iio/temperature/adi,ltc2983.yaml
@@ -29,6 +29,9 @@ properties:
       Extra delay prior to each conversion, in addition to the internal 1ms
       delay, for the multiplexer to switch input configurations and
       excitation values.
+
+      This property is supposed to be in microseconds, but to maintain
+      compatibility, this value will be multiplied by 100 before usage.
     maximum: 255
     default: 0
 
-- 
2.38.1
Re: [PATCH v4 10/13] dt-bindings: iio: temperature: ltc2983: describe broken mux delay property
Posted by Rob Herring 3 years, 5 months ago
On Thu, 03 Nov 2022 15:00:38 +0200, Cosmin Tanislav wrote:
> From: Cosmin Tanislav <cosmin.tanislav@analog.com>
> 
> The 'adi,mux-delay-config-us' property is broken. It was supposed to
> be in us, but the value is actually written directly to the register.
> Describe the fact that it is broken and how it actually works.
> 
> Signed-off-by: Cosmin Tanislav <cosmin.tanislav@analog.com>
> ---
>  .../devicetree/bindings/iio/temperature/adi,ltc2983.yaml       | 3 +++
>  1 file changed, 3 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>