Re: [v2 1/2] dt-bindings: add extended-range-enable property to lm90.yaml

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 5/11/22 08:32, Krzysztof Kozlowski wrote:
On 10/05/2022 10:08, Holger Brunck wrote:
Some devices can operate in an extended temperature mode.
Therefore add a boolean onsemi,extended-range-enable to be able to
select this feature in the device tree node.

Signed-off-by: Holger Brunck <holger.brunck@xxxxxxxxxxxxxxxxx>
cc: Jean Delvare <jdelvare@xxxxxxxx>
cc: Guenter Roeck <linux@xxxxxxxxxxxx>
cc: Rob Herring <robh+dt@xxxxxxxxxx>
cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>
---
  Documentation/devicetree/bindings/hwmon/national,lm90.yaml | 4 ++++
  1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml
index 30db92977937..92afa01380eb 100644
--- a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml
+++ b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml
@@ -52,6 +52,10 @@ properties:
    vcc-supply:
      description: phandle to the regulator that provides the +VCC supply
+ onsemi,extended-range-enable:
+    description: Set to enable extended range temperature.
+    type: boolean


There is no such vendor and it does not match the existing vendor for
these bindings (nor the current owner of National). Was there some
change? What is onsemi?

My bad, I should have looked up official prefixes before suggesting onsemi
as an option. That should have been "onnn".

It should be either onnn (for adt7461/adt7461a) or ti for tmp451
and tmp461. adi instead of onnn may make sense since that is already
used in the driver. I personally don't have a preference.

Guenter



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux