mbox series

[V2,00/11] PM / genpd & OPP: Parse performance state from DT

Message ID cover.1523273291.git.viresh.kumar@linaro.org
Headers show
Series PM / genpd & OPP: Parse performance state from DT | expand

Message

Viresh Kumar April 9, 2018, 11:43 a.m. UTC 
Hi,

Here is the V2 of the series which contains several modifications since
the previous version that was mostly reviewed by Ulf.

This series provides support to parse the performance states of devices
and their PM domains from DT instead of static in-kernel tables.

This is tested on Hikey620 and works just fine.

V1->V2:
- It contains 4 new patches which were required to complete all pending
  stuff in this activity.
  - Patch 1 and 2 does minor changes to the DT bindings as suggested
    earlier on the lists.
  - Patch 3 updates code based on them.
  - Patch 6 adds a new helper which is required by genpd drivers. This
    was already posted by two different people separately on the lists,
    while all the versions want to do the same thing. Signed-off-by's
    from all of them is added to this commit.
- The genpd bus is no more required.
- The genpd device isn't registered anymore.
- Some renames of the APIs and callbacks.
- Commit log updates, along with tags from reviewers.

--
viresh

Viresh Kumar (11):
  PM / OPP: dt-bindings: Rename "required-opp" as "required-opps"
  PM / OPP: dt-bindings: Make "opp-hz" optional for power domains
  PM / OPP: "opp-hz" is optional for power domains
  PM / OPP: Implement dev_pm_opp_of_add_table_indexed()
  PM / OPP: Implement of_dev_pm_opp_find_required_opp()
  PM / OPP: Implement dev_pm_opp_get_of_node()
  PM / Domain: Add struct device to genpd
  PM / Domain: Add support to parse domain's OPP table
  PM / Domain: Implement of_dev_pm_genpd_opp_to_performance_state()
  PM / OPP: Get performance state using genpd helper
  PM / OPP: Remove dev_pm_opp_{un}register_get_pstate_helper()

 Documentation/devicetree/bindings/opp/opp.txt      |   7 +-
 .../devicetree/bindings/power/power_domain.txt     |   6 +-
 drivers/base/power/domain.c                        | 127 +++++++++++++--
 drivers/opp/core.c                                 | 174 +++++++--------------
 drivers/opp/debugfs.c                              |  15 +-
 drivers/opp/of.c                                   | 169 +++++++++++++++++---
 drivers/opp/opp.h                                  |   6 +-
 include/linux/pm_domain.h                          |  13 ++
 include/linux/pm_opp.h                             |  27 ++--
 9 files changed, 375 insertions(+), 169 deletions(-)

-- 
2.15.0.194.g9af6a3dea062

Comments

Rob Herring May 8, 2018, 3:42 p.m. UTC | #1 
On Tue, May 08, 2018 at 09:04:42AM +0530, Viresh Kumar wrote:
> The "opp-hz" property is not relevant across all the devices that use

> the OPP tables now. For example, for a power domain a frequency value

> wouldn't mean anything. Though they must have another property, which

> may be implementation defined, which uniquely identifies the OPP nodes.

> 

> Make "opp-hz" optional for such devices.

> 

> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

> Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>

> ---

> Resending to get Ack from Rob.

> 

>  Documentation/devicetree/bindings/opp/opp.txt | 5 ++++-

>  1 file changed, 4 insertions(+), 1 deletion(-)


Reviewed-by: Rob Herring <robh@kernel.org>