leds: Fix ifdef check for gpio_led_register_device()
authorArnd Bergmann <arnd@arndb.de>
Wed, 28 Feb 2024 09:38:26 +0000 (10:38 +0100)
committerLee Jones <lee@kernel.org>
Thu, 7 Mar 2024 08:48:14 +0000 (08:48 +0000)
commit08b7dab9f025e20dc02bb4ad19b8e519c3948d20
tree944392cc6fb176cf7ef61daf6de7517c6fdd761e
parent6c27bf4c6d3073caff0acc7abb5f6a5332f3319d
leds: Fix ifdef check for gpio_led_register_device()

gpio_led_register_device() is built whenever CONFIG_LEDS_GPIO_REGISTER is
enabled, and this may be used even when CONFIG_NEW_LEDS is turned off.

However, the stub declaration in the header is provided for all configs
without CONFIG_NEW_LEDS, resulting in a build failure:

drivers/leds/leds-gpio-register.c:24:1: error: redefinition of 'gpio_led_register_device'
   24 | gpio_led_register_device(int id, const struct gpio_led_platform_data *pdata)
      | ^
include/linux/leds.h:646:39: note: previous definition is here

Change the #ifdef check to match the definition.
Note: this apparently took years of randconfig builds to hit, since
a number of other drivers just 'select NEW_LEDS' anyway.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20240228093834.2230004-1-arnd@kernel.org
Signed-off-by: Lee Jones <lee@kernel.org>
include/linux/leds.h