Heiko Stübner wrote: > > S3C-SoCs starting with the S3C2443 can share a lot of functionality. > The file can collect more common code of these SocS later on and > therefore gets a new name to reflect this future purpose. > > Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx> > --- > for completenes sake, the patch with -M format > > arch/arm/mach-s3c24xx/Kconfig | 14 ++++++++++++-- > arch/arm/mach-s3c24xx/Makefile | 4 ++++ > .../common-s3c2443.c} | 17 +++++++++++++---- > arch/arm/plat-s3c24xx/Kconfig | 6 ------ > arch/arm/plat-s3c24xx/Makefile | 1 - > 5 files changed, 29 insertions(+), 13 deletions(-) > rename arch/arm/{plat-s3c24xx/s3c2443-clock.c => mach-s3c24xx/common- > s3c2443.c} (95%) > > diff --git a/arch/arm/mach-s3c24xx/Kconfig b/arch/arm/mach-s3c24xx/Kconfig > index e537577..506d55b 100644 > --- a/arch/arm/mach-s3c24xx/Kconfig > +++ b/arch/arm/mach-s3c24xx/Kconfig > @@ -41,7 +41,7 @@ config CPU_S3C2416 > select CPU_ARM926T > select CPU_LLSERIAL_S3C2440 > select SAMSUNG_CLKSRC > - select S3C2443_CLOCK > + select S3C2443_COMMON > select S3C2416_DMA if S3C24XX_DMA > select S3C2416_PM if PM > help > @@ -76,7 +76,7 @@ config CPU_S3C2443 > select CPU_ARM920T > select CPU_LLSERIAL_S3C2440 > select SAMSUNG_CLKSRC > - select S3C2443_CLOCK > + select S3C2443_COMMON > select S3C2443_DMA if S3C24XX_DMA > help > Support for the S3C2443 SoC from the S3C24XX line > @@ -470,6 +470,16 @@ config SMDK2440_CPU2442 > > endif # CPU_S3C2440 > > +if CPU_S3C2443 || CPU_S3C2416 > + > +config S3C2443_COMMON > + bool > + help > + Common code for the S3C2443 and similar processors, which > includes > + the S3C2416 and S3C2450. > + > +endif # CPU_S3C2443 || CPU_S3C2416 > + > if CPU_S3C2443 > > config S3C2443_DMA > diff --git a/arch/arm/mach-s3c24xx/Makefile b/arch/arm/mach- > s3c24xx/Makefile > index 08b44a3..876e5e5 100644 > --- a/arch/arm/mach-s3c24xx/Makefile > +++ b/arch/arm/mach-s3c24xx/Makefile > @@ -34,6 +34,10 @@ obj-$(CONFIG_S3C2440_DMA) += dma-s3c2440.o > obj-$(CONFIG_CPU_S3C2443) += s3c2443.o irq-s3c2443.o clock-s3c2443.o > obj-$(CONFIG_S3C2443_DMA) += dma-s3c2443.o > > +# common code > + > +obj-$(CONFIG_S3C2443_COMMON) += common-s3c2443.o > + > # > # machine support > # following is ordered alphabetically by option text. > diff --git a/arch/arm/plat-s3c24xx/s3c2443-clock.c b/arch/arm/mach- > s3c24xx/common-s3c2443.c > similarity index 95% > rename from arch/arm/plat-s3c24xx/s3c2443-clock.c > rename to arch/arm/mach-s3c24xx/common-s3c2443.c > index 3633060..2745616 100644 > --- a/arch/arm/plat-s3c24xx/s3c2443-clock.c > +++ b/arch/arm/mach-s3c24xx/common-s3c2443.c > @@ -1,9 +1,18 @@ > -/* linux/arch/arm/plat-s3c24xx/s3c2443-clock.c > +/* > + * Common code for SoCs starting with the S3C2443 > * > * Copyright (c) 2007, 2010 Simtec Electronics > * Ben Dooks <ben@xxxxxxxxxxxx> > * > - * S3C2443 Clock control suport - common code > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > */ > > #include <linux/init.h> > @@ -552,7 +561,7 @@ void __init_or_cpufreq > s3c2443_common_setup_clocks(pll_fn get_mpll) > s3c24xx_setup_clocks(fclk, hclk, pclk); > > printk("CPU: MPLL %s %ld.%03ld MHz, cpu %ld.%03ld MHz, > mem %ld.%03ld MHz, pclk %ld.%03ld MHz\n", > - (mpllcon & S3C2443_PLLCON_OFF) ? "off":"on", > + (mpllcon & S3C2443_PLLCON_OFF) ? "off" : "on", > print_mhz(pll), print_mhz(fclk), > print_mhz(hclk), print_mhz(pclk)); > > @@ -567,7 +576,7 @@ void __init_or_cpufreq > s3c2443_common_setup_clocks(pll_fn get_mpll) > } > > printk("CPU: EPLL %s %ld.%03ld MHz, usb-bus %ld.%03ld MHz\n", > - (epllcon & S3C2443_PLLCON_OFF) ? "off":"on", > + (epllcon & S3C2443_PLLCON_OFF) ? "off" : "on", > print_mhz(clk_get_rate(&clk_epll)), > print_mhz(clk_get_rate(&clk_usb_bus))); > } > diff --git a/arch/arm/plat-s3c24xx/Kconfig b/arch/arm/plat-s3c24xx/Kconfig > index 0c183fd..74f76e0 100644 > --- a/arch/arm/plat-s3c24xx/Kconfig > +++ b/arch/arm/plat-s3c24xx/Kconfig > @@ -44,12 +44,6 @@ config S3C2410_CLOCK > Clock code for the S3C2410, and similar processors which > is currently includes the S3C2410, S3C2440, S3C2442. > > -config S3C2443_CLOCK > - bool > - help > - Clock code for the S3C2443 and similar processors, which includes > - the S3C2416 and S3C2450. > - > config S3C24XX_DCLK > bool > help > diff --git a/arch/arm/plat-s3c24xx/Makefile b/arch/arm/plat- > s3c24xx/Makefile > index bce27ca..a7e8843 100644 > --- a/arch/arm/plat-s3c24xx/Makefile > +++ b/arch/arm/plat-s3c24xx/Makefile > @@ -28,7 +28,6 @@ obj-$(CONFIG_PM) += pm.o > obj-$(CONFIG_PM) += irq-pm.o > obj-$(CONFIG_PM) += sleep.o > obj-$(CONFIG_S3C2410_CLOCK) += s3c2410-clock.o > -obj-$(CONFIG_S3C2443_CLOCK) += s3c2443-clock.o > obj-$(CONFIG_S3C24XX_DMA) += dma.o > obj-$(CONFIG_S3C2410_IOTIMING) += s3c2410-iotiming.o > obj-$(CONFIG_S3C2412_IOTIMING) += s3c2412-iotiming.o > -- > 1.7.5.4 OK, looks good to me and will apply. Thanks. Best regards, Kgene. -- Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html