| Message ID | 20220525073657.573327-3-sst@poczta.fm |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | [1/7] dt-bindings: hwmon: Add compatible string for ADT7481 in lm90 | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/patch-applied | fail | build log |
On Wed, 25 May 2022 09:36:52 +0200, Slawomir Stepien wrote: > From: Slawomir Stepien <slawomir.stepien@nokia.com> > > Add binding description for temperature channels. Currently, support for > label and temperature-offset-millicelsius is implemented. > > Signed-off-by: Slawomir Stepien <slawomir.stepien@nokia.com> > --- > .../bindings/hwmon/national,lm90.yaml | 129 ++++++++++++++++++ > 1 file changed, 129 insertions(+) > Reviewed-by: Rob Herring <robh@kernel.org>
On Wed, May 25, 2022 at 09:36:52AM +0200, Slawomir Stepien wrote: > From: Slawomir Stepien <slawomir.stepien@nokia.com> > > Add binding description for temperature channels. Currently, support for > label and temperature-offset-millicelsius is implemented. > > Signed-off-by: Slawomir Stepien <slawomir.stepien@nokia.com> > Reviewed-by: Rob Herring <robh@kernel.org> Applied to hwmon-next. Thanks, Guenter > --- > .../bindings/hwmon/national,lm90.yaml | 129 ++++++++++++++++++ > 1 file changed, 129 insertions(+) > > diff --git a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml > index 82fce96498c7..e1719839faf0 100644 > --- a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml > +++ b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml > @@ -51,6 +51,12 @@ properties: > "#thermal-sensor-cells": > const: 1 > > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 0 > + > vcc-supply: > description: phandle to the regulator that provides the +VCC supply > > @@ -62,6 +68,29 @@ required: > - compatible > - reg > > +patternProperties: > + "^channel@([0-2])$": > + type: object > + description: Represents channels of the device and their specific configuration. > + > + properties: > + reg: > + description: The channel number. 0 is local channel, 1-2 are remote channels. > + items: > + minimum: 0 > + maximum: 2 > + > + label: > + description: A descriptive name for this channel, like "ambient" or "psu". > + > + temperature-offset-millicelsius: > + description: Temperature offset to be added to or subtracted from remote temperature measurements. > + > + required: > + - reg > + > + additionalProperties: false > + > allOf: > - if: > not: > @@ -78,6 +107,77 @@ allOf: > properties: > ti,extended-range-enable: false > > + - if: > + properties: > + compatible: > + contains: > + enum: > + - dallas,max6646 > + - dallas,max6647 > + - dallas,max6649 > + - dallas,max6657 > + - dallas,max6658 > + - dallas,max6659 > + - dallas,max6695 > + - dallas,max6696 > + then: > + patternProperties: > + "^channel@([0-2])$": > + properties: > + temperature-offset-millicelsius: false > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - adi,adt7461 > + - adi,adt7461a > + - adi,adt7481 > + - onnn,nct1008 > + then: > + patternProperties: > + "^channel@([0-2])$": > + properties: > + temperature-offset-millicelsius: > + maximum: 127750 > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - adi,adm1032 > + - dallas,max6680 > + - dallas,max6681 > + - gmt,g781 > + - national,lm86 > + - national,lm89 > + - national,lm90 > + - national,lm99 > + - nxp,sa56004 > + - winbond,w83l771 > + then: > + patternProperties: > + "^channel@([0-2])$": > + properties: > + temperature-offset-millicelsius: > + maximum: 127875 > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - ti,tmp451 > + - ti,tmp461 > + then: > + patternProperties: > + "^channel@([0-2])$": > + properties: > + temperature-offset-millicelsius: > + maximum: 127937 > + > additionalProperties: false > > examples: > @@ -96,3 +196,32 @@ examples: > #thermal-sensor-cells = <1>; > }; > }; > + - | > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + sensor@4c { > + compatible = "adi,adt7481"; > + reg = <0x4c>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + channel@0 { > + reg = <0x0>; > + label = "local"; > + }; > + > + channel@1 { > + reg = <0x1>; > + label = "front"; > + temperature-offset-millicelsius = <4000>; > + }; > + > + channel@2 { > + reg = <0x2>; > + label = "back"; > + temperature-offset-millicelsius = <750>; > + }; > + }; > + };
diff --git a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml index 82fce96498c7..e1719839faf0 100644 --- a/Documentation/devicetree/bindings/hwmon/national,lm90.yaml +++ b/Documentation/devicetree/bindings/hwmon/national,lm90.yaml @@ -51,6 +51,12 @@ properties: "#thermal-sensor-cells": const: 1 + '#address-cells': + const: 1 + + '#size-cells': + const: 0 + vcc-supply: description: phandle to the regulator that provides the +VCC supply @@ -62,6 +68,29 @@ required: - compatible - reg +patternProperties: + "^channel@([0-2])$": + type: object + description: Represents channels of the device and their specific configuration. + + properties: + reg: + description: The channel number. 0 is local channel, 1-2 are remote channels. + items: + minimum: 0 + maximum: 2 + + label: + description: A descriptive name for this channel, like "ambient" or "psu". + + temperature-offset-millicelsius: + description: Temperature offset to be added to or subtracted from remote temperature measurements. + + required: + - reg + + additionalProperties: false + allOf: - if: not: @@ -78,6 +107,77 @@ allOf: properties: ti,extended-range-enable: false + - if: + properties: + compatible: + contains: + enum: + - dallas,max6646 + - dallas,max6647 + - dallas,max6649 + - dallas,max6657 + - dallas,max6658 + - dallas,max6659 + - dallas,max6695 + - dallas,max6696 + then: + patternProperties: + "^channel@([0-2])$": + properties: + temperature-offset-millicelsius: false + + - if: + properties: + compatible: + contains: + enum: + - adi,adt7461 + - adi,adt7461a + - adi,adt7481 + - onnn,nct1008 + then: + patternProperties: + "^channel@([0-2])$": + properties: + temperature-offset-millicelsius: + maximum: 127750 + + - if: + properties: + compatible: + contains: + enum: + - adi,adm1032 + - dallas,max6680 + - dallas,max6681 + - gmt,g781 + - national,lm86 + - national,lm89 + - national,lm90 + - national,lm99 + - nxp,sa56004 + - winbond,w83l771 + then: + patternProperties: + "^channel@([0-2])$": + properties: + temperature-offset-millicelsius: + maximum: 127875 + + - if: + properties: + compatible: + contains: + enum: + - ti,tmp451 + - ti,tmp461 + then: + patternProperties: + "^channel@([0-2])$": + properties: + temperature-offset-millicelsius: + maximum: 127937 + additionalProperties: false examples: @@ -96,3 +196,32 @@ examples: #thermal-sensor-cells = <1>; }; }; + - | + i2c { + #address-cells = <1>; + #size-cells = <0>; + + sensor@4c { + compatible = "adi,adt7481"; + reg = <0x4c>; + #address-cells = <1>; + #size-cells = <0>; + + channel@0 { + reg = <0x0>; + label = "local"; + }; + + channel@1 { + reg = <0x1>; + label = "front"; + temperature-offset-millicelsius = <4000>; + }; + + channel@2 { + reg = <0x2>; + label = "back"; + temperature-offset-millicelsius = <750>; + }; + }; + };