Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934027AbbGUXpb (ORCPT ); Tue, 21 Jul 2015 19:45:31 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:40703 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933665AbbGUXp3 (ORCPT ); Tue, 21 Jul 2015 19:45:29 -0400 X-AuditID: cbfec7f4-f79c56d0000012ee-b6-55aed997950a Message-id: <55AED996.9090208@samsung.com> Date: Wed, 22 Jul 2015 08:45:26 +0900 From: Krzysztof Kozlowski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.8.0 MIME-version: 1.0 To: Kukjin Kim Cc: linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, =?UTF-8?B?QmFydMWCb21pZWogxbtvxYJuaWVya2lld2ljeg==?= Subject: Re: [GIT PULL] ARM: EXYNOS: mach: Improvements for 4.3 References: <1437014519-19571-1-git-send-email-k.kozlowski@samsung.com> <1437014519-19571-2-git-send-email-k.kozlowski@samsung.com> <06bb01d0c372$756f2250$604d66f0$@kernel.org> In-reply-to: Content-type: text/plain; charset=utf-8 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrHLMWRmVeSWpSXmKPExsVy+t/xa7rTb64LNfg5n8di44z1rBavXxha 9D9+zWyx6fE1VovLu+awWcw4v4/Jgc1j06pONo/NS+o9+rasYvT4vEkugCWKyyYlNSezLLVI 3y6BK6Pn0Qumgv/qFWu+mjYwdip0MXJySAiYSHTvXMQEYYtJXLi3nq2LkYtDSGApo8TDZV0s EM4XRokvG8+xgFTxCmhJ9P//zghiswioSmztPcgKYrMJGEtsXr6EDcQWFYiQWL76JCNEvaDE j8n3gHo5OEQEFCU2L1AAmckssI9R4ua3o+wgNcIC9hLth66wQiw7xCRx7Vk32EmcAsESk/eu ZQJpZhZQl5gyJRckzCwgL7F5zVvmCYwCs5CsmIVQNQtJ1QJG5lWMoqmlyQXFSem5hnrFibnF pXnpesn5uZsYIaH8ZQfj4mNWhxgFOBiVeHgnHF0XKsSaWFZcmXuIUYKDWUmEd+kJoBBvSmJl VWpRfnxRaU5q8SFGaQ4WJXHeubvehwgJpCeWpGanphakFsFkmTg4pRoY/YzfbI+J1Snc3vKH LeWm64V3Aia7lLXfSO+Pa+ZpMPiY+SSaYyaDZ0arv9mdR9f26MZFJPxlvFnJ/Gsuk/RVR73F /624Vx/3Tn6bzGovvWVlcXed9BqRpUdPqM313FAa9DDx+aRE20M/3k77dd5/y764TQx31Iw4 bq1YLawk+infzSvfY/ZzJZbijERDLeai4kQAptbpymECAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5243 Lines: 126 On 21.07.2015 14:21, Krzysztof Kozlowski wrote: > 2015-07-21 14:13 GMT+09:00 Krzysztof Kozlowski : >> 2015-07-21 14:02 GMT+09:00 Kukjin Kim : >>> Krzysztof Kozlowski wrote: >>>> >>>> Dear Kukjin, >>>> >>>> Exynos mach-code related improvements. Description along with a tag. >>>> You can find them also on the lists with my reviewed-by. >>>> >>>> Best regards, >>>> Krzysztof >>>> >>>> >>>> The following changes since commit 1c4c7159ed2468f3ac4ce5a7f08d79663d381a93: >>>> >>>> Merge tag 'ext4_for_linus_stable' of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4 (2015- >>>> 07-05 16:24:54 -0700) >>>> >>>> are available in the git repository at: >>>> >>>> >>>> https://github.com/krzk/linux.git tags/samsung-mach-4.3 >>>> >>>> for you to fetch changes up to 70f83b6716ea0e5944071c12ff1716f93a9c2d8d: >>>> >>>> cpufreq: exynos: remove Exynos5250 specific cpufreq driver support (2015-07-16 10:39:56 +0900) >>>> >>>> ---------------------------------------------------------------- >>>> Improvements for Exynos based boards: >>>> 1. Switch to generic cpufreq-dt driver for Exynos5250. The old driver >>>> is removed. >>>> 2. Fix memory leak in cpufreq error path. >>>> 3. Cleanups: remove duplicated define with bootloader's sleep magic >>>> constant, staticize local function, drop 'owner' from >>>> platform driver, fix cast of iomem to ERR_PTR. >>>> >>>> ---------------------------------------------------------------- >>>> Bartlomiej Zolnierkiewicz (1): >>>> cpufreq: exynos: remove Exynos5250 specific cpufreq driver support >>>> >>>> Krzysztof Kozlowski (4): >>>> ARM: EXYNOS: pmu: Make local function static >>>> ARM: EXYNOS: Remove duplicated define of SLEEP_MAGIC >>>> ARM: EXYNOS: pmu: Drop owner assignment >>>> ARM: EXYNOS: Use IOMEM_ERR_PTR when function returns iomem >>>> >>>> Shailendra Verma (1): >>>> cpufreq: exynos: Fix for memory leak in case SOC name does not match >>>> >>>> Thomas Abraham (3): >>>> clk: samsung: exynos5250: add cpu clock configuration data and instantiate cpu clock >>>> ARM: dts: Exynos5250: add CPU OPP and regulator supply property >>>> ARM: Exynos: switch to using generic cpufreq driver for Exynos5250 >>>> >>>> arch/arm/boot/dts/exynos5250-arndale.dts | 4 + >>>> arch/arm/boot/dts/exynos5250-smdk5250.dts | 4 + >>>> arch/arm/boot/dts/exynos5250-snow.dts | 4 + >>>> arch/arm/boot/dts/exynos5250-spring.dts | 4 + >>>> arch/arm/boot/dts/exynos5250.dtsi | 22 ++++ >>>> arch/arm/mach-exynos/common.h | 6 + >>>> arch/arm/mach-exynos/exynos.c | 1 + >>>> arch/arm/mach-exynos/firmware.c | 2 - >>>> arch/arm/mach-exynos/platsmp.c | 2 +- >>>> arch/arm/mach-exynos/pmu.c | 3 +- >>>> arch/arm/mach-exynos/suspend.c | 4 +- >>>> drivers/clk/samsung/clk-exynos5250.c | 31 +++++ >>>> drivers/cpufreq/Kconfig.arm | 11 -- >>>> drivers/cpufreq/Makefile | 1 - >>>> drivers/cpufreq/exynos-cpufreq.c | 9 +- >>>> drivers/cpufreq/exynos-cpufreq.h | 17 --- >>>> drivers/cpufreq/exynos5250-cpufreq.c | 210 ------------------------------ >>>> include/dt-bindings/clock/exynos5250.h | 1 + >>>> 18 files changed, 84 insertions(+), 252 deletions(-) >>>> delete mode 100644 drivers/cpufreq/exynos5250-cpufreq.c >>>> -- >>> >>> Applied above changes into fixes/non-critical, next/cleanup-samsung, >>> next/dt-samsung and next/cpufreq-clk-samsung according to the change. > > Cc-ed Bartlomiej, > > One issue found: splitting DTS change for Exynos 5250 cpufreq to > separate branch will break bisectability. The commit "ARM: dts: > Exynos5250: add CPU OPP and regulator supply property": > 1. MUST be put after "clk: samsung: exynos5250: add cpu clock > configuration data and instantiate cpu clock" to prevent build > failures, > 2. should be put before "ARM: Exynos: switch to using generic cpufreq > driver for Exynos5250" to prevent lost cpufreq functionality. > > Doing otherwise would lose bisectability which is really unfortunate > when you can simply avoid it. This should remain in the same branch - > next/cpufreq-clk-samsung. Dear Kukjin, Nothing changed since yesterday... so let me point this in a more clear way. The way you divided the patchset causes build errors: -------------- HEAD is now at 565872a601c4... ARM: dts: add CPU OPP and regulator supply property for exynos5250 ERROR: Failed build: -A arm -c exynos on 565872a601c4d91a75b917b1fb40ff27ad7ebb08 LOGS: Error: ../arch/arm/boot/dts/exynos5250.dtsi:65.21-22 syntax error FATAL ERROR: Unable to parse input tree make[2]: *** [arch/arm/boot/dts/exynos5250-arndale.dtb] Error 1 make[1]: *** [dtbs] Error 2 make[1]: *** Waiting for unfinished jobs.... make: *** [sub-make] Error 2 -------------- This breaks bisectability without any special valid reason. Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/