Re: [PATCH v2 0/3] Qualcomm Light Pulse Generator

From: Bjorn Andersson
Date: Sun Jul 16 2017 - 01:34:57 EST


On Sat 15 Jul 02:10 PDT 2017, Pavel Machek wrote:

> Hi!
>
> > This series introduces a generic pattern interface in the LED class and a
> > driver for the Qualcomm Light Pulse Generator.
> >
> > Bjorn Andersson (3):
> > leds: core: Introduce generic pattern interface
>
> This one should be last. Let me review that, it is important to get
> this one right.
>
> > leds: Add driver for Qualcomm LPG
>
> > DT: leds: Add Qualcomm Light Pulse Generator binding
>
> This one should be first.
>

Okay, no problems.

> And I guess I'd prefer the driver to go in first, before the generic
> pattern interface.
>

The driver won't compile without the additions to the header file. Would
you like the rest of the driver to go in first, then the generic
interface and finally the pattern part of the driver?

Large portions of the driver doesn't make sense without the pattern
part, so I think I would prefer it to go in as one patch.

Please let me know and I'll update the series.

Regards,
Bjorn