From: Kukjin Kim <kgene.kim@samsung.com>
To: 'Omar Ramirez Luna' <omar.ramirez@ti.com>,
'Russell King' <linux@arm.linux.org.uk>
Cc: 'Jiandong Zheng' <jdzheng@broadcom.com>,
'Scott Branden' <sbranden@broadcom.com>,
'David Brown' <davidb@codeaurora.org>,
'Daniel Walker' <dwalker@fifo99.com>,
'Bryan Huntsman' <bryanh@codeaurora.org>,
'Tony Lindgren' <tony@atomide.com>,
'Benoit Cousson' <b-cousson@ti.com>,
'Eric Miao' <eric.y.miao@gmail.com>,
'Paul Mundt' <lethal@linux-sh.org>,
'Magnus Damm' <magnus.damm@gmail.com>,
'Srinidhi Kasagar' <srinidhi.kasagar@stericsson.com>,
'Linus Walleij' <linus.walleij@stericsson.com>,
'Paul Walmsley' <paul@pwsan.com>,
'Ben Dooks' <ben-linux@fluff.org>,
'Simtec Linux Team' <linux@simtec.co.uk>,
'Daniel Silverstone' <dsilvers@simtec.co.uk>,
'Stephen Boyd' <sboyd@codeaurora.org>,
'Krzysztof Halasa' <khc@pm.waw.pl>,
'Lei Wen' <leiwen@marvell.com>,
linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org
Subject: RE: [PATCH] ARM: remove header files included more than once
Date: Wed, 21 Dec 2011 13:10:23 +0900 [thread overview]
Message-ID: <061b01ccbf96$763f95e0$62bec1a0$%kim@samsung.com> (raw)
In-Reply-To: <1323447943-23595-1-git-send-email-omar.ramirez@ti.com>
Omar Ramirez Luna 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.ramirez@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(-)
>
(snip)
> diff --git a/arch/arm/plat-samsung/dev-backlight.c
b/arch/arm/plat-samsung/dev-backlight.c
> index e657305..a976c02 100644
> --- a/arch/arm/plat-samsung/dev-backlight.c
> +++ b/arch/arm/plat-samsung/dev-backlight.c
> @@ -15,7 +15,6 @@
> #include <linux/slab.h>
> #include <linux/io.h>
> #include <linux/pwm_backlight.h>
> -#include <linux/slab.h>
>
> #include <plat/devs.h>
> #include <plat/gpio-cfg.h>
> --
> 1.7.4.1
Hi,
Above has been already fixed in mainline.
:commit d7a4c45ec24b6c6773ff302c3c9cb5ed46c8e211 ("ARM: SAMSUNG: remove
duplicated header include").
Anyway, thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
prev parent reply other threads:[~2011-12-21 4:10 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-09 16:25 [PATCH] ARM: remove header files included more than once Omar Ramirez Luna
2011-12-12 1:10 ` Haojian Zhuang
2011-12-12 20:25 ` Ramirez Luna, Omar
2011-12-21 4:10 ` Kukjin Kim [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='061b01ccbf96$763f95e0$62bec1a0$%kim@samsung.com' \
--to=kgene.kim@samsung.com \
--cc=b-cousson@ti.com \
--cc=ben-linux@fluff.org \
--cc=bryanh@codeaurora.org \
--cc=davidb@codeaurora.org \
--cc=dsilvers@simtec.co.uk \
--cc=dwalker@fifo99.com \
--cc=eric.y.miao@gmail.com \
--cc=jdzheng@broadcom.com \
--cc=khc@pm.waw.pl \
--cc=leiwen@marvell.com \
--cc=lethal@linux-sh.org \
--cc=linus.walleij@stericsson.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=linux@simtec.co.uk \
--cc=magnus.damm@gmail.com \
--cc=omar.ramirez@ti.com \
--cc=paul@pwsan.com \
--cc=sboyd@codeaurora.org \
--cc=sbranden@broadcom.com \
--cc=srinidhi.kasagar@stericsson.com \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).