On 15/3/2021 5:44 pm, Rahul Tanwar wrote: > From: Arnd Bergmann<a...@kernel.org> > To: Pavel Machek<pa...@ucw.cz>, > Amireddy Mallikarjuna reddy > <mallikarjunax.re...@linux.intel.com> > Cc: Arnd Bergmann<a...@arndb.de>, Dan Murphy<dmur...@ti.com>, > linux-l...@vger.kernel.org,linux-kernel@vger.kernel.org > Subject:[PATCH] leds: lgm: fix gpiolib dependency > <https://lore.kernel.org/lkml/20210308153052.2353885-1-a...@kernel.org/#r> > Date: Mon, 8 Mar 2021 16:30:46 +0100 > Message-ID:<20210308153052.2353885-1-a...@kernel.org> (raw > <https://lore.kernel.org/lkml/20210308153052.2353885-1-a...@kernel.org/raw>) > > From: Arnd Bergmann<a...@arndb.de> > > Without gpiolib, the driver fails to build: > > drivers/leds/blink/leds-lgm-sso.c:123:19: error: field has incomplete > type 'struct gpio_chip' > struct gpio_chip chip; > ^ > include/linux/gpio.h:107:8: note: forward declaration of 'struct > gpio_chip' > struct gpio_chip; > ^ > drivers/leds/blink/leds-lgm-sso.c:263:3: error: implicit declaration of > function 'gpiod_set_value' [-Werror,-Wimplicit-function-declaration] > gpiod_set_value(led->gpiod, val); > ^ > drivers/leds/blink/leds-lgm-sso.c:263:3: note: did you mean > 'gpio_set_value'? > include/linux/gpio.h:168:20: note: 'gpio_set_value' declared here > static inline void gpio_set_value(unsigned gpio, int value) > ^ > drivers/leds/blink/leds-lgm-sso.c:345:3: error: implicit declaration of > function 'gpiod_set_value' [-Werror,-Wimplicit-function-declaration] > gpiod_set_value(led->gpiod, 1); > ^ > > Add the dependency in Kconfig. > > Fixes: c3987cd2bca3 ("leds: lgm: Add LED controller driver for LGM SoC") > Signed-off-by: Arnd Bergmann<a...@arndb.de> > --- > drivers/leds/blink/Kconfig > <https://lore.kernel.org/lkml/20210308153052.2353885-1-a...@kernel.org/#Z30drivers:leds:blink:Kconfig> > | 1 + > 1 file changed, 1 insertion(+) > > diff > <https://lore.kernel.org/lkml/20210308153052.2353885-1-a...@kernel.org/#iZ30drivers:leds:blink:Kconfig> > > --git a/drivers/leds/blink/Kconfig b/drivers/leds/blink/Kconfig index > 265b53476a80..6dedc58c47b3 100644 --- a/drivers/leds/blink/Kconfig +++ > b/drivers/leds/blink/Kconfig @@ -9,6 +9,7 @@ if LEDS_BLINK > config LEDS_BLINK_LGM > tristate "LED support for Intel LGM SoC series" > + depends on GPIOLIB depends on LEDS_CLASS > depends on MFD_SYSCON > depends on OF > -- > 2.29.2 >
Acked-by: Rahul Tanwar <rtan...@maxlinear.com>
