lkml.org 
[lkml]   [2021]   [Feb]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2 2/4] leds: lp50xx: add setting of default intensity from DT
    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
    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2021-02-05 11:40    [W:3.776 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site