It seems most users of <linux/i2c.h> pick up their dependency of <linux/gpio/consumer.h> from other paths but that is not a universal law, so include the header explicitly so we have struct gpio_desc available. Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> --- include/linux/i2c.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/i2c.h b/include/linux/i2c.h index 5d7f3c1853ae..8b968c44ddfa 100644 --- a/include/linux/i2c.h +++ b/include/linux/i2c.h @@ -34,6 +34,7 @@ #include <linux/irqdomain.h> /* for Host Notify IRQ */ #include <linux/of.h> /* for struct device_node */ #include <linux/swab.h> /* for swab16 */ +#include <linux/gpio/consumer.h> #include <uapi/linux/i2c.h> extern struct bus_type i2c_bus_type; -- 2.14.3