Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752069AbbGMO2T (ORCPT ); Mon, 13 Jul 2015 10:28:19 -0400 Received: from mailout4.samsung.com ([203.254.224.34]:37042 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751961AbbGMO2O (ORCPT ); Mon, 13 Jul 2015 10:28:14 -0400 X-AuditID: cbfee61b-f79416d0000014c0-ce-55a3cafcccf8 From: Bartlomiej Zolnierkiewicz To: Krzysztof Kozlowski 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 Date: Mon, 13 Jul 2015 16:27:51 +0200 Message-id: <2096384.dMD4qgzAC2@amdc1976> User-Agent: KMail/4.13.3 (Linux/3.13.0-55-generic; KDE/4.13.3; x86_64; ; ) In-reply-to: <1549704.AH5CyPJE5q@amdc1976> References: <'@samsung.com> <1549704.AH5CyPJE5q@amdc1976> MIME-version: 1.0 Content-transfer-encoding: 7Bit Content-type: text/plain; charset=us-ascii X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrAIsWRmVeSWpSXmKPExsVy+t9jQd0/pxaHGtzbJ2vRvKnY4vqX56wW Z5cdZLP4/+g1q8Wbt2uYLF6/MLTof/ya2eLr4RWMFm8ebma02PT4GqvF5V1z2Cw+9x5htJhx fh+TxcVTrhaH37SzWpz808to0bGM0WLVrj+MFhu/ejgIe7y/0cruMbvhIovHzll32T02repk 87hzbQ+bx+Yl9R5b+oFCfVtWMXpsvzaP2WPz6WqPz5vkArijuGxSUnMyy1KL9O0SuDLetn5g LpitUTGjqZu5gXGqQhcjJ4eEgInE0gv7WSBsMYkL99azdTFycQgJTGeUWL9tMZTzlVFi3f4D rCBVbAJWEhPbVzGC2CIChhIHd29nAiliFjjAKvHm1CxmkISwQJTEocNn2EFsFgFVifOXlgDZ HBy8ApoSN3skQMKiAl4Sx46tBCvhFNCSWLP2JxPEsiZGiaN/74DN4RUQlPgx+R7YecwC8hL7 9k9lhbC1JNbvPM40gVFgFpKyWUjKZiEpW8DIvIpRNLUguaA4KT3XSK84Mbe4NC9dLzk/dxMj OB6fSe9gXNVgcYhRgINRiYe3YcuiUCHWxLLiytxDjBIczEoivIlHFocK8aYkVlalFuXHF5Xm pBYfYpTmYFES5z2Z7xMqJJCeWJKanZpakFoEk2Xi4JRqYEw47KW6Yt+k6PlVG5cEfKxZwPD4 k5zY1c9CR69IfuP8Z6VUa8meWblj6rwznxL+9O1mSOdrmXmz3GoCv0ZLVeFt57kxfDbWJ1tT L6saqXD27u7X/MbmLcu9a1de7ur3pucnpvzbkRLGcbOy1/e+p4a7EmuQ3w+5rpXXPTIkhE6w /3byaFDkUGIpzkg01GIuKk4EAKFrL/jDAgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5273 Lines: 124 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. ] Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- 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/