Re: [PATCH 6.1] leds: an30259a: Use devm_mutex_init() for mutex initialization

[Greg KH <gregkh@linuxfoundation.org>]

On Fri, Dec 13, 2024 at 11:52:06AM +0800, bin.lan.cn@eng.windriver.com wrote:
> From: George Stark <gnstark@salutedevices.com>
> 
> [ Upstream commit c382e2e3eccb6b7ca8c7aff5092c1668428e7de6 ]
> 
> In this driver LEDs are registered using devm_led_classdev_register()
> so they are automatically unregistered after module's remove() is done.
> led_classdev_unregister() calls module's led_set_brightness() to turn off
> the LEDs and that callback uses mutex which was destroyed already
> in module's remove() so use devm API instead.
> 
> Signed-off-by: George Stark <gnstark@salutedevices.com>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Link: https://lore.kernel.org/r/20240411161032.609544-9-gnstark@salutedevices.com
> Signed-off-by: Lee Jones <lee@kernel.org>
> [ Resolve merge conflict in drivers/leds/leds-an30259a.c ]
> Signed-off-by: Bin Lan <bin.lan.cn@windriver.com>
> ---
>  drivers/leds/leds-an30259a.c | 14 ++++----------
>  1 file changed, 4 insertions(+), 10 deletions(-)


Now deleted, please see:
        https://lore.kernel.org/r/2024121322-conjuror-gap-b542@gregkh
for what you all need to do, TOGETHER, to get this fixed and so that I
can accept patches from your company in the future.

thanks,

greg k-h