From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754158AbcA0JLL (ORCPT ); Wed, 27 Jan 2016 04:11:11 -0500 Received: from mailout2.w1.samsung.com ([210.118.77.12]:38012 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753192AbcA0JK6 (ORCPT ); Wed, 27 Jan 2016 04:10:58 -0500 X-AuditID: cbfec7f5-f79b16d000005389-e1-56a8899ee49b Subject: Re: [PATCH v3 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol To: Kukjin Kim , Kyungmin Park , Catalin Marinas , Will Deacon , Michael Turquette , Stephen Boyd , Sylwester Nawrocki , Tomasz Figa , Olof Johansson , Arnd Bergmann , Kevin Hilman , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-clk@vger.kernel.org References: <1453795504-12302-1-git-send-email-k.kozlowski@samsung.com> From: Krzysztof Kozlowski Message-id: <56A88998.5040602@samsung.com> Date: Wed, 27 Jan 2016 18:10:48 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-version: 1.0 In-reply-to: <1453795504-12302-1-git-send-email-k.kozlowski@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprKIsWRmVeSWpSXmKPExsVy+t/xa7rzOleEGZz/K2Pxd9Ixdov3y3oY LeYfOcdq8fqFoUX/49fMFk83P2ayONv0ht1i0+NrrBYfe+6xWlzeNYfNYsb5fUwWF0+5Wpy6 /pnN4vCbdlaLH2e6WSxW7frDaPHy4wkWB0GPNfPWMHr8/jWJ0eP9jVZ2j8t9vUweO2fdZffY tKqTzWPzknqPKyeaWD36tqxi9Pi8SS6AK4rLJiU1J7MstUjfLoEro+9/F2PBNOGK58fXMTUw PufrYuTkkBAwkdhz8DIThC0mceHeerYuRi4OIYGljBLzt+xgBkkICTxllNh7xB7EFhbwlOjf /IcJpEhE4CKLxO/1a6CK3CUeH7nAAmKzCRhLbF6+hA3E5hXQktjeOIWxi5GDg0VAVWLtZyOQ sKhAhMThzi52iBJBiR+T74G1cgp4SKz+c58VpJxZQE/i/kUtkDCzgLzE5jVvmScw8s9C0jEL oWoWkqoFjMyrGEVTS5MLipPSc430ihNzi0vz0vWS83M3MUIi6+sOxqXHrA4xCnAwKvHw3ti/ PEyINbGsuDL3EKMEB7OSCK945IowId6UxMqq1KL8+KLSnNTiQ4zSHCxK4rwzd70PERJITyxJ zU5NLUgtgskycXBKNTCufr+Sc9L+In2+nqip631PPX6qNP3xdH9Grqms617cd1qpyVEQEKm5 POvqAdYU5scn3Xa+L9Ez27ovbKrUuw2zUq3n/ujMjmBSdFv8rmurN2flrFk3ykJ//NDJv308 Qv1P7PUo8QUdiwJW3e+JTvy39yu/V5v8YZaDlYJxfO8qZpZJ9PQ46r9VYinOSDTUYi4qTgQA evy8eqgCAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 26.01.2016 17:05, Krzysztof Kozlowski wrote: > Hi, > > Third attempt to remove ARCH_EXYNOS7 symbol. > > Patches cannot be cherry-picked. I was thinking about merging this > through arm-soc tree after getting needed acks (clk, arm-soc?). The patchset was rebased on quite old linux-next. Rebasing on current code makes only one difference - the s3c64xx gained multiplatform support. I don't want to spam the lists so, I'll respin everything tomorrow after proper rebasing. Best regards, Krzysztof > > Changes since v2: > ================= > 1. Follow Arnd's advice about not enabling silently the newly added > EXYNOS_ARM64_COMMON_CLK, when COMPILE_TEST is enabled. This was > also a trigger for some changes in placement of Samsung clocks > Kconfig entries... > 2. ... which lead to introducing patch 1. > > > Logic: > ====== > Now the clock selection algorithm works like: > 1. For Exynos ARMv7 and ARMv8 the platforms select COMMON_CLK > and COMMON_CLK_SAMSUNG. > 2. As before: clock drivers for Exynos ARMv7 will be selected by > respective SOC/ARCH_EXYNOS_XXX symbol. > 3. For Exynos ARMv8 the clock drivers will be enabled by default > by EXYNOS_ARM64_COMMON_CLK symbol. > > Rationale: > ========== > We don't need ARCH_EXYNOS7 symbol because all ARMv8 platforms should > fall under generic ARCH_EXYNOS. In the same time there is no sense > in building clocks for these ARMv8 SoCs on ARMv7 builds. In future > this will also influence PMU [1] driver. > > > Best regards, > Krzysztof > > [1] https://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg48090.html > > > Krzysztof Kozlowski (3): > clk: samsung: Enable COMPILE_TEST for Samsung clocks > clk: samsung: Don't build ARMv8 clock drivers on ARMv7 > arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS > > arch/arm/Kconfig | 1 + > arch/arm/mach-exynos/Kconfig | 1 + > arch/arm/mach-s3c24xx/Kconfig | 8 ++++++++ > arch/arm/mach-s5pv210/Kconfig | 1 + > arch/arm64/Kconfig.platforms | 12 +++--------- > arch/arm64/boot/dts/exynos/Makefile | 2 +- > arch/arm64/configs/defconfig | 2 +- > drivers/clk/Kconfig | 2 +- > drivers/clk/samsung/Kconfig | 14 +++++++++----- > drivers/clk/samsung/Makefile | 4 ++-- > 10 files changed, 28 insertions(+), 19 deletions(-) >