Messages in this thread | | | Date | Tue, 2 Mar 2021 09:11:24 +0100 | From | Pavel Machek <> | Subject | Re: drivers/leds/blink/leds-lgm-sso.c:123:19: error: field has incomplete type 'struct gpio_chip' |
| |
Hi!
> >>>> 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'? > > > > > > It looks like missing #include? > > > > While at it, can you do something with Kconfig help text? Move > > existing one into comment in the driver, add something useful for end-users. > > The .config file does not set/enable GPIOLIB nor OF. > > AFAICT, this driver needs the addition of: > > #include <linux/gpio/consumer.h> > #include <linux/gpio/driver.h> > #include <linux/of.h> > > and the reference to > gc->of_node = dev->of_node; > needs to be inside an > #ifdef CONFIG_GPIO/#endif block. > > Also, <linux/gpio/driver.h> needs a stub for gpiochip_get_data(). > > I have most of that ready (it's in the margin) but I can't > send it tonight. Feel free to fix it up however you want to.
Looking forward. I may be confused, but it looks like original author's email bounces, which is a bad sign.
What I meant... if you take a look at Kconfig help test, it is really useless.
Best regards, Pavel -- http://www.livejournal.com/~pavelmachek [unhandled content-type:application/pgp-signature] | |