On Fri 2021-02-05 09:59:19, Sven Schuchmann wrote:
> Hello Pavel,
>
> > > diff --git a/drivers/leds/leds-lp50xx.c b/drivers/leds/leds-lp50xx.c
> > > index f13117eed976..4b40bf66483c 100644
> > > --- a/drivers/leds/leds-lp50xx.c
> > > +++ b/drivers/leds/leds-lp50xx.c
> > > @@ -267,7 +267,6 @@ struct lp50xx_led {
> > > struct led_classdev_mc mc_cdev;
> > > struct lp50xx *priv;
> > > unsigned long bank_modules;
> > > - int led_intensity[LP50XX_LEDS_PER_MODULE];
> > > u8 ctrl_bank_enabled;
> > > int led_number;
> > > };
> >
> > ? Does not make sense and changelog does not help.
>
> This is an unused variable which is in the driver
> (same as the regulator). Should I provide a patch on its own for that
> or just describe in the changelog?
Lets do separate patch here. DT changes will need Ack from Rob, this
can go in directly.
Can you or Dan submit patch getting the regulator support to work? If
not, I guess we should remove the regulator support after all.
Best regards,
Pavel
--
http://www.livejournal.com/~pavelmachek
Attachment:
signature.asc
Description: Digital signature
