Re: [PATCH v2 2/2] leds: max597x: Add support for max597x

[Naresh Solanki]

Hi Christophe, Pavel,

On 24-03-2023 06:37 pm, Pavel Machek wrote:
> Hi!
>
> > > > +++ b/drivers/leds/Kconfig
> > > > @@ -590,6 +590,17 @@ config LEDS_ADP5520
> > > >          To compile this driver as a module, choose M here: the
> > > > module will
> > > >          be called leds-adp5520.
> > > > +config LEDS_MAX597X
> > > > +    tristate "LED Support for Maxim 597x"
> > > > +    depends on LEDS_CLASS
> > > > +    depends on MFD_MAX597X
> > > > +    help
> > > > +      This option enables support for the Maxim 597x smart switch
> > > > indication LEDs
> > > > +      via the I2C bus.
> > > > +
> > > > +      To compile this driver as a module, choose M here: the module will
> > > > +      be called max597x-led.
> > >
> > > leds-max597x?
> > As per struct max597x_led_driver, driver name is max597x-led
>
> Please test the modular build and double check the module name.
Thank you for your suggestion, it turns out that you were right. The 
correct module name is leds-max597x(./drivers/leds/leds-max597x.ko).
Will update Kconfig message accordingly. Thanks!
> BR,
> 									Pavel
Regards,
Naresh