[PATCH] ARM: EXYNOS: remove CONFIG_MACH_EXYNOS[4,5]_DT config options
Bartlomiej Zolnierkiewicz
b.zolnierkie at samsung.com
Mon Sep 23 06:23:51 EDT 2013
Hi Tomek,
On Tuesday, September 17, 2013 12:45:21 PM Tomasz Figa wrote:
> Hi Bart,
>
> On Friday 13 of September 2013 11:28:31 Bartlomiej Zolnierkiewicz wrote:
> > EXYNOS is now Device Tree (DT) only platform so it makes no sense to have
> > config options responsible for enabling platform specific DT support.
> >
> > Moreover the kernel image won't even link if neither
> > CONFIG_MACH_EXYNOS4_DT nor CONFIG_MACH_EXYNOS5_DT config option is
> > enabled (linker fails with "no machine record defined" error).
> >
> > Remove CONFIG_MACH_EXYNOS[4,5]_DT config options and just use the
> > standard CONFIG_ARCH_EXYNOS[4,5] ones instead.
> >
> > Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie at samsung.com>
> > Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
> > ---
> > arch/arm/configs/exynos_defconfig | 1 -
> > arch/arm/mach-exynos/Kconfig | 38
> > +++++++++----------------------------- arch/arm/mach-exynos/Makefile
> > | 4 ++--
> > 3 files changed, 11 insertions(+), 32 deletions(-)
>
> Generally I see this as a nice cleanup, but I'd go even further than this.
> See below.
>
> > diff --git a/arch/arm/configs/exynos_defconfig
> > b/arch/arm/configs/exynos_defconfig index ad7dfbb..5181d0d 100644
> > --- a/arch/arm/configs/exynos_defconfig
> > +++ b/arch/arm/configs/exynos_defconfig
> > @@ -11,7 +11,6 @@ CONFIG_ARCH_EXYNOS=y
> > CONFIG_S3C_LOWLEVEL_UART_PORT=3
> > CONFIG_S3C24XX_PWM=y
> > CONFIG_ARCH_EXYNOS5=y
> > -CONFIG_MACH_EXYNOS4_DT=y
> > CONFIG_SMP=y
> > CONFIG_NR_CPUS=2
> > CONFIG_PREEMPT=y
> > diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
> > index cb36807..b810f2f 100644
> > --- a/arch/arm/mach-exynos/Kconfig
> > +++ b/arch/arm/mach-exynos/Kconfig
> > @@ -14,19 +14,28 @@ menu "SAMSUNG EXYNOS SoCs Support"
> > config ARCH_EXYNOS4
> > bool "SAMSUNG EXYNOS4"
> > default y
> > + select ARM_AMBA
> > + select CLKSRC_OF
> > + select CLKSRC_SAMSUNG_PWM if CPU_EXYNOS4210
> > + select CPU_EXYNOS4210
>
> The line above shows the brokenness of current layout of Exynos Kconfig
> options. You need to select some SoC in SAMSUNG EXYNOS4, because otherwise
> the build will fail if user doesn't select at least one of Exynos4 SoCs.
>
> Instead, I would hide the ARCH_EXYNOS4 and ARCH_EXYNOS4 options and select
> them from entries of particular SoCs, i.e. ARCH_EXYNOS4 would be selected
> by CPU_EXYNOS4210, SOC_EXYNOS4212 and SOC_EXYNOS4412, ARCH_EXYNOS5 by
> SOC_EXYNOS5250 and SOC_EXYNOS5420 and so on.
This is a good idea but I think that it is better to address it later in
an incremental patch (which I will also do unless someone beats me to it).
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
More information about the linux-arm-kernel
mailing list