diff mbox series

[1/2] dt-bindings: power/supply: Add ltc4162-f/s and ltc4015

Message ID 20241027071852.56240-2-kimseer.paller@analog.com
State Superseded
Headers show
Series [1/2] dt-bindings: power/supply: Add ltc4162-f/s and ltc4015 | expand

Commit Message

Kim Seer Paller Oct. 27, 2024, 7:18 a.m. UTC
Add support for ltc4162-f/s and ltc4015

- Add compatible entries for ltc4162-f/s and ltc4015
- Include datasheets for new devices

Signed-off-by: Kim Seer Paller <kimseer.paller@analog.com>
---
 .../devicetree/bindings/power/supply/ltc4162-l.yaml         | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Rob Herring Oct. 27, 2024, 11:13 p.m. UTC | #1
On Sun, Oct 27, 2024 at 03:18:51PM +0800, Kim Seer Paller wrote:
> Add support for ltc4162-f/s and ltc4015
> 
> - Add compatible entries for ltc4162-f/s and ltc4015
> - Include datasheets for new devices

What's the difference between the l, f, and s variants? Please make the 
commit msg describe that rather than what I can read in the diff.

> 
> Signed-off-by: Kim Seer Paller <kimseer.paller@analog.com>
> ---
>  .../devicetree/bindings/power/supply/ltc4162-l.yaml         | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml b/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> index 29d536541..9b546150d 100644
> --- a/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> +++ b/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> @@ -17,12 +17,18 @@ description: |
>    panels, etc., and a rechargeable Lithium-Ion/Polymer battery.
>  
>    Specifications about the charger can be found at:
> +    https://www.analog.com/en/products/ltc4162-l.html
> +    https://www.analog.com/en/products/ltc4162-f.html
>      https://www.analog.com/en/products/ltc4162-s.html
> +    https://www.analog.com/en/products/ltc4015.html
>  
>  properties:
>    compatible:
>      enum:
>        - lltc,ltc4162-l
> +      - lltc,ltc4162-f
> +      - lltc,ltc4162-s
> +      - lltc,ltc4015
>  
>    reg:
>      maxItems: 1
> -- 
> 2.34.1
>
Kim Seer Paller Nov. 4, 2024, 5:30 a.m. UTC | #2
> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: Monday, October 28, 2024 7:13 AM
> To: Paller, Kim Seer <KimSeer.Paller@analog.com>
> Cc: linux-pm@vger.kernel.org; devicetree@vger.kernel.org; linux-
> kernel@vger.kernel.org; Sebastian Reichel <sre@kernel.org>; Krzysztof
> Kozlowski <krzk+dt@kernel.org>; Conor Dooley <conor+dt@kernel.org>; Mike
> Looijmans <mike.looijmans@topic.nl>
> Subject: Re: [PATCH 1/2] dt-bindings: power/supply: Add ltc4162-f/s and
> ltc4015
> 
> [External]
> 
> On Sun, Oct 27, 2024 at 03:18:51PM +0800, Kim Seer Paller wrote:
> > Add support for ltc4162-f/s and ltc4015
> >
> > - Add compatible entries for ltc4162-f/s and ltc4015
> > - Include datasheets for new devices
> 
> What's the difference between the l, f, and s variants? Please make the commit
> msg describe that rather than what I can read in the diff.

I'll take note of that. Here is the difference between the variants:
LTC4162-L 35V/3.2A Multi-Cell Lithium-Ion Step-Down Battery Charger
LTC4162-F 35V/3.2A Multi-Cell LiFePO4 Step-Down Battery Charger
LTC4162-S 35V/3.2A Lead-Acid Step-Down Battery Charger
LTC4015 35V/3.2A Multichemistry Buck Battery Charger Controller

> 
> >
> > Signed-off-by: Kim Seer Paller <kimseer.paller@analog.com>
> > ---
> >  .../devicetree/bindings/power/supply/ltc4162-l.yaml         | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git
> > a/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> > b/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> > index 29d536541..9b546150d 100644
> > --- a/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> > +++ b/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
> > @@ -17,12 +17,18 @@ description: |
> >    panels, etc., and a rechargeable Lithium-Ion/Polymer battery.
> >
> >    Specifications about the charger can be found at:
> > +    https://www.analog.com/en/products/ltc4162-l.html
> > +    https://www.analog.com/en/products/ltc4162-f.html
> >      https://www.analog.com/en/products/ltc4162-s.html
> > +    https://www.analog.com/en/products/ltc4015.html
> >
> >  properties:
> >    compatible:
> >      enum:
> >        - lltc,ltc4162-l
> > +      - lltc,ltc4162-f
> > +      - lltc,ltc4162-s
> > +      - lltc,ltc4015
> >
> >    reg:
> >      maxItems: 1
> > --
> > 2.34.1
> >
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml b/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
index 29d536541..9b546150d 100644
--- a/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
+++ b/Documentation/devicetree/bindings/power/supply/ltc4162-l.yaml
@@ -17,12 +17,18 @@  description: |
   panels, etc., and a rechargeable Lithium-Ion/Polymer battery.
 
   Specifications about the charger can be found at:
+    https://www.analog.com/en/products/ltc4162-l.html
+    https://www.analog.com/en/products/ltc4162-f.html
     https://www.analog.com/en/products/ltc4162-s.html
+    https://www.analog.com/en/products/ltc4015.html
 
 properties:
   compatible:
     enum:
       - lltc,ltc4162-l
+      - lltc,ltc4162-f
+      - lltc,ltc4162-s
+      - lltc,ltc4015
 
   reg:
     maxItems: 1