Re: [PATCH v5 9/9] leds: Update the lp55xx to use the multi color framework

[Dan Murphy]

On 9/15/19 11:28 AM, Jacek Anaszewski wrote:
> Hi Dan,
>
> On 9/11/19 8:01 PM, Dan Murphy wrote:
> > Update the lp5523 to use the multi color framework.
> >
> > Signed-off-by: Dan Murphy <dmurphy@xxxxxx>
> > ---
> >   drivers/leds/leds-lp5523.c                |  13 +++
> >   drivers/leds/leds-lp55xx-common.c         | 131 ++++++++++++++++++----
> >   drivers/leds/leds-lp55xx-common.h         |   9 ++
> >   include/linux/platform_data/leds-lp55xx.h |   6 +
> >   modules.builtin.modinfo                   | Bin 0 -> 43550 bytes
> >   5 files changed, 137 insertions(+), 22 deletions(-)
> >   create mode 100644 modules.builtin.modinfo
> >
> > diff --git a/drivers/leds/leds-lp5523.c b/drivers/leds/leds-lp5523.c
> > index d0b931a136b9..8b2cdb98fed6 100644
> > --- a/drivers/leds/leds-lp5523.c
> > +++ b/drivers/leds/leds-lp5523.c
> > @@ -791,6 +791,18 @@ static ssize_t store_master_fader_leds(struct device *dev,
> >   	return ret;
> >   }
> [...]
> >   struct lp55xx_predef_pattern {
> Below file seems to have been added to the index by mistake.
I will remove it

Dan


> > diff --git a/modules.builtin.modinfo b/modules.builtin.modinfo
> > new file mode 100644
> > index 0000000000000000000000000000000000000000..e528d8f57796621b6cfef52ad0da44551a482481
> > GIT binary patch
> > literal 43550
> > zcmcJ2?Q+|=(x(2ta}}twKjNIV<j*)6t(~$X+ldo9*0G$-%sF+IMNyQ*NF)`K@<-~u
> > zoV|v<f%nGt=>|YGNXqfd?pCE75g*+RpaC?x8;w!ycl>2BO|zrh@WId3MUofUG7gi1
> > zeg~})3e2M*O!DyPegBfbM`0dh(V~db<mme9q^|~l3~tY^)aliUI=#90dUn&gPqW$g
> > zWw;DG3qOmC$3=KF3Wv*yH%_x4^s>wg7R&B1P3KKT7wLVNdEs~*1raD&TW%EP{%{_8
> [...]