From: "Heiko Stübner" <heiko@sntech.de>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Kukjin Kim <kgene.kim@samsung.com>,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, linux-i2c@vger.kernel.org,
Wolfram Sang <wsa@the-dreams.de>, Ben Dooks <ben-linux@fluff.org>
Subject: Re: [PATCH v3 05/10] i2c: s3c2410: make header file local
Date: Thu, 11 Apr 2013 23:43:34 +0200 [thread overview]
Message-ID: <201304112343.34534.heiko@sntech.de> (raw)
In-Reply-To: <1365716245-99231-6-git-send-email-arnd@arndb.de>
Hi Arnd,
Am Donnerstag, 11. April 2013, 23:37:20 schrieb Arnd Bergmann:
> No other file in the kernel besides i2c-s3c2410.c uses the current
> plat/regs-iic.h, so we can simply move the header file to live in the
> same directory as the driver, as a preparation to multiplatform builds.
There is already a patch doing similar changes [0] in the i2c tree for 3.10
Heiko
[0]
https://git.kernel.org/cgit/linux/kernel/git/wsa/linux.git/commit/?h=i2c/for-
next&id=e636602ac2613da8c1777cb42443223994be4107
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Cc: linux-i2c@vger.kernel.org
> Cc: Wolfram Sang <wsa@the-dreams.de>
> Cc: Ben Dooks <ben-linux@fluff.org>
> ---
> arch/arm/mach-s3c24xx/mach-rx1950.c | 1
> - arch/arm/plat-samsung/devs.c |
> 1 - drivers/i2c/busses/i2c-s3c2410.c
> | 3 ++- .../include/plat/regs-iic.h => drivers/i2c/busses/i2c-s3c2410.h
> | 0 4 files changed, 2 insertions(+), 3 deletions(-)
> rename arch/arm/plat-samsung/include/plat/regs-iic.h =>
> drivers/i2c/busses/i2c-s3c2410.h (100%)
>
> diff --git a/arch/arm/mach-s3c24xx/mach-rx1950.c
> b/arch/arm/mach-s3c24xx/mach-rx1950.c index 1f9ba2a..43f3ac5 100644
> --- a/arch/arm/mach-s3c24xx/mach-rx1950.c
> +++ b/arch/arm/mach-s3c24xx/mach-rx1950.c
> @@ -56,7 +56,6 @@
> #include <plat/cpu.h>
> #include <plat/devs.h>
> #include <plat/pm.h>
> -#include <plat/regs-iic.h>
> #include <plat/regs-serial.h>
>
> #include "common.h"
> diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-samsung/devs.c
> index de9ad27..5e37051 100644
> --- a/arch/arm/plat-samsung/devs.c
> +++ b/arch/arm/plat-samsung/devs.c
> @@ -62,7 +62,6 @@
> #include <linux/platform_data/usb-s3c2410_udc.h>
> #include <linux/platform_data/usb-ohci-s3c2410.h>
> #include <plat/usb-phy.h>
> -#include <plat/regs-iic.h>
> #include <plat/regs-serial.h>
> #include <plat/regs-spi.h>
> #include <linux/platform_data/spi-s3c64xx.h>
> diff --git a/drivers/i2c/busses/i2c-s3c2410.c
> b/drivers/i2c/busses/i2c-s3c2410.c index f6b880b..d042ad0 100644
> --- a/drivers/i2c/busses/i2c-s3c2410.c
> +++ b/drivers/i2c/busses/i2c-s3c2410.c
> @@ -42,9 +42,10 @@
>
> #include <asm/irq.h>
>
> -#include <plat/regs-iic.h>
> #include <linux/platform_data/i2c-s3c2410.h>
>
> +#include "i2c-s3c2410.h"
> +
> /* Treat S3C2410 as baseline hardware, anything else is supported via
> quirks */ #define QUIRK_S3C2440 (1 << 0)
> #define QUIRK_HDMIPHY (1 << 1)
> diff --git a/arch/arm/plat-samsung/include/plat/regs-iic.h
> b/drivers/i2c/busses/i2c-s3c2410.h similarity index 100%
> rename from arch/arm/plat-samsung/include/plat/regs-iic.h
> rename to drivers/i2c/busses/i2c-s3c2410.h
WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stübner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 05/10] i2c: s3c2410: make header file local
Date: Thu, 11 Apr 2013 23:43:34 +0200 [thread overview]
Message-ID: <201304112343.34534.heiko@sntech.de> (raw)
In-Reply-To: <1365716245-99231-6-git-send-email-arnd@arndb.de>
Hi Arnd,
Am Donnerstag, 11. April 2013, 23:37:20 schrieb Arnd Bergmann:
> No other file in the kernel besides i2c-s3c2410.c uses the current
> plat/regs-iic.h, so we can simply move the header file to live in the
> same directory as the driver, as a preparation to multiplatform builds.
There is already a patch doing similar changes [0] in the i2c tree for 3.10
Heiko
[0]
https://git.kernel.org/cgit/linux/kernel/git/wsa/linux.git/commit/?h=i2c/for-
next&id=e636602ac2613da8c1777cb42443223994be4107
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Cc: linux-i2c at vger.kernel.org
> Cc: Wolfram Sang <wsa@the-dreams.de>
> Cc: Ben Dooks <ben-linux@fluff.org>
> ---
> arch/arm/mach-s3c24xx/mach-rx1950.c | 1
> - arch/arm/plat-samsung/devs.c |
> 1 - drivers/i2c/busses/i2c-s3c2410.c
> | 3 ++- .../include/plat/regs-iic.h => drivers/i2c/busses/i2c-s3c2410.h
> | 0 4 files changed, 2 insertions(+), 3 deletions(-)
> rename arch/arm/plat-samsung/include/plat/regs-iic.h =>
> drivers/i2c/busses/i2c-s3c2410.h (100%)
>
> diff --git a/arch/arm/mach-s3c24xx/mach-rx1950.c
> b/arch/arm/mach-s3c24xx/mach-rx1950.c index 1f9ba2a..43f3ac5 100644
> --- a/arch/arm/mach-s3c24xx/mach-rx1950.c
> +++ b/arch/arm/mach-s3c24xx/mach-rx1950.c
> @@ -56,7 +56,6 @@
> #include <plat/cpu.h>
> #include <plat/devs.h>
> #include <plat/pm.h>
> -#include <plat/regs-iic.h>
> #include <plat/regs-serial.h>
>
> #include "common.h"
> diff --git a/arch/arm/plat-samsung/devs.c b/arch/arm/plat-samsung/devs.c
> index de9ad27..5e37051 100644
> --- a/arch/arm/plat-samsung/devs.c
> +++ b/arch/arm/plat-samsung/devs.c
> @@ -62,7 +62,6 @@
> #include <linux/platform_data/usb-s3c2410_udc.h>
> #include <linux/platform_data/usb-ohci-s3c2410.h>
> #include <plat/usb-phy.h>
> -#include <plat/regs-iic.h>
> #include <plat/regs-serial.h>
> #include <plat/regs-spi.h>
> #include <linux/platform_data/spi-s3c64xx.h>
> diff --git a/drivers/i2c/busses/i2c-s3c2410.c
> b/drivers/i2c/busses/i2c-s3c2410.c index f6b880b..d042ad0 100644
> --- a/drivers/i2c/busses/i2c-s3c2410.c
> +++ b/drivers/i2c/busses/i2c-s3c2410.c
> @@ -42,9 +42,10 @@
>
> #include <asm/irq.h>
>
> -#include <plat/regs-iic.h>
> #include <linux/platform_data/i2c-s3c2410.h>
>
> +#include "i2c-s3c2410.h"
> +
> /* Treat S3C2410 as baseline hardware, anything else is supported via
> quirks */ #define QUIRK_S3C2440 (1 << 0)
> #define QUIRK_HDMIPHY (1 << 1)
> diff --git a/arch/arm/plat-samsung/include/plat/regs-iic.h
> b/drivers/i2c/busses/i2c-s3c2410.h similarity index 100%
> rename from arch/arm/plat-samsung/include/plat/regs-iic.h
> rename to drivers/i2c/busses/i2c-s3c2410.h
next prev parent reply other threads:[~2013-04-11 21:43 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-11 21:37 [PATCH v3 00/10] ARM: exynos multiplatform series, part 1 Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 01/10] ARM: exynos: introduce EXYNOS_ATAGS symbol Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 02/10] ARM: exynos: prepare for sparse IRQ Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 03/10] ARM: exynos: move debug-macro.S to include/debug/ Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 04/10] ARM: samsung: move mfc device definition to s5p-dev-mfc.c Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
[not found] ` <1365716245-99231-1-git-send-email-arnd-r2nGTMty4D4@public.gmane.org>
2013-04-11 21:37 ` [PATCH v3 05/10] i2c: s3c2410: make header file local Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:43 ` Heiko Stübner [this message]
2013-04-11 21:43 ` Heiko Stübner
2013-04-12 8:11 ` Arnd Bergmann
2013-04-12 8:11 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 06/10] mmc: sdhci-s3c: remove platform dependencies Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 07/10] thermal/exynos: remove unnecessary header inclusions Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 08/10] mtd: onenand/samsung: make regs-onenand.h file local Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 09/10] rtc: s3c: make header " Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-11 21:37 ` [PATCH v3 10/10] ARM: exynos: enable multiplatform support Arnd Bergmann
2013-04-11 21:37 ` Arnd Bergmann
2013-04-19 13:36 ` [PATCH v3 00/10] ARM: exynos multiplatform series, part 1 Arnd Bergmann
2013-04-19 13:36 ` Arnd Bergmann
2013-04-19 13:51 ` Sylwester Nawrocki
2013-04-19 13:51 ` Sylwester Nawrocki
2013-04-22 11:40 ` Sachin Kamat
2013-04-22 11:40 ` Sachin Kamat
2013-04-22 13:12 ` Arnd Bergmann
2013-04-22 13:12 ` Arnd Bergmann
2013-04-22 13:36 ` Thomas Abraham
2013-04-22 13:36 ` Thomas Abraham
2013-04-23 4:02 ` Sachin Kamat
2013-04-23 4:02 ` Sachin Kamat
2013-04-23 9:09 ` Arnd Bergmann
2013-04-23 9:09 ` Arnd Bergmann
2013-04-22 16:44 ` Kukjin Kim
2013-04-22 16:44 ` Kukjin Kim
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=201304112343.34534.heiko@sntech.de \
--to=heiko@sntech.de \
--cc=arnd@arndb.de \
--cc=ben-linux@fluff.org \
--cc=kgene.kim@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=wsa@the-dreams.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.