From patchwork Tue Sep 23 13:35:14 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bartlomiej Zolnierkiewicz X-Patchwork-Id: 4956471 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 707A9BEEA6 for ; Tue, 23 Sep 2014 13:38:39 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 4775420253 for ; Tue, 23 Sep 2014 13:38:31 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 594E82025B for ; Tue, 23 Sep 2014 13:38:28 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1XWQFs-0008Lw-ME; Tue, 23 Sep 2014 13:35:44 +0000 Received: from mailout2.samsung.com ([203.254.224.25]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1XWQFq-0008Ff-Dz for linux-arm-kernel@lists.infradead.org; Tue, 23 Sep 2014 13:35:43 +0000 Received: from epcpsbgm1.samsung.com (epcpsbgm1 [203.254.230.26]) by mailout2.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0NCC00APBWEUQJD0@mailout2.samsung.com> for linux-arm-kernel@lists.infradead.org; Tue, 23 Sep 2014 22:35:18 +0900 (KST) X-AuditID: cbfee61a-f79c06d000004e71-05-54217715b2d6 Received: from epmmp2 ( [203.254.227.17]) by epcpsbgm1.samsung.com (EPCPMTA) with SMTP id 98.59.20081.51771245; Tue, 23 Sep 2014 22:35:18 +0900 (KST) Received: from amdc1032.localnet ([106.116.147.136]) by mmp2.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTPA id <0NCC00DL9WESSLN0@mmp2.samsung.com>; Tue, 23 Sep 2014 22:35:17 +0900 (KST) Sun-Java-System-SMTP-Warning: Lines longer than SMTP allows found and wrapped. From: Bartlomiej Zolnierkiewicz To: Lorenzo Pieralisi Subject: Re: [PATCH v8 6/8] drivers: cpuidle: CPU idle ARM64 driver Date: Tue, 23 Sep 2014 15:35:14 +0200 Message-id: <4228539.nCGoXxuRMK@amdc1032> User-Agent: KMail/4.8.4 (Linux/3.2.0-54-generic-pae; KDE/4.8.5; i686; ; ) In-reply-to: <20140912112643.GA10623@red-moon> References: <1409585324-3678-7-git-send-email-lorenzo.pieralisi@arm.com> <20140912103257.GB24416@localhost> <20140912112643.GA10623@red-moon> MIME-version: 1.0 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrKIsWRmVeSWpSXmKPExsVy+t9jQV2xcsUQg1WT2S2aGxYzWXxeeJrV 4sKJf8wWUx8+YbN4v6yH0WJjz2d2i3mfZS3mHznHarHkpZ3FgT87GC1WbVjCYvH18ApGiwmv l7NbbHp8jdXic+8RRos3v1+wWyy9fpHJYvqxv2wWH3b8ZbaYtHYqo0Xr3iPsFq/71jBb/DjT zWJxvPE8u8XyUztYLFbt+sNo0XHkG7PFy48nWBxkPNbMW8Pocbmvl8lj56y77B6Hvy5k8di8 Qstj06pONo871/aweWxeUu/R2/yOzePG7152j74tqxg9jt/YzuTxeZNcAG8Ul01Kak5mWWqR vl0CV8aNHSdYCxapVMzet5GpgXGiTBcjJ4eEgIlEw50lLBC2mMSFe+vZuhi5OIQEpjNK3Nnx khXCaWGSWD53LTNIlbCAn0TTnElgNpuAlcTE9lWMILaIgKFEz6nVTCANzAKLOCWOLt/PAtHg LHF7+mlWEJtFQFViz8TVYHFeAU2JNS0TweKiAp4SO7avZAOxOQX0JF79+QG1uZ9R4t3Ot1AN ghI/Jt8Ds5kF5CX27Z8KVMQBZOtJLN6ZNoFRcBaSqllIqmYhVC1gZF7FKJpakFxQnJSea6hX nJhbXJqXrpecn7uJEZwYnkntYFzZYHGIUYCDUYmH13ONQogQa2JZcWXuIUYJDmYlEV6xDMUQ Id6UxMqq1KL8+KLSnNTiQ4zSHCxK4rwHWq0DhQTSE0tSs1NTC1KLYLJMHJxSDYx6KyZbaz1M lYrzaN75NfLJsTZ50x4/y7VcClcWi05at6Ku/uhLWeHtVavjbquui1vWEvBAL788K0t6QlmE SaN05QnWFpW2xm87XGZlR2pNn/fK22btZ0X/5pXKXlG3Q10XxE80teXInu0v/9Hn18EdHNc1 X60+KX7qpp7ie2t+tdOu+h3/JiuxFGckGmoxFxUnAgBdfyxKCAMAAA== X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20140923_063542_619049_35345CBB X-CRM114-Status: GOOD ( 27.30 ) X-Spam-Score: -5.7 (-----) Cc: Mark Rutland , Catalin Marinas , Will Deacon , Tomasz Figa , Rob Herring , Lina Iyer , Chander Kashyap , Vincent Guittot , Nicolas Pitre , Daniel Lezcano , "linux-arm-kernel@lists.infradead.org" , "grant.likely@linaro.org" , Charles Garcia-Tobin , "devicetree@vger.kernel.org" , Kevin Hilman , "linux-pm@vger.kernel.org" , Sebastian Capella , Mark Brown , Antti Miettinen , Paul Walmsley , Geoff Levand , Peter De Schrijver , Stephen Boyd , Amit Kucheria , Ashwin Chaugule , Santosh Shilimkar , Sudeep Holla X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_NONE, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Hi, On Friday, September 12, 2014 12:26:43 PM Lorenzo Pieralisi wrote: > On Fri, Sep 12, 2014 at 11:32:57AM +0100, Catalin Marinas wrote: > > Daniel, Lorenzo, > > > > On Thu, Sep 11, 2014 at 10:32:48AM +0100, Daniel Lezcano wrote: > > > On 09/11/2014 10:57 AM, Lorenzo Pieralisi wrote: > > > > There is no ARM code in my series. So to sum it up: > > > > > > > > a) I send a pull request to Catalin for arm64 patches on top of the branch > > > > you are creating with my patches > > > > > > > > b) You take care of merging the CPUidle related patches through your > > > > tree > > > > > > > > Is the above what you meant ? > > > > > > Right, that allows to share a branch across the trees and resolve the > > > dependencies when a patchset is touching different subsystems. > > > > > > I realize the dependency is inverted regarding what I proposed > > > initially, so it is up to Catalin to create the branch and I will share > > > it with him. > > > > I created the branch below with the arm64 specific patches for cpuidle. > > I won't rebase it and it should appear in -next tomorrow. > > > > Lorenzo, please send a pull request to Daniel with the additional > > patches on top of the arm64 branch. Thanks. > > Thank you very much, done. Pull request attached for information. > > As Daniel knows, last two patches are still pending, waiting for Samsung > acks, and will be dropped if we do not get them in time. > > Lorenzo > > The following changes since commit 18910ab0d916b1a87016d69efd027714a80521dd: > > arm64: add PSCI CPU_SUSPEND based cpu_suspend support (2014-09-12 10:48:56 +0100) > > are available in the git repository at: > > git://linux-arm.org/linux-2.6-lp.git for-upstream/arm-generic-cpuidle > > for you to fetch changes up to dda773655f9b97570884c4396ad4d63daf177107: > > drivers: cpuidle: initialize Exynos driver through DT (2014-09-12 11:37:43 +0100) > > ---------------------------------------------------------------- > Bartlomiej Zolnierkiewicz (1): > ARM: dts: add CPU nodes for Exynos4 SoCs This patch got broken during merge. It differs from the version I posted on the list by not having the closing "};" which causes a dtb build failure: Error: arch/arm/boot/dts/exynos4210-origen.dts:21.1-2 syntax error FATAL ERROR: Unable to parse input tree make[1]: *** [arch/arm/boot/dts/exynos4210-origen.dtb] Error 1 make: *** [dtbs] Error 2 make: *** Waiting for unfinished jobs.... Please regenarate your branch with the fixed version of the patch (you can squash the fix below to the broken patch). --- arch/arm/boot/dts/exynos4210.dtsi | 1 + 1 file changed, 1 insertion(+) > Lorenzo Pieralisi (4): > drivers: cpuidle: implement DT based idle states infrastructure > drivers: cpuidle: CPU idle ARM64 driver > drivers: cpuidle: initialize big.LITTLE driver through DT > drivers: cpuidle: initialize Exynos driver through DT > > arch/arm/boot/dts/exynos4210.dtsi | 27 ++++ > arch/arm/boot/dts/exynos4212.dtsi | 17 +++ > arch/arm/boot/dts/exynos4412.dtsi | 29 ++++ > arch/arm/boot/dts/exynos5250.dtsi | 11 ++ > arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts | 23 ++++ > drivers/cpuidle/Kconfig | 8 ++ > drivers/cpuidle/Kconfig.arm | 2 + > drivers/cpuidle/Kconfig.arm64 | 14 ++ > drivers/cpuidle/Makefile | 5 + > drivers/cpuidle/cpuidle-arm64.c | 133 ++++++++++++++++++ > drivers/cpuidle/cpuidle-big_little.c | 19 +++ > drivers/cpuidle/cpuidle-exynos.c | 18 ++- > drivers/cpuidle/dt_idle_states.c | 213 +++++++++++++++++++++++++++++ > drivers/cpuidle/dt_idle_states.h | 7 + > 14 files changed, 525 insertions(+), 1 deletion(-) > create mode 100644 drivers/cpuidle/Kconfig.arm64 > create mode 100644 drivers/cpuidle/cpuidle-arm64.c > create mode 100644 drivers/cpuidle/dt_idle_states.c > create mode 100644 drivers/cpuidle/dt_idle_states.h Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics Index: b/arch/arm/boot/dts/exynos4210.dtsi =================================================================== --- a/arch/arm/boot/dts/exynos4210.dtsi 2014-09-23 14:14:29.793571708 +0200 +++ b/arch/arm/boot/dts/exynos4210.dtsi 2014-09-23 14:17:22.029565827 +0200 @@ -39,6 +39,7 @@ <&clock CLK_OUT_CPU>, <&clock CLK_XXTI>, <&clock CLK_XUSBXTI>; #clock-cells = <1>; + }; cpus { #address-cells = <1>;