From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> To: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> Cc: Catalin Marinas <Catalin.Marinas@arm.com>, Daniel Lezcano <daniel.lezcano@linaro.org>, Will Deacon <Will.Deacon@arm.com>, Mark Rutland <Mark.Rutland@arm.com>, Lina Iyer <lina.iyer@linaro.org>, Chander Kashyap <k.chander@samsung.com>, Vincent Guittot <vincent.guittot@linaro.org>, Nicolas Pitre <nico@linaro.org>, Ashwin Chaugule <ashwin.chaugule@linaro.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "grant.likely@linaro.org" <grant.likely@linaro.org>, Charles Garcia-Tobin <Charles.Garcia-Tobin@arm.com>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Kevin Hilman <khilman@linaro.org>, "linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>, Sebastian Capella <sebcape@gmail.com>, Mark Brown <broonie@kernel.org>, Antti Miettinen <ananaza@iki.fi>, Paul Walmsley <paul@pwsan.com>, Geoff Levand <geoff@infradead.org>, Peter Subject: Re: [PATCH v8 6/8] drivers: cpuidle: CPU idle ARM64 driver Date: Tue, 23 Sep 2014 19:14:47 +0100 [thread overview] Message-ID: <20140923181447.GA6269@e102568-lin.cambridge.arm.com> (raw) In-Reply-To: <4228539.nCGoXxuRMK@amdc1032> On Tue, Sep 23, 2014 at 02:35:14PM +0100, Bartlomiej Zolnierkiewicz wrote: > > 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). I won't regenerate the branch, Daniel knows that the last two patches (Samsung changes) were subject to testing and he has an mbox (and the branch above) that he can use to cherry-pick the patches going through his tree, as planned. Daniel, can you squash the fix below into Bart's original patch (that is part of the mbox I sent you and also part of the pull request) and stick his tested-by to the Exynos DT CPUidle patch (last in the series) please ? I won't be able to regenerate a pull request till Monday, and honestly there is no need for that. Thanks a lot, Lorenzo > > --- > arch/arm/boot/dts/exynos4210.dtsi | 1 + > 1 file changed, 1 insertion(+) > > 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>; > > > 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 > > >
WARNING: multiple messages have this Message-ID (diff)
From: lorenzo.pieralisi@arm.com (Lorenzo Pieralisi) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v8 6/8] drivers: cpuidle: CPU idle ARM64 driver Date: Tue, 23 Sep 2014 19:14:47 +0100 [thread overview] Message-ID: <20140923181447.GA6269@e102568-lin.cambridge.arm.com> (raw) In-Reply-To: <4228539.nCGoXxuRMK@amdc1032> On Tue, Sep 23, 2014 at 02:35:14PM +0100, Bartlomiej Zolnierkiewicz wrote: > > 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). I won't regenerate the branch, Daniel knows that the last two patches (Samsung changes) were subject to testing and he has an mbox (and the branch above) that he can use to cherry-pick the patches going through his tree, as planned. Daniel, can you squash the fix below into Bart's original patch (that is part of the mbox I sent you and also part of the pull request) and stick his tested-by to the Exynos DT CPUidle patch (last in the series) please ? I won't be able to regenerate a pull request till Monday, and honestly there is no need for that. Thanks a lot, Lorenzo > > --- > arch/arm/boot/dts/exynos4210.dtsi | 1 + > 1 file changed, 1 insertion(+) > > 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>; > > > 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 > > >
next prev parent reply other threads:[~2014-09-23 18:14 UTC|newest] Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-09-01 15:28 [PATCH v8 0/8] ARM generic idle states Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi 2014-09-01 15:28 ` [PATCH v8 1/8] arm64: kernel: refactor the CPU suspend API for retention states Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi [not found] ` <1409585324-3678-1-git-send-email-lorenzo.pieralisi-5wv7dgnIgG8@public.gmane.org> 2014-09-01 15:28 ` [PATCH v8 2/8] Documentation: arm: define DT idle states bindings Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi 2014-09-01 15:28 ` [PATCH v8 3/8] drivers: cpuidle: implement DT based idle states infrastructure Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi 2014-09-03 13:25 ` Daniel Lezcano 2014-09-03 13:25 ` Daniel Lezcano 2014-09-03 17:30 ` Lorenzo Pieralisi 2014-09-03 17:30 ` Lorenzo Pieralisi 2014-09-01 15:28 ` [PATCH v8 4/8] arm64: kernel: introduce cpu_init_idle CPU operation Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi 2014-09-03 17:34 ` Lina Iyer 2014-09-03 17:34 ` Lina Iyer 2014-09-03 17:46 ` Lorenzo Pieralisi 2014-09-03 17:46 ` Lorenzo Pieralisi 2014-09-03 19:16 ` Lina Iyer 2014-09-03 19:16 ` Lina Iyer 2014-09-01 15:28 ` [PATCH v8 5/8] arm64: add PSCI CPU_SUSPEND based cpu_suspend support Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi 2014-09-01 15:28 ` [PATCH v8 6/8] drivers: cpuidle: CPU idle ARM64 driver Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi 2014-09-03 17:37 ` Lorenzo Pieralisi 2014-09-03 17:37 ` Lorenzo Pieralisi 2014-09-04 16:03 ` Catalin Marinas 2014-09-04 16:03 ` Catalin Marinas 2014-09-04 17:29 ` Lorenzo Pieralisi 2014-09-04 17:29 ` Lorenzo Pieralisi 2014-09-05 9:21 ` Will Deacon 2014-09-05 9:21 ` Will Deacon 2014-09-05 15:34 ` Lorenzo Pieralisi 2014-09-05 15:34 ` Lorenzo Pieralisi 2014-09-11 8:28 ` Daniel Lezcano 2014-09-11 8:28 ` Daniel Lezcano 2014-09-11 8:57 ` Lorenzo Pieralisi 2014-09-11 8:57 ` Lorenzo Pieralisi [not found] ` <20140911085727.GA25773-7AyDDHkRsp3ZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org> 2014-09-11 9:32 ` Daniel Lezcano 2014-09-11 9:32 ` Daniel Lezcano 2014-09-12 10:32 ` Catalin Marinas 2014-09-12 10:32 ` Catalin Marinas 2014-09-12 11:26 ` Lorenzo Pieralisi 2014-09-12 11:26 ` Lorenzo Pieralisi 2014-09-23 13:35 ` Bartlomiej Zolnierkiewicz 2014-09-23 13:35 ` Bartlomiej Zolnierkiewicz 2014-09-23 18:14 ` Lorenzo Pieralisi [this message] 2014-09-23 18:14 ` Lorenzo Pieralisi 2014-09-14 16:59 ` Tomasz Figa 2014-09-14 16:59 ` Tomasz Figa 2014-09-23 13:42 ` Bartlomiej Zolnierkiewicz 2014-09-23 13:42 ` Bartlomiej Zolnierkiewicz 2014-09-23 18:16 ` Lorenzo Pieralisi 2014-09-23 18:16 ` Lorenzo Pieralisi 2014-09-29 11:08 ` Lorenzo Pieralisi 2014-09-29 11:08 ` Lorenzo Pieralisi 2014-09-01 15:28 ` [PATCH v8 7/8] drivers: cpuidle: initialize big.LITTLE driver through DT Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi 2014-09-03 13:29 ` Daniel Lezcano 2014-09-03 13:29 ` Daniel Lezcano 2014-09-01 15:28 ` [PATCH v8 8/8] drivers: cpuidle: initialize Exynos " Lorenzo Pieralisi 2014-09-01 15:28 ` Lorenzo Pieralisi 2014-09-03 13:32 ` Daniel Lezcano 2014-09-03 13:32 ` Daniel Lezcano 2014-09-03 17:29 ` Lorenzo Pieralisi 2014-09-03 17:29 ` Lorenzo Pieralisi
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20140923181447.GA6269@e102568-lin.cambridge.arm.com \ --to=lorenzo.pieralisi@arm.com \ --cc=Catalin.Marinas@arm.com \ --cc=Charles.Garcia-Tobin@arm.com \ --cc=Mark.Rutland@arm.com \ --cc=Will.Deacon@arm.com \ --cc=ananaza@iki.fi \ --cc=ashwin.chaugule@linaro.org \ --cc=b.zolnierkie@samsung.com \ --cc=broonie@kernel.org \ --cc=daniel.lezcano@linaro.org \ --cc=devicetree@vger.kernel.org \ --cc=geoff@infradead.org \ --cc=grant.likely@linaro.org \ --cc=k.chander@samsung.com \ --cc=khilman@linaro.org \ --cc=lina.iyer@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-pm@vger.kernel.org \ --cc=nico@linaro.org \ --cc=paul@pwsan.com \ --cc=sebcape@gmail.com \ --cc=vincent.guittot@linaro.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.