Thread (26 messages) flat view 26 messages, 6 authors, 2012-09-04
STALE5087d

[PATCH 02/10] gpio: pxa: add explicit includes for cpu_is_X macros

From: Linus Walleij <hidden>
Date: 2012-08-30 22:17:39

On Wed, Aug 29, 2012 at 3:31 PM, Rob Herring [off-list ref] wrote:
From: Rob Herring <redacted>

In preparation to remove implicit inclusion of mach/gpio.h, add the
necessary includes for cpu_is_X macros.
The commit message talks about gpio.h, but...
 #include <mach/irqs.h>
+#include <mach/hardware.h>
+#ifdef CONFIG_ARCH_MMP
+#include <mach/cputype.h>
+#endif
This seems completely unrelated, but correct in the sense that every driver
should include its resources.

But isn't the proper fix to move arch/arm/mach-pxa/include/mach/cputype.h to
include/linux/*/pxa-cputype.h or something (note: no clue what "*" should be!),
or even better, modify this one driver to take a flag from platform data and/or
DT?

Anyway, we need the PXA maintainers to look at this.

Yours,
Linus Walleij
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help