Re: [PATCH] ARM: remove header files included more than once

2011-12-12 Thread Ramirez Luna, Omar 
Hi Haojian,

On Sun, Dec 11, 2011 at 7:10 PM, Haojian Zhuang
haojian.zhu...@gmail.com wrote:
...
 Hi Omar,

 Thanks for your patch. Could you help to split your patches into small
 pieces? Since different people are handling for different files. For
 example, I can help to handle files in arch-pxa. But I can't handle
 other files.

No problem, I'll split the patch to be included by independent trees.

Thanks,

Omar
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: remove header files included more than once

2011-12-11 Thread Haojian Zhuang 
On Sat, Dec 10, 2011 at 12:25 AM, Omar Ramirez Luna omar.rami...@ti.com wrote:
 checkincludes.pl complains about these:
        linux/debugfs.h
        linux/dma-mapping.h
        linux/gpio.h
        linux/sched.h
        linux/slab.h
        plat/common.h
        plat/i2c.h

 Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
 ---
  arch/arm/mach-bcmring/dma.c                 |    1 -
  arch/arm/mach-ks8695/leds.c                 |    1 -
  arch/arm/mach-mmp/aspenite.c                |    1 -
  arch/arm/mach-mmp/tavorevb.c                |    1 -
  arch/arm/mach-msm/board-msm7x30.c           |    1 -
  arch/arm/mach-msm/board-qsd8x50.c           |    1 -
  arch/arm/mach-omap2/board-ldp.c             |    1 -
  arch/arm/mach-omap2/io.c                    |    1 -
  arch/arm/mach-omap2/omap_hwmod_44xx_data.c  |    1 -
  arch/arm/mach-pxa/pxa25x.c                  |    1 -
  arch/arm/mach-pxa/pxa27x.c                  |    1 -
  arch/arm/mach-pxa/saarb.c                   |    1 -
  arch/arm/mach-shmobile/board-ag5evm.c       |    1 -
  arch/arm/mach-ux500/board-mop500-u8500uib.c |    1 -
  arch/arm/plat-omap/clock.c                  |    1 -
  arch/arm/plat-samsung/dev-backlight.c       |    1 -
  16 files changed, 0 insertions(+), 16 deletions(-)


Hi Omar,

Thanks for your patch. Could you help to split your patches into small
pieces? Since different people are handling for different files. For
example, I can help to handle files in arch-pxa. But I can't handle
other files.

Best Regards
Haojian
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] ARM: remove header files included more than once

2011-12-09 Thread Omar Ramirez Luna 
checkincludes.pl complains about these:
linux/debugfs.h
linux/dma-mapping.h
linux/gpio.h
linux/sched.h
linux/slab.h
plat/common.h
plat/i2c.h

Signed-off-by: Omar Ramirez Luna omar.rami...@ti.com
---
 arch/arm/mach-bcmring/dma.c |1 -
 arch/arm/mach-ks8695/leds.c |1 -
 arch/arm/mach-mmp/aspenite.c|1 -
 arch/arm/mach-mmp/tavorevb.c|1 -
 arch/arm/mach-msm/board-msm7x30.c   |1 -
 arch/arm/mach-msm/board-qsd8x50.c   |1 -
 arch/arm/mach-omap2/board-ldp.c |1 -
 arch/arm/mach-omap2/io.c|1 -
 arch/arm/mach-omap2/omap_hwmod_44xx_data.c  |1 -
 arch/arm/mach-pxa/pxa25x.c  |1 -
 arch/arm/mach-pxa/pxa27x.c  |1 -
 arch/arm/mach-pxa/saarb.c   |1 -
 arch/arm/mach-shmobile/board-ag5evm.c   |1 -
 arch/arm/mach-ux500/board-mop500-u8500uib.c |1 -
 arch/arm/plat-omap/clock.c  |1 -
 arch/arm/plat-samsung/dev-backlight.c   |1 -
 16 files changed, 0 insertions(+), 16 deletions(-)

diff --git a/arch/arm/mach-bcmring/dma.c b/arch/arm/mach-bcmring/dma.c
index f4d4d6d..b52b8de 100644
--- a/arch/arm/mach-bcmring/dma.c
+++ b/arch/arm/mach-bcmring/dma.c
@@ -36,7 +36,6 @@
 #include linux/mm.h
 #include linux/pfn.h
 #include linux/atomic.h
-#include linux/sched.h
 #include mach/dma.h
 
 /* I don't quite understand why dc4 fails when this is set to 1 and DMA is 
enabled */
diff --git a/arch/arm/mach-ks8695/leds.c b/arch/arm/mach-ks8695/leds.c
index d6f6502..4bd7075 100644
--- a/arch/arm/mach-ks8695/leds.c
+++ b/arch/arm/mach-ks8695/leds.c
@@ -11,7 +11,6 @@
 #include linux/kernel.h
 #include linux/module.h
 #include linux/init.h
-#include linux/gpio.h
 
 #include asm/leds.h
 #include mach/devices.h
diff --git a/arch/arm/mach-mmp/aspenite.c b/arch/arm/mach-mmp/aspenite.c
index 7a60bbb..2b3b0c6 100644
--- a/arch/arm/mach-mmp/aspenite.c
+++ b/arch/arm/mach-mmp/aspenite.c
@@ -17,7 +17,6 @@
 #include linux/mtd/partitions.h
 #include linux/mtd/nand.h
 #include linux/interrupt.h
-#include linux/gpio.h
 
 #include asm/mach-types.h
 #include asm/mach/arch.h
diff --git a/arch/arm/mach-mmp/tavorevb.c b/arch/arm/mach-mmp/tavorevb.c
index eb5be87..143e52e 100644
--- a/arch/arm/mach-mmp/tavorevb.c
+++ b/arch/arm/mach-mmp/tavorevb.c
@@ -12,7 +12,6 @@
 #include linux/kernel.h
 #include linux/platform_device.h
 #include linux/smc91x.h
-#include linux/gpio.h
 
 #include asm/mach-types.h
 #include asm/mach/arch.h
diff --git a/arch/arm/mach-msm/board-msm7x30.c 
b/arch/arm/mach-msm/board-msm7x30.c
index db81ed5..8a941bb 100644
--- a/arch/arm/mach-msm/board-msm7x30.c
+++ b/arch/arm/mach-msm/board-msm7x30.c
@@ -14,7 +14,6 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
  * 02110-1301, USA.
  */
-#include linux/gpio.h
 #include linux/kernel.h
 #include linux/irq.h
 #include linux/gpio.h
diff --git a/arch/arm/mach-msm/board-qsd8x50.c 
b/arch/arm/mach-msm/board-qsd8x50.c
index 7e8909c..d82a73e 100644
--- a/arch/arm/mach-msm/board-qsd8x50.c
+++ b/arch/arm/mach-msm/board-qsd8x50.c
@@ -14,7 +14,6 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
  * 02110-1301, USA.
  */
-#include linux/gpio.h
 #include linux/kernel.h
 #include linux/irq.h
 #include linux/gpio.h
diff --git a/arch/arm/mach-omap2/board-ldp.c b/arch/arm/mach-omap2/board-ldp.c
index e179da0..eed5c78 100644
--- a/arch/arm/mach-omap2/board-ldp.c
+++ b/arch/arm/mach-omap2/board-ldp.c
@@ -10,7 +10,6 @@
  * it under the terms of the GNU General Public License version 2 as
  * published by the Free Software Foundation.
  */
-#include linux/gpio.h
 #include linux/kernel.h
 #include linux/init.h
 #include linux/platform_device.h
diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c
index 25d20ce..cf25a60 100644
--- a/arch/arm/mach-omap2/io.c
+++ b/arch/arm/mach-omap2/io.c
@@ -43,7 +43,6 @@
 #include clockdomain.h
 #include plat/omap_hwmod.h
 #include plat/multi.h
-#include plat/common.h
 
 /*
  * The machine specific code may provide the extra mapping besides the
diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c 
b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
index daaf165..5f78e90 100644
--- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
+++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c
@@ -22,7 +22,6 @@
 
 #include plat/omap_hwmod.h
 #include plat/cpu.h
-#include plat/i2c.h
 #include plat/gpio.h
 #include plat/dma.h
 #include plat/mcspi.h
diff --git a/arch/arm/mach-pxa/pxa25x.c b/arch/arm/mach-pxa/pxa25x.c
index f05f948..f6ef67f 100644
--- a/arch/arm/mach-pxa/pxa25x.c
+++ b/arch/arm/mach-pxa/pxa25x.c
@@ -24,7 +24,6 @@
 #include linux/suspend.h
 #include linux/syscore_ops.h
 #include linux/irq.h
-#include linux/gpio.h
 
 #include asm/mach/map.h
 #include asm/suspend.h
diff --git a/arch/arm/mach-pxa/pxa27x.c b/arch/arm/mach-pxa/pxa27x.c
index bc5a98e..2dd93e9 100644