[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<PH0PR03MB65258CC31A76A0CC9695A75CED97A@PH0PR03MB6525.namprd03.prod.outlook.com>
Date: Thu, 22 Jan 2026 07:03:54 +0000
From: "Escala, Edelweise" <Edelweise.Escala@...log.com>
To: Conor Dooley <conor@...nel.org>
CC: Lee Jones <lee@...nel.org>, Pavel Machek <pavel@...nel.org>,
Rob Herring
<robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley
<conor+dt@...nel.org>,
"linux-leds@...r.kernel.org"
<linux-leds@...r.kernel.org>,
"devicetree@...r.kernel.org"
<devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v3 1/2] dt-bindings: leds: Add LTC3220 18 channel LED
Driver
> -----Original Message-----
> From: Conor Dooley <conor@...nel.org>
> Sent: Wednesday, January 21, 2026 3:56 AM
> To: Escala, Edelweise <Edelweise.Escala@...log.com>
> Cc: Lee Jones <lee@...nel.org>; Pavel Machek <pavel@...nel.org>; Rob Herring
> <robh@...nel.org>; Krzysztof Kozlowski <krzk+dt@...nel.org>; Conor Dooley
> <conor+dt@...nel.org>; linux-leds@...r.kernel.org; devicetree@...r.kernel.org;
> linux-kernel@...r.kernel.org
> Subject: Re: [PATCH v3 1/2] dt-bindings: leds: Add LTC3220 18 channel LED Driver
>
> [External]
>
> On Tue, Jan 20, 2026 at 01:35:06PM +0800, Edelweise Escala wrote:
> > Document device tree bindings for the LTC3220 18-channel LED driver
> > with I2C interface, individual brightness control, and
> > hardware-assisted blink/gradation features.
> >
> > Signed-off-by: Edelweise Escala <edelweise.escala@...log.com>
> > ---
> > .../devicetree/bindings/leds/leds-ltc3220.yaml | 120
> +++++++++++++++++++++
> > MAINTAINERS | 7 ++
> > 2 files changed, 127 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/leds/leds-ltc3220.yaml
> > b/Documentation/devicetree/bindings/leds/leds-ltc3220.yaml
> > new file mode 100644
> > index 000000000000..6274c6c6ad79
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/leds/leds-ltc3220.yaml
>
> filename should match the compatible, so s/leds-/adi,/
>
> Otherwise, this looks pretty okay to me. With that changed,
> Reviewed-by: Conor Dooley <conor.dooley@...rochip.com>
> pw-bot: changes-requested
>
> Cheers,
> Conor.
>
Noted on this. Filename will be changed to adi,ltc3220.yaml
Best Regards,
Edelweise Escala
> > @@ -0,0 +1,120 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/leds/leds-ltc3220.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Analog Devices LTC3220 LED Driver
> > +
> > +maintainers:
> > + - Edelweise Escala <edelweise.escala@...log.com>
> > +
> > +description:
> > + The LTC3220 is a multi-display LED driver, which contains a
> > +high-efficiency,
> > + low-noise charge pump to provide power to up to 18 LED current sources.
> > + The LEDs are individually configurable to 64-step linear brightness
> > +control,
> > + blinking and gradation control via 2-wire I2C interface.
> > +
> > + For more product information please see the link below
> > + https://www.analog.com/en/products/ltc3220.html
> > +
> > +properties:
> > + compatible:
> > + const: adi,ltc3220
Powered by blists - more mailing lists