On Fri, Dec 8, 2017 at 11:35 AM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > <linux/of_gpio.h> is not used in this file, by > <linux/gpio/consumer.h> is. s/by/but > > Someone is just lucky with their implicit includes. > > Cc: Shawn Guo <shawnguo@xxxxxxxxxx> > Cc: Sascha Hauer <kernel@xxxxxxxxxxxxxx> > Cc: Fabio Estevam <fabio.estevam@xxxxxxx> > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Other than this nit: Reviewed-by: Fabio Estevam <fabio.estevam@xxxxxxx>