linux-kernel - Re: [PATCH v4 2/3] dt-bindings: hwmon: ti,ina2xx: Add ti,shunt-gain property

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20211109203216.GA3693367@roeck-us.net>
Date:   Tue, 9 Nov 2021 12:32:16 -0800
From:   Guenter Roeck <linux@...ck-us.net>
To:     Nathan Rossi <nathan@...hanrossi.com>
Cc:     linux-hwmon@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, Nathan Rossi <nathan.rossi@...i.com>,
        Jean Delvare <jdelvare@...e.com>,
        Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH v4 2/3] dt-bindings: hwmon: ti,ina2xx: Add ti,shunt-gain
 property

On Tue, Nov 02, 2021 at 05:27:54AM +0000, Nathan Rossi wrote:
> From: Nathan Rossi <nathan.rossi@...i.com>
> 
> Add a property to the binding to define the selected shunt voltage gain.
> This specifies the range and accuracy that applies to the shunt circuit.
> This property only applies to devices that have a selectable shunt
> voltage range via PGA or ADCRANGE register configuration.
> 
> Signed-off-by: Nathan Rossi <nathan.rossi@...i.com>
> Reviewed-by: Rob Herring <robh@...nel.org>

Applied to hwmon-next.

Thanks,
Guenter

> ---
> Changes in v2:
> - Added binding for shunt-gain
> 
> Changes in v3:
> - Fix schema error, setting $ref to uint32
> - Improve the description to detail exactly how to define the property
>   and how the property affects initial device configuration and
>   calculation of values
> ---
>  .../devicetree/bindings/hwmon/ti,ina2xx.yaml        | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 
> ---
> 2.33.0
> 
> diff --git a/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml b/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
> index 180573f26c..47af97bb4c 100644
> --- a/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/ti,ina2xx.yaml
> @@ -36,6 +36,27 @@ properties:
>        Shunt resistor value in micro-Ohm.
>      $ref: /schemas/types.yaml#/definitions/uint32
>  
> +  ti,shunt-gain:
> +    description: |
> +      Programmable gain divisor for the shunt voltage accuracy and range. This
> +      property only applies to devices that have configurable PGA/ADCRANGE. The
> +      gain value is used configure the gain and to convert the shunt voltage,
> +      current and power register values when reading measurements from the
> +      device.
> +
> +      For devices that have a configurable PGA (e.g. INA209, INA219, INA220),
> +      the gain value maps directly with the PG bits of the config register.
> +
> +      For devices that have ADCRANGE configuration (e.g. INA238) a shunt-gain
> +      value of 1 maps to ADCRANGE=1 where no gain divisor is applied to the
> +      shunt voltage, and a value of 4 maps to ADCRANGE=0 such that a wider
> +      voltage range is used.
> +
> +      The default value is device dependent, and is defined by the reset value
> +      of PGA/ADCRANGE in the respective configuration registers.
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    enum: [1, 2, 4, 8]
> +
>  required:
>    - compatible
>    - reg