Skip to content

OPP: Simplify set_required_opp handling

Viresh Kumar linux-pm lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 11 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3OPP: Handle all genpd cases together in _set_required_opps()drivers/opp/core.c+2 -71 reply
2/3OPP: Move required opps configuration to specialized callbackdrivers/opp/core.c drivers/opp/of.c drivers/opp/opp.h+49 -272 replies
3/3OPP: Allow platforms to add a set_required_opps() callbackdrivers/opp/core.c drivers/opp/opp.h include/linux/pm_opp.h+42 -20 replies

Reviews trailers per patch

2 of 3 patches have a Reviewed-by or Acked-by.

PatchUlf Hansson
1/3R-b
2/3R-b
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/3] OPP: Simplify set_required_opp handling

Hello,

The required-opps configuration is closely tied to genpd and performance states at the moment and it is not very obvious that required-opps can live without genpds. Though we don't support configuring required-opps for non-genpd cases currently.

This patchset aims at cleaning up this a bit, just like what's done for clk and regulators. This also makes it possible for platforms to provide their own version of set_required_opps() helper, which can be used to configure the devfreq device propertly.

Jun,

I haven't found time to test this through yet, though there isn't much anyway I guess. Can you see if these can solve your problem properly ?

Viresh Kumar (3):
OPP: Handle all genpd cases together in _set_required_opps()
OPP: Move required opps configuration to specialized callback
OPP: Allow platforms to add a set_required_opps() callback

 drivers/opp/core.c     | 113 ++++++++++++++++++++++++++++-------------
 drivers/opp/of.c       |   3 ++
 drivers/opp/opp.h      |   4 ++
 include/linux/pm_opp.h |   5 ++
 4 files changed, 91 insertions(+), 34 deletions(-)
Signature · 1 line
2.31.1.272.g89b43f80a514