linux-kernel - Re: [PATCH] dt-bindings: power/supply: Add ltc4162-l-charger

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20201026144331.GA123160@bogus>
Date:   Mon, 26 Oct 2020 09:43:31 -0500
From:   Rob Herring <robh@...nel.org>
To:     Mike Looijmans <mike.looijmans@...ic.nl>
Cc:     sre@...nel.org, devicetree@...r.kernel.org,
        linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: power/supply: Add ltc4162-l-charger

On Wed, Oct 21, 2020 at 04:10:30PM +0200, Mike Looijmans wrote:
> Add support for the LTC4162-L Li-Ion battery charger. The driver allows
> reading back telemetry and to set some charging options like the input
> current limit.
> 
> This adds the devicetree bindings.
> 
> Signed-off-by: Mike Looijmans <mike.looijmans@...ic.nl>
> ---
>  .../bindings/power/supply/ltc4162-l.yaml      | 60 +++++++++++++++++++
>  1 file changed, 60 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> 
> diff --git a/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml b/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> new file mode 100644
> index 000000000000..a23dd6f3fae0
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> @@ -0,0 +1,60 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +# Copyright (C) 2020 Topic Embedded Products
> +%YAML 1.2
> +---
> +$id: "http://devicetree.org/schemas/power/supply/ltc4162-l.yaml#"
> +$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +
> +title: Linear Technology (Analog Devices) LTC4162-L Charger
> +
> +maintainers:
> +  - Mike Looijmans <mike.looijmans@...ic.nl>
> +
> +description: |
> +  The LTC ® 4162-L is an advanced monolithic synchronous step-down switching
> +  battery charger and PowerPath (TM) manager that seamlessly manages power
> +  distribution between input sources such as wall adapters, backplanes, solar
> +  panels, etc., and a rechargeable Lithium-Ion/Polymer battery.
> +
> +  Specifications about the charger can be found at:
> +    https://www.analog.com/en/products/ltc4162-s.html
> +
> +properties:
> +  compatible:
> +    enum:
> +      - lltc,ltc4162-l
> +
> +  reg:
> +    maxItems: 1
> +    description: I2C address of the charger.
> +
> +  lltc,rsnsb:
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    description: Battery sense resistor in milli Ohm.
> +    minimum: 1
> +
> +  lltc,rsnsi:
> +    $ref: /schemas/types.yaml#/definitions/uint32
> +    description: Input current sense resistor in milli Ohm.
> +    minimum: 1

These should have a unit suffix as defined in property-units.txt. 
There's not one for milli Ohms, does micro Ohms give you enough range? 
If not, you can add it though that's discouraged simply so that we don't 
have differring units in each binding for the same thing.

Rob