On Wed, Jan 25, 2023, at 21:10, Andy Shevchenko wrote: > There is a few things done: > - include only the headers we are direct user of > - when pointer is in use, provide a forward declaration > - add missing headers > - group generic headers and subsystem headers > - sort each group alphabetically > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > include/asm-generic/gpio.h | 8 -------- > include/linux/gpio.h | 9 +++------ > include/linux/gpio/consumer.h | 14 ++++++++++---- > include/linux/gpio/driver.h | 34 ++++++++++++++++++++++++---------- > 4 files changed, 37 insertions(+), 28 deletions(-) This change looks fine, but it conflicts with a slightly broader cleanup that I meant to have already submitted, folding include/asm-generic/gpio.h into linux/gpio.h and removing the driver-side interface from that. Let me try to dig out my series again, we should be able to either use my version, or merge parts of this patch into it. Arnd