leds: lp3952: Use devm API to cleanup module's resources
authorGeorge Stark <gnstark@salutedevices.com>
Thu, 11 Apr 2024 16:10:28 +0000 (19:10 +0300)
committerLee Jones <lee@kernel.org>
Thu, 11 Apr 2024 16:35:03 +0000 (17:35 +0100)
commitb5a0b81605c70b86aa5e8e502613f32b408340ad
tree1547dfc2e1fa2a7571f47f0bfa725ac2d2003ab9
parenta59d8824d7303297718c496a24ac4209eb3c0195
leds: lp3952: Use devm API to cleanup module's resources

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 resources which were destroyed already
in module's remove() so use devm API instead of remove().
Also drop explicit turning LEDs off from remove() due to they will be off
anyway by led_classdev_unregister().

Signed-off-by: George Stark <gnstark@salutedevices.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20240411161032.609544-5-gnstark@salutedevices.com
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/leds/leds-lp3952.c