From: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
To: Lorenzo Pieralisi <lorenzo.pieralisi@arm.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 15:35:14 +0200 [thread overview]
Message-ID: <4228539.nCGoXxuRMK@amdc1032> (raw)
In-Reply-To: <20140912112643.GA10623@red-moon>
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(+)
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: b.zolnierkie@samsung.com (Bartlomiej Zolnierkiewicz)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v8 6/8] drivers: cpuidle: CPU idle ARM64 driver
Date: Tue, 23 Sep 2014 15:35:14 +0200 [thread overview]
Message-ID: <4228539.nCGoXxuRMK@amdc1032> (raw)
In-Reply-To: <20140912112643.GA10623@red-moon>
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(+)
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 13:35 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 [this message]
2014-09-23 13:35 ` Bartlomiej Zolnierkiewicz
2014-09-23 18:14 ` Lorenzo Pieralisi
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=4228539.nCGoXxuRMK@amdc1032 \
--to=b.zolnierkie@samsung.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=broonie@kernel.org \
--cc=catalin.marinas@arm.com \
--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=lorenzo.pieralisi@arm.com \
--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: link
Be 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.