From: Ulf Hansson <ulf.hansson@linaro.org>
To: Sudeep Holla <sudeep.holla@arm.com>,
Lorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>
Cc: Rob Herring <robh+dt@kernel.org>,
Vincent Guittot <vincent.guittot@linaro.org>,
Stephen Boyd <sboyd@kernel.org>, Andy Gross <agross@kernel.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Kevin Hilman <khilman@kernel.org>,
Linux ARM <linux-arm-kernel@lists.infradead.org>,
linux-arm-msm <linux-arm-msm@vger.kernel.org>,
Lina Iyer <ilina@codeaurora.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
"Rafael J . Wysocki" <rjw@rjwysocki.net>,
Mark Rutland <mark.rutland@arm.com>,
Linux PM <linux-pm@vger.kernel.org>
Subject: Re: [PATCH v2 00/13] cpuidle: psci: Support hierarchical CPU arrangement
Date: Mon, 11 Nov 2019 12:00:52 +0100 [thread overview]
Message-ID: <CAPDyKFpiMK_P+4+n9wHc+68X6j44XOoTm=J8OXz5HkqoMxOsOg@mail.gmail.com> (raw)
In-Reply-To: <20191029164438.17012-1-ulf.hansson@linaro.org>
On Tue, 29 Oct 2019 at 17:44, Ulf Hansson <ulf.hansson@linaro.org> wrote:
>
> Changes in v2:
> - Avoid to affect the non-OSI path with specific changes for OSI. This
> forced me to re-order the series and a caused more or less minor changes
> to most of the patches.
> - Updated the DT bindings for PSCI to clarify and to include the "psci"
> name of the PM domain to attach to.
> - Replaced patch1 with another patch from Sudeep, solving the same
> problem, but in a different way.
Hi Sudeep and Lorenzo,
Apologize for nagging you about reviews, again. Can you please have a
look at the new version!?
Kind regards
Uffe
>
> This is the same coverletter as in v1:
>
> This series enables initial support for hierarchical CPU arrangement, managed
> by PSCI and its corresponding cpuidle driver. It's based on using the generic
> PM domain (genpd), which nowadays also supports devices belonging to CPUs.
>
> The last DTS patch enables the hierarchical topology to be used for the Qcom
> 410c Dragonboard, which supports the PSCI OS-initiated mode.
>
> Do note, most of the code in this series have been posted earlier, but from the
> latest version being reviewed, we agreed on that it was better to re-work the
> PSCI backend driver as a first step, simply to get a clean interface towards the
> cpuidle driver.
>
> Summary of the main-changes since the last submission [1]:
>
> - Moved implementation from the psci FW dricer into cpuidle-psci.
>
> - Re-requesting review of the DT bindings, as we have moved to yaml. No
> changes as such, but tried to clarify a few things in the text.
>
> - Drop the patch enabling support for CPU hotplug, postponing that to the next
> step.
>
> - Respect the hierarchical topology in DT only when OSI mode is supported.
> This is to start simple and we can always extend the support on top.
>
> The series is also available at:
> git.linaro.org/people/ulf.hansson/linux-pm.git next
>
> Kind regards
> Ulf Hansson
>
> [1]
> https://lwn.net/Articles/788306/
>
> Lina Iyer (1):
> cpuidle: dt: Support hierarchical CPU idle states
>
> Sudeep Holla (1):
> cpuidle: psci: Align psci_power_state count with idle state count
>
> Ulf Hansson (11):
> dt: psci: Update DT bindings to support hierarchical PSCI states
> firmware: psci: Export functions to manage the OSI mode
> of: base: Add of_get_cpu_state_node() to get idle states for a CPU
> node
> cpuidle: psci: Simplify OF parsing of CPU idle state nodes
> cpuidle: psci: Support hierarchical CPU idle states
> cpuidle: psci: Add a helper to attach a CPU to its PM domain
> cpuidle: psci: Attach CPU devices to their PM domains
> cpuidle: psci: Prepare to use OS initiated suspend mode via PM domains
> cpuidle: psci: Manage runtime PM in the idle path
> cpuidle: psci: Add support for PM domains by using genpd
> arm64: dts: Convert to the hierarchical CPU topology layout for
> MSM8916
>
> .../devicetree/bindings/arm/cpus.yaml | 15 +
> .../devicetree/bindings/arm/psci.yaml | 102 ++++++
> arch/arm64/boot/dts/qcom/msm8916.dtsi | 57 +++-
> drivers/cpuidle/Makefile | 4 +-
> drivers/cpuidle/cpuidle-psci-domain.c | 302 ++++++++++++++++++
> drivers/cpuidle/cpuidle-psci.c | 123 +++++--
> drivers/cpuidle/cpuidle-psci.h | 17 +
> drivers/cpuidle/dt_idle_states.c | 5 +-
> drivers/firmware/psci/psci.c | 18 +-
> drivers/of/base.c | 36 +++
> include/linux/of.h | 8 +
> include/linux/psci.h | 2 +
> 12 files changed, 654 insertions(+), 35 deletions(-)
> create mode 100644 drivers/cpuidle/cpuidle-psci-domain.c
> create mode 100644 drivers/cpuidle/cpuidle-psci.h
>
> --
> 2.17.1
>
next prev parent reply other threads:[~2019-11-11 11:01 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-29 16:44 [PATCH v2 00/13] cpuidle: psci: Support hierarchical CPU arrangement Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 01/13] cpuidle: psci: Align psci_power_state count with idle state count Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 02/13] dt: psci: Update DT bindings to support hierarchical PSCI states Ulf Hansson
2019-11-15 17:12 ` Sudeep Holla
2019-11-19 15:15 ` Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 03/13] firmware: psci: Export functions to manage the OSI mode Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 04/13] of: base: Add of_get_cpu_state_node() to get idle states for a CPU node Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 05/13] cpuidle: dt: Support hierarchical CPU idle states Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 06/13] cpuidle: psci: Simplify OF parsing of CPU idle state nodes Ulf Hansson
2019-11-15 17:13 ` Sudeep Holla
2019-10-29 16:44 ` [PATCH v2 07/13] cpuidle: psci: Support hierarchical CPU idle states Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 08/13] cpuidle: psci: Add a helper to attach a CPU to its PM domain Ulf Hansson
2019-11-07 9:13 ` Niklas Cassel
2019-11-07 10:22 ` Ulf Hansson
2019-11-11 11:36 ` Niklas Cassel
2019-11-15 17:13 ` Sudeep Holla
2019-11-18 12:28 ` Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 09/13] cpuidle: psci: Attach CPU devices to their PM domains Ulf Hansson
2019-11-15 17:15 ` Sudeep Holla
2019-11-18 12:50 ` Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 10/13] cpuidle: psci: Prepare to use OS initiated suspend mode via " Ulf Hansson
2019-11-15 17:30 ` Sudeep Holla
2019-11-18 13:37 ` Ulf Hansson
2019-11-22 18:26 ` Lorenzo Pieralisi
2019-11-25 10:44 ` Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 11/13] cpuidle: psci: Manage runtime PM in the idle path Ulf Hansson
2019-11-15 17:32 ` Sudeep Holla
2019-10-29 16:44 ` [PATCH v2 12/13] cpuidle: psci: Add support for PM domains by using genpd Ulf Hansson
2019-10-29 16:44 ` [PATCH v2 13/13] arm64: dts: Convert to the hierarchical CPU topology layout for MSM8916 Ulf Hansson
2019-11-11 11:00 ` Ulf Hansson [this message]
2019-11-11 14:31 ` [PATCH v2 00/13] cpuidle: psci: Support hierarchical CPU arrangement Sudeep Holla
2019-11-22 8:14 ` Ulf Hansson
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='CAPDyKFpiMK_P+4+n9wHc+68X6j44XOoTm=J8OXz5HkqoMxOsOg@mail.gmail.com' \
--to=ulf.hansson@linaro.org \
--cc=Lorenzo.Pieralisi@arm.com \
--cc=agross@kernel.org \
--cc=bjorn.andersson@linaro.org \
--cc=daniel.lezcano@linaro.org \
--cc=ilina@codeaurora.org \
--cc=khilman@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=rjw@rjwysocki.net \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=sudeep.holla@arm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).