Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752501AbbGNACf (ORCPT ); Mon, 13 Jul 2015 20:02:35 -0400 Received: from mailout3.w1.samsung.com ([210.118.77.13]:62958 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751757AbbGNACc (ORCPT ); Mon, 13 Jul 2015 20:02:32 -0400 X-AuditID: cbfec7f4-f79c56d0000012ee-cc-55a45195267c Message-id: <55A45190.6090400@samsung.com> Date: Tue, 14 Jul 2015 09:02:24 +0900 From: Krzysztof Kozlowski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-version: 1.0 To: Bartlomiej Zolnierkiewicz Cc: Michael Turquette , Kukjin Kim , linux-arm-kernel@lists.infradead.org, Lukasz Majewski , Heiko Stuebner , Kevin Hilman , Sachin Kamat , Viresh Kumar , linux-pm@vger.kernel.org, Tomasz Figa , linux-kernel@vger.kernel.org, Chanwoo Choi , Thomas Abraham , Sylwester Nawrocki , Andreas Faerber , Doug Anderson , Javier Martinez Canillas , linux-samsung-soc@vger.kernel.org Subject: Re: [PATCH 4/6] ARM: dts: Exynos4210: add CPU OPP and regulator supply property References: <'@samsung.com> <1549704.AH5CyPJE5q@amdc1976> <2096384.dMD4qgzAC2@amdc1976> In-reply-to: <2096384.dMD4qgzAC2@amdc1976> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrMIsWRmVeSWpSXmKPExsVy+t/xq7pTA5eEGvxrErdo3lRssXHGelaL 61+es1qcXXaQzeL/o9esFm/ermGyeP3C0KL/8Wtmi6+HVzBavHm4mdFi0+NrrBaXd81hs/jc e4TRYsb5fUwWF0+5Whx+085qcfJPL6NFxzJGi1W7/jBabPzq4SDi8f5GK7vH7IaLLB47Z91l 99i0qpPN4861PWwem5fUe2zpBwr1bVnF6LH92jxmj82nqz0+b5IL4I7isklJzcksSy3St0vg ynh37RZzQZdWxaTuLtYGxgnKXYycHBICJhLTXqxgh7DFJC7cW8/WxcjFISSwlFHi5qODjBDO U0aJF18mMoNU8QpoSXR0nGDtYuTgYBFQlWg+7AsSZhMwlti8fAkbiC0qECHx9vJJJohyQYkf k++xgNgiAhYSa1e8ZQGZySxwgFXizalZYDOFBaIkDh0+ww6xbAGjxJyHz8EmcQIt2zFzDhPI MmYBPYn7F7VAwswC8hKb17xlnsAoMAvJjlkIVbOQVC1gZF7FKJpamlxQnJSea6hXnJhbXJqX rpecn7uJERKVX3YwLj5mdYhRgINRiYf3hfWSUCHWxLLiytxDjBIczEoivOw+QCHelMTKqtSi /Pii0pzU4kOM0hwsSuK8c3e9DxESSE8sSc1OTS1ILYLJMnFwSjUwcnVZTDvu/0JHNDRpa316 N8/29fNvTfcuYUp3nysnH9/S+HOXz6ulvF6pP7TmC9xUVZjePn/x1zkGj1YY/Jw513be+k6F C6WVPC+s0mJNu9v9bvz7PC3q1K3ZVw5umnKqdbL65Ek3r17q+Lspof0ho+b0g2I2rzq2qqkt f72//37TluuHJfZfOa3EUpyRaKjFXFScCACI7CcQxgIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5504 Lines: 132 On 13.07.2015 23:27, Bartlomiej Zolnierkiewicz wrote: > On Monday, July 13, 2015 01:20:41 PM Bartlomiej Zolnierkiewicz wrote: >> On Monday, July 13, 2015 08:10:21 PM Krzysztof Kozlowski wrote: >>> 2015-07-13 20:02 GMT+09:00 Bartlomiej Zolnierkiewicz : >>>> >>>> Hi, >>>> >>>> On Tuesday, June 23, 2015 09:24:40 AM Krzysztof Kozlowski wrote: >>>>> On 23.06.2015 08:46, Krzysztof Kozlowski wrote: >>>>>> On 23.06.2015 00:04, Michael Turquette wrote: >>>>>>> Quoting Kukjin Kim (2015-06-21 18:46:26) >>>>>>>> Krzysztof Kozlowski wrote: >>>>>>>>> >>>>>>>>> On 22.06.2015 10:38, Kukjin Kim wrote: >>>>>>>>>> Krzysztof Kozlowski wrote: >>>>>>>>>>> 2015-05-08 9:18 GMT+09:00 Krzysztof Kozlowski : >>>>>>>>>>>> 2015-04-04 1:43 GMT+09:00 Bartlomiej Zolnierkiewicz : >>>>>>>>>>>>> From: Thomas Abraham >>>>>>>>>>>>> >>>>>>>>>>>>> For Exynos4210 platforms, add CPU operating points and CPU >>>>>>>>>>>>> regulator supply properties for migrating from Exynos specific >>>>>>>>>>>>> cpufreq driver to using generic cpufreq driver. >>>>>>>>>>>>> >>>>>>>>>>>>> Changes by Bartlomiej: >>>>>>>>>>>>> - removed Exynos5250 and Exynos5420 support for now >>>>>>>>>>>>> >>>>>>>>>>>>> Cc: Kukjin Kim >>>>>>>>>>>>> Cc: Doug Anderson >>>>>>>>>>>>> Cc: Javier Martinez Canillas >>>>>>>>>>>>> Cc: Andreas Faerber >>>>>>>>>>>>> Cc: Sachin Kamat >>>>>>>>>>>>> Cc: Andreas Farber >>>>>>>>>>>>> Cc: Javier Martinez Canillas >>>>>>>>>>>>> Signed-off-by: Thomas Abraham >>>>>>>>>>>>> Signed-off-by: Bartlomiej Zolnierkiewicz >>>>>>>>>>>> >>>>>>>>>>>> Acked-by: Krzysztof Kozlowski >>>>>>>>>>> >>>>>>>>>>> Rebased and applied to my tree, I'll sent it later to Kukjin unless he >>>>>>>>>>> picks it by himself from LKML. >>>>>>>>>>> >>>>>>>>>> Hi, as far as I know, this is for v4.2 not v4.1 so it will be applied based on >>>>>>>>>> v4.2-rc1 after v4.2-rc1 release. >>>>>>>>> >>>>>>>>> You mean it is for v4.3, not v4.2? >>>>>>>>> >>>>>>>> Oops, yes v4.3. >>>>>>>> >>>>>>>> Thanks for the correction. >>>>>>> >>>>>>> Kukjin & Krzysztof, >>>>>>> >>>>>>> I'm confused on this point. I was planning to take patches 1, 2, 3, 5 >>>>>>> and 6 towards 4.2 (e.g. in the pull request that I'll send out this >>>>>>> week). >>>>>>> >>>>>>> Is patch 4 going out for 4.2 or 4.3? >>>>>> >>>>>> It is quite late for sending pull request to arm-soc for 4.2. >>>>>> For example SoCFPGA pull request from last week was rejected: >>>>>> http://comments.gmane.org/gmane.linux.ports.arm.kernel/417980 >>>>> >>>>> Oh, that was wrong link. Here it is: >>>>> http://www.spinics.net/lists/arm-kernel/msg425911.html >>>>> >>>>>> If you want to take it for 4.2 then I am fine with it but this will >>>>>> cause some easy but annoying conflicts. There aren't difficult - just >>>>>> most of nodes in board DTS changed their place. >>>>>> >>>>>> Example of resolution (target file after merge, with cpu nodes reordered >>>>>> alphabetically): >>>>>> https://github.com/krzk/linux/commit/2cec3cb48abaf44848c62f1c0836b772eb4680dd >>>> >>>> This patch is needed for v4.2 as other changes has been already >>>> merged. >>>> >>>> Krzysztof/Kukjin, could you please take care of it? >>> >>> Of course! It is already in my queue. I'll send it later to Kukjin for >>> 4.3 (unless he picks it also). >> >> It is in your queue for v4.3 but the patch is needed for v4.2, >> without it cpufreq support will not work for Exynos4210 platforms. >> >>> BTW for other patchsets you still need acks from Samsung clock >>> maintainers. Did you poke Sylwester or Tomasz about it? >> >> Sylwester, please review/ack Samsung specific clock changes in >> Exynos5250 cpufreq and Exynos4x12 cpufreq patch series. >> >> Patch series: >> >> * [PATCH v3 0/4] cpufreq: use generic cpufreq drivers for Exynos5250 platform >> (http://lkml.org/lkml/2015/7/1/311) >> >> * [PATCH v2 0/7] cpufreq: use generic cpufreq drivers for Exynos4x12 platform >> (http://lkml.org/lkml/2015/7/9/419) >> >> Samsung clock specific changes: >> >> * [PATCH v3 1/4] clk: samsung: exynos5250: add cpu clock configuration data >> and instantiate cpu clock >> (http://lkml.org/lkml/2015/7/1/313) >> >> * [PATCH v2 4/7] clk: samsung: exynos4x12: add cpu clock configuration data >> and instantiate cpu clock >> (http://lkml.org/lkml/2015/7/9/424) > > BTW corresponding changes for Exynos4210 has already been merged and 2 above > patches are quite obvious so review should be very quick & simple. > > [ Unfortunately Sylwester is currently very busy with a more priority work so > it may still take some time to get his ACKs. ] The review would be fast and already Javier or me provided such. Anyway if you want to merge it through different tree it would be expected to get an ack from bypassed maintainer. Especially that the distance between you and the bypassed maintainer was ~1 meter for the most of the time :) . It is much longer distance between you and me now! 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/