Re: [ORLinux] [PATCH] gpiolib/arches: Centralise bolierplate asm/gpio.h
From: Jonas Bonn <jonas@southpole.se>
Date: 2012-04-16 07:53:40
Also in:
linux-alpha, linux-arch, lkml, sparclinux
Acked-by: Jonas Bonn <jonas@southpole.se> (for OpenRISC) On Mon, 2012-04-16 at 09:21 +0200, Linus Walleij wrote:
On Sun, Apr 15, 2012 at 11:52 AM, Mark Brown [off-list ref] wrote:quoted
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 <redacted> --- 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 _______________________________________________ Linux mailing list Linux@lists.openrisc.net http://lists.openrisc.net/listinfo/linux