> > Well, maybe this version ? > > static int idt821034_spi_probe(struct spi_device *spi) > { > ... > > if (IS_ENABLED(CONFIG_GPIOLIB)) > return idt821034_gpio_init(idt821034); > > return 0; > } > Yes that's fine for me. Christophe