On Tue, Oct 03, 2023 at 11:00:39AM +0300, Antoniu Miclaus wrote: > Add dt-bindings for ltc2991 octal i2c voltage, current and temperature > monitor. > > Signed-off-by: Antoniu Miclaus <antoniu.miclaus@xxxxxxxxxx> > --- > changes in v3: > - expand description for differential channels and temperature channels by > mentioning the pin configuration that is set via these properties. > - add `additionalProperties: false` for the channel properties > - make `reg` required for channel properties > - make `adi,temperature-enable` and `shunt-resistor-micro-ohms` mutually > exclusive following the approach suggested in review. Thanks for the updates. Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> Cheers, Conor. > .../bindings/hwmon/adi,ltc2991.yaml | 128 ++++++++++++++++++ > 1 file changed, 128 insertions(+) > create mode 100644 Documentation/devicetree/bindings/hwmon/adi,ltc2991.yaml > > diff --git a/Documentation/devicetree/bindings/hwmon/adi,ltc2991.yaml b/Documentation/devicetree/bindings/hwmon/adi,ltc2991.yaml > new file mode 100644 > index 000000000000..011e5b65c79c > --- /dev/null > +++ b/Documentation/devicetree/bindings/hwmon/adi,ltc2991.yaml > @@ -0,0 +1,128 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > + > +$id: http://devicetree.org/schemas/hwmon/adi,ltc2991.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Analog Devices LTC2991 Octal I2C Voltage, Current and Temperature Monitor > + > +maintainers: > + - Antoniu Miclaus <antoniu.miclaus@xxxxxxxxxx> > + > +description: | > + The LTC2991 is used to monitor system temperatures, voltages and currents. > + Through the I2C serial interface, the eight monitors can individually measure > + supply voltages and can be paired for differential measurements of current > + sense resistors or temperature sensing transistors. > + > + Datasheet: > + https://www.analog.com/en/products/ltc2991.html > + > +properties: > + compatible: > + const: adi,ltc2991 > + > + reg: > + maxItems: 1 > + > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 0 > + > + vcc-supply: true > + > +patternProperties: > + "^channel@[0-3]$": > + type: object > + description: > + Represents the differential/temperature channels. > + > + properties: > + reg: > + description: > + The channel number. LTC2991 can monitor 4 currents/temperatures. > + items: > + minimum: 0 > + maximum: 3 > + > + shunt-resistor-micro-ohms: > + description: > + The value of curent sense resistor in micro ohms. Pin configuration is > + set for differential input pair. > + > + adi,temperature-enable: > + description: > + Enables temperature readings. Pin configuration is set for remote > + diode temperature measurement. > + type: boolean > + > + required: > + - reg > + > + allOf: > + - if: > + required: > + - shunt-resistor-micro-ohms > + then: > + properties: > + adi,temperature-enable: false > + > + additionalProperties: false > + > +required: > + - compatible > + - reg > + - vcc-supply > + > +additionalProperties: false > + > +examples: > + - | > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + hwmon@48 { > + compatible = "adi,ltc2991"; > + reg = <0x48>; > + vcc-supply = <&vcc>; > + }; > + }; > + - | > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + hwmon@48 { > + #address-cells = <1>; > + #size-cells = <0>; > + > + compatible = "adi,ltc2991"; > + reg = <0x48>; > + vcc-supply = <&vcc>; > + > + channel@0 { > + reg = <0x0>; > + shunt-resistor-micro-ohms = <100000>; > + }; > + > + channel@1 { > + reg = <0x1>; > + shunt-resistor-micro-ohms = <100000>; > + }; > + > + channel@2 { > + reg = <0x2>; > + adi,temperature-enable; > + }; > + > + channel@3 { > + reg = <0x3>; > + adi,temperature-enable; > + }; > + }; > + }; > +... > -- > 2.42.0 >
Attachment:
signature.asc
Description: PGP signature