Thanks Arnd, How about one liner: depends on USB_HID && I2C && GPIOLIB Reviewed-by: Rishi Gupta <gupt21@xxxxxxxxx> On Wed, Apr 29, 2020 at 3:00 AM Arnd Bergmann <arnd@xxxxxxxx> wrote: > > Without gpiolib, this driver fails to link: > > arm-linux-gnueabi-ld: drivers/hid/hid-mcp2221.o: in function `mcp2221_probe': > hid-mcp2221.c:(.text+0x1b0): undefined reference to `devm_gpiochip_add_data' > arm-linux-gnueabi-ld: drivers/hid/hid-mcp2221.o: in function `mcp_gpio_get': > hid-mcp2221.c:(.text+0x30c): undefined reference to `gpiochip_get_data' > > Fixes: 328de1c519c5 ("HID: mcp2221: add GPIO functionality support") > Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> > --- > drivers/hid/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig > index 008bf44bc2c3..d54e7ae80de5 100644 > --- a/drivers/hid/Kconfig > +++ b/drivers/hid/Kconfig > @@ -1155,6 +1155,7 @@ config HID_ALPS > config HID_MCP2221 > tristate "Microchip MCP2221 HID USB-to-I2C/SMbus host support" > depends on USB_HID && I2C > + depends on GPIOLIB > ---help--- > Provides I2C and SMBUS host adapter functionality over USB-HID > through MCP2221 device. > -- > 2.26.0 >