From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933779AbbKSEDU (ORCPT ); Wed, 18 Nov 2015 23:03:20 -0500 Received: from mailout2.samsung.com ([203.254.224.25]:41756 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932099AbbKSEDR (ORCPT ); Wed, 18 Nov 2015 23:03:17 -0500 X-AuditID: cbfee68e-f79f56d000004437-2d-564d4a03424d Message-id: <564D4A36.8040803@samsung.com> Date: Thu, 19 Nov 2015 09:34:06 +0530 From: Alim Akhtar User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-version: 1.0 To: Krzysztof Kozlowski , Sylwester Nawrocki , Tomasz Figa , Michael Turquette , Stephen Boyd , linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Catalin Marinas , Will Deacon , Kukjin Kim , Olof Johansson , Arnd Bergmann , linux-arm-kernel@lists.infradead.org, Kevin Hilman Cc: Pankaj Dubey Subject: Re: [2/2] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS References: <1447637775-9887-3-git-send-email-k.kozlowski@samsung.com> In-reply-to: <1447637775-9887-3-git-send-email-k.kozlowski@samsung.com> Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrJIsWRmVeSWpSXmKPExsWyRsSkVpfZyzfMYOlzdYu/k46xW7xf1sNo 8fqFoUX/49fMFl8Pr2C02PT4GqvFx557rBaXd81hs5hxfh+TxcVTrhanrn9ms1i09Qu7xeE3 7awWP850s1is2vWH0eLlxxMsDgIea+atYfT4/WsSo8f7G63sHpf7epk8ds66y+6xaVUnm8ed a3vYPDYvqfe4cqKJ1aNvyypGj8+b5AK4o7hsUlJzMstSi/TtErgyWv8YFxwSrTi2dTJ7A+N1 wS5GDg4JAROJdXczuhg5gUwxiQv31rN1MXJxCAmsYJT4c/skI0TCRKLt5ixmiMQsRomzZ2ez gCSEBB4wSnx47AZi8wpoSXR/7QJrYBFQlfh+8wRYDZuAtsTd6VuYQJaJCkRIPL4gBFEuKPFj 8j0WkJkiAjtZJNY97mcFSTAD1U/e28IOYgsLhEgc6N/ABrHLTeLY9LVMIDangLvEyrl/mSDq rSVWTtrGCGHLS2xe8xbsUAmBAxwSS5YshzpIQOLb5EMsEB/LSmw6wAzxmKTEwRU3WCYwis1C ctMsJGNnIRm7gJF5FaNoakFyQXFSepGRXnFibnFpXrpecn7uJkZgCjj971nfDsabB6wPMQpw MCrx8G445RMmxJpYVlyZe4jRFOiKicxSosn5wESTVxJvaGxmZGFqYmpsZG5ppiTOmyD1M1hI ID2xJDU7NbUgtSi+qDQntfgQIxMHp1QD47z0DqZT9wOmhay51+XaUj2vZxOvy+kG8RfGkkLW c5j8hLtuKepn6i9+IvLu6xmB3zarI/+7eis/mHyd6cUPkSkTiy7tkuPTmdrF9GRGuuWZeONs peXtz9k4Dx7llpGtWHI/TC4zJ3Arp2axurBd1vIj/LeX8a7euKjxkNveP7OO7Ar7KFMvr8RS nJFoqMVcVJwIAAEanzD8AgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrOKsWRmVeSWpSXmKPExsVy+t9jQV1mL98wg3kP5S3+TjrGbvF+WQ+j xesXhhb9j18zW3w9vILRYtPja6wWH3vusVpc3jWHzWLG+X1MFhdPuVqcuv6ZzWLR1i/sFoff tLNa/DjTzWKxatcfRouXH0+wOAh4rJm3htHj969JjB7vb7Sye1zu62Xy2DnrLrvHplWdbB53 ru1h89i8pN7jyokmVo++LasYPT5vkgvgjmpgtMlITUxJLVJIzUvOT8nMS7dV8g6Od443NTMw 1DW0tDBXUshLzE21VXLxCdB1y8wB+kdJoSwxpxQoFJBYXKykb4dpQmiIm64FTGOErm9IEFyP kQEaSFjDmNH6x7jgkGjFsa2T2RsYrwt2MXJySAiYSLTdnMUMYYtJXLi3nq2LkYtDSGAWo8TZ s7NZQBJCAg8YJT48dgOxeQW0JLq/djGC2CwCqhLfb54Aq2ET0Ja4O30LUxcjB4eoQITE4wtC EOWCEj8m32MBmSkisJNFYt3jflaQBDNQ/eS9LewgtrBAiMSB/g1sELvcJI5NX8sEYnMKuEus nPuXCaLeWmLlpG2MELa8xOY1b5knMAJdibBjFpKyWUjKFjAyr2KUSC1ILihOSs81ykst1ytO zC0uzUvXS87P3cQITjTPpHcwHt7lfohRgINRiYd3wymfMCHWxLLiytxDjBIczEoivGWXgEK8 KYmVValF+fFFpTmpxYcYTYGBMJFZSjQ5H5gE80riDY1NzE2NTS1NLEzMLJXEefU9jcKEBNIT S1KzU1MLUotg+pg4OKUaGOc36xSv/dpx1GCT97N5e8tOv2K3aDFSEm7P63ry0v4/S3GHb8C0 bUt/N90M+/r7cbdW8wHJOTp/D/7ZpLT3j4fzkhxZ4S8r0jSunLvHIf7poIyuUvBP6/klT+qj fa7qC4vPnvU6YK1JZdW6uV2+YpNcGeWXLnxTsMb5Rp6x6CbxjW06CTcUnyixFGckGmoxFxUn AgDiaCNSSgMAAA== DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Krzysztof, On 11/16/2015 07:06 AM, Krzysztof Kozlowski wrote: > The ARMv8 Exynos family SoCs in Linux kernel are currently: > - Exynos5433 (controlled by ARCH_EXYNOS), > - Exynos7 (controlled by ARCH_EXYNOS7). > > It duplicates Kconfig symbols unnecessarily, so consolidate them into > one ARCH_EXYNOS. Future SoCs could fall also under the ARCH_EXYNOS > symbol. > > The commit should not bring any visible functional change. > > Signed-off-by: Krzysztof Kozlowski > Reviewed-by: Chanwoo Choi > Reviewed-by: Pankaj Dubey > Tested on exynos7-espresso board, so Tested-by: Alim Akhtar > --- > arch/arm64/Kconfig.platforms | 11 ++--------- > arch/arm64/boot/dts/exynos/Makefile | 2 +- > arch/arm64/configs/defconfig | 2 +- > 3 files changed, 4 insertions(+), 11 deletions(-) > > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms > index 4043c35962cc..afa19baca94e 100644 > --- a/arch/arm64/Kconfig.platforms > +++ b/arch/arm64/Kconfig.platforms > @@ -13,21 +13,14 @@ config ARCH_BERLIN > This enables support for Marvell Berlin SoC Family > > config ARCH_EXYNOS > - bool > - help > - This enables support for Samsung Exynos SoC family > - > -config ARCH_EXYNOS7 > - bool "ARMv8 based Samsung Exynos7" > - select ARCH_EXYNOS > + bool "ARMv8 based Samsung Exynos SoC family" > select COMMON_CLK_SAMSUNG > select HAVE_S3C2410_WATCHDOG if WATCHDOG > select HAVE_S3C_RTC if RTC_CLASS > select PINCTRL > select PINCTRL_EXYNOS > - > help > - This enables support for Samsung Exynos7 SoC family > + This enables support for Samsung Exynos ARMv8 SoC family > > config ARCH_LAYERSCAPE > bool "ARMv8 based Freescale Layerscape SoC family" > diff --git a/arch/arm64/boot/dts/exynos/Makefile b/arch/arm64/boot/dts/exynos/Makefile > index 20310e5b6d6f..50c9b9383cfa 100644 > --- a/arch/arm64/boot/dts/exynos/Makefile > +++ b/arch/arm64/boot/dts/exynos/Makefile > @@ -1,4 +1,4 @@ > -dtb-$(CONFIG_ARCH_EXYNOS7) += exynos7-espresso.dtb > +dtb-$(CONFIG_ARCH_EXYNOS) += exynos7-espresso.dtb > > always := $(dtb-y) > subdir-y := $(dts-dirs) > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig > index bdd7aa358d2a..a396fa7c69a5 100644 > --- a/arch/arm64/configs/defconfig > +++ b/arch/arm64/configs/defconfig > @@ -33,7 +33,7 @@ CONFIG_MODULE_UNLOAD=y > # CONFIG_IOSCHED_DEADLINE is not set > CONFIG_ARCH_BCM_IPROC=y > CONFIG_ARCH_BERLIN=y > -CONFIG_ARCH_EXYNOS7=y > +CONFIG_ARCH_EXYNOS=y > CONFIG_ARCH_LAYERSCAPE=y > CONFIG_ARCH_HISI=y > CONFIG_ARCH_MEDIATEK=y > From mboxrd@z Thu Jan 1 00:00:00 1970 From: alim.akhtar@samsung.com (Alim Akhtar) Date: Thu, 19 Nov 2015 09:34:06 +0530 Subject: [2/2] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS In-Reply-To: <1447637775-9887-3-git-send-email-k.kozlowski@samsung.com> References: <1447637775-9887-3-git-send-email-k.kozlowski@samsung.com> Message-ID: <564D4A36.8040803@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Krzysztof, On 11/16/2015 07:06 AM, Krzysztof Kozlowski wrote: > The ARMv8 Exynos family SoCs in Linux kernel are currently: > - Exynos5433 (controlled by ARCH_EXYNOS), > - Exynos7 (controlled by ARCH_EXYNOS7). > > It duplicates Kconfig symbols unnecessarily, so consolidate them into > one ARCH_EXYNOS. Future SoCs could fall also under the ARCH_EXYNOS > symbol. > > The commit should not bring any visible functional change. > > Signed-off-by: Krzysztof Kozlowski > Reviewed-by: Chanwoo Choi > Reviewed-by: Pankaj Dubey > Tested on exynos7-espresso board, so Tested-by: Alim Akhtar > --- > arch/arm64/Kconfig.platforms | 11 ++--------- > arch/arm64/boot/dts/exynos/Makefile | 2 +- > arch/arm64/configs/defconfig | 2 +- > 3 files changed, 4 insertions(+), 11 deletions(-) > > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms > index 4043c35962cc..afa19baca94e 100644 > --- a/arch/arm64/Kconfig.platforms > +++ b/arch/arm64/Kconfig.platforms > @@ -13,21 +13,14 @@ config ARCH_BERLIN > This enables support for Marvell Berlin SoC Family > > config ARCH_EXYNOS > - bool > - help > - This enables support for Samsung Exynos SoC family > - > -config ARCH_EXYNOS7 > - bool "ARMv8 based Samsung Exynos7" > - select ARCH_EXYNOS > + bool "ARMv8 based Samsung Exynos SoC family" > select COMMON_CLK_SAMSUNG > select HAVE_S3C2410_WATCHDOG if WATCHDOG > select HAVE_S3C_RTC if RTC_CLASS > select PINCTRL > select PINCTRL_EXYNOS > - > help > - This enables support for Samsung Exynos7 SoC family > + This enables support for Samsung Exynos ARMv8 SoC family > > config ARCH_LAYERSCAPE > bool "ARMv8 based Freescale Layerscape SoC family" > diff --git a/arch/arm64/boot/dts/exynos/Makefile b/arch/arm64/boot/dts/exynos/Makefile > index 20310e5b6d6f..50c9b9383cfa 100644 > --- a/arch/arm64/boot/dts/exynos/Makefile > +++ b/arch/arm64/boot/dts/exynos/Makefile > @@ -1,4 +1,4 @@ > -dtb-$(CONFIG_ARCH_EXYNOS7) += exynos7-espresso.dtb > +dtb-$(CONFIG_ARCH_EXYNOS) += exynos7-espresso.dtb > > always := $(dtb-y) > subdir-y := $(dts-dirs) > diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig > index bdd7aa358d2a..a396fa7c69a5 100644 > --- a/arch/arm64/configs/defconfig > +++ b/arch/arm64/configs/defconfig > @@ -33,7 +33,7 @@ CONFIG_MODULE_UNLOAD=y > # CONFIG_IOSCHED_DEADLINE is not set > CONFIG_ARCH_BCM_IPROC=y > CONFIG_ARCH_BERLIN=y > -CONFIG_ARCH_EXYNOS7=y > +CONFIG_ARCH_EXYNOS=y > CONFIG_ARCH_LAYERSCAPE=y > CONFIG_ARCH_HISI=y > CONFIG_ARCH_MEDIATEK=y >