On Sat, Aug 21, 2021 at 1:00 AM Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > > <linux/spi/max7301.h> despite the placement of the header, is > used by drivers/gpio/gpio-max730*. > > The include needs struct gpio_chip and needs to include > <linux/gpio/driver.h> not the legacy <linux/gpio.h> include. > > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > include/linux/spi/max7301.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/spi/max7301.h b/include/linux/spi/max7301.h > index 433c20e2f46e..21449067aedb 100644 > --- a/include/linux/spi/max7301.h > +++ b/include/linux/spi/max7301.h > @@ -2,7 +2,7 @@ > #ifndef LINUX_SPI_MAX7301_H > #define LINUX_SPI_MAX7301_H > > -#include <linux/gpio.h> > +#include <linux/gpio/driver.h> > > /* > * Some registers must be read back to modify. > -- > 2.31.1 > Applied, thanks! Bartosz