Re: [PATCH v2 2/2] leds: add aw20xx driver

From: Lee Jones
Date: Wed Mar 01 2023 - 03:56:56 EST


On Tue, 28 Feb 2023, Pavel Machek wrote:

> Hi!
>
> > +config LEDS_AW200XX
> > + tristate "LED support for Awinic AW20036/AW20054/AW20072"
> > + depends on LEDS_CLASS
> > + depends on I2C
> > + help
> > + This option enables support for the AW20036/AW20054/AW20072 LED driver.
> > + It is a 3x12/6x9/6x12 matrix LED driver programmed via
> > + an I2C interface, up to 36/54/72 LEDs or 12/18/24 RGBs,
> > + 3 pattern controllers for auto breathing or group dimming control.
>
> I'm afraid this should be handled as a display, not as an array of
> individual LEDs.

Just for my own information, where do we draw the line on this?

Is 4x4 okay? How about 6x6?

--
Lee Jones [李琼斯]