On Sun, Apr 15, 2012 at 11:52 AM, Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > Rather than requiring architectures that use gpiolib but don't have any > need to define anything custom to copy an asm/gpio.h provide a Kconfig > symbol which architectures must select in order to include gpio.h and > for other architectures just provide the trivial implementation directly. > > This makes it much easier to do gpiolib updates and is also a step towards > making gpiolib APIs available on every architecture. > > For architectures with existing boilerplate code leave a stub header in > place which warns on direct inclusion of asm/gpio.h and includes > linux/gpio.h to catch code that's doing this. Direct inclusion of > asm/gpio.h has long been deprecated. > > Signed-off-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> > --- > arch/alpha/include/asm/gpio.h | 59 ++---------------------------- > arch/arm/Kconfig | 1 + > arch/avr32/Kconfig | 1 + > arch/blackfin/Kconfig | 1 + > arch/ia64/include/asm/gpio.h | 59 ++---------------------------- > arch/m68k/Kconfig.cpu | 1 + > arch/microblaze/include/asm/gpio.h | 57 ++--------------------------- > arch/mips/Kconfig | 1 + > arch/openrisc/include/asm/gpio.h | 69 ++--------------------------------- > arch/powerpc/include/asm/gpio.h | 57 ++--------------------------- > arch/sh/Kconfig | 1 + > arch/sparc/include/asm/gpio.h | 40 ++------------------- > arch/unicore32/Kconfig | 1 + > arch/x86/include/asm/gpio.h | 57 ++--------------------------- > arch/xtensa/include/asm/gpio.h | 60 ++----------------------------- > drivers/gpio/Kconfig | 8 ++++ > include/linux/gpio.h | 34 +++++++++++++++++ > 17 files changed, 81 insertions(+), 426 deletions(-) This looks good but I think we need to page the alpha, ia64, m68k, microblaze, openrisc etc subarch maintainers on this patch so they have their say. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
