[RFC PATCH 15/24] gpio: include mach/irqs.h directly
nicolas.pitre at linaro.org
Wed Aug 24 17:48:48 UTC 2011
On Wed, 24 Aug 2011, Venkatraman S wrote:
> On 23 August 2011 23:56, Nicolas Pitre <nicolas.pitre at linaro.org> wrote:
> > On Tue, 23 Aug 2011, Venkatraman S wrote:
> >> From: Venkatraman S <venkat at linaro.org>
> >> Signed-off-by: Venkatraman S <venkat at linaro.org>
> >> ---
> >> drivers/gpio/gpio-exynos4.c | 2 +-
> >> drivers/gpio/gpio-nomadik.c | 2 +-
> >> drivers/gpio/gpio-s5pv210.c | 1 +
> >> drivers/gpio/pl061.c | 1 +
> > Same issue as for patch #14/24. One of the big issue with the GPIO code
> > is its reliance on <mach/gpio.h> which has to be solved already, since
> > in a multi SOC kernel we just don't know which of the many <mach/irqs.h>
> > we want to include.
> > I think that one of the solution for this would involve making
> > <mach/*.h> into <mach/$soc_name/*.h> up front... meaning that we recurse
> > again another level down with all its implications.
> Thanks for the review. Can't we just rename the files then ?
> gpio-exynos.h gpio-nomadik.h and so on, and move all, or part of the
> contents to drivers ?
Something along those lines was being looked at by Linus Walleij and
More information about the linaro-dev