Skip to content

OPP: Disallow "opp-hz" property without a corresponding clk

Viresh Kumar linux-arm-msm linux-pm lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 15 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2cpufreq: qcom-cpufreq-hw: Register config_clks helperdrivers/cpufreq/qcom-cpufreq-hw.c+31 -30 replies
2/2OPP: Disallow "opp-hz" property without a corresponding clkdrivers/opp/core.c drivers/opp/debugfs.c+1 -1515 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/2] OPP: Disallow "opp-hz" property without a corresponding clk

This removes the special code added by the commit 2083da24eb56 ("OPP:
Allow multiple clocks for a device"), to make it work for Qcom SoC.

In qcom-cpufreq-hw driver, we want to skip clk configuration that happens via dev_pm_opp_set_opp(), but still want the OPP core to parse the "opp-hz" property so we can use the freq based OPP helpers.

The DT for Qcom SoCs is fixed now and contain a valid clk entry, and we no longer need the special handling of the same in the OPP core. Update Qcom driver as well to disallow freq update to the clk.

Mani, can you give this a try again please, thanks ? I have only compile tested this.

Viresh Kumar (2):
cpufreq: qcom-cpufreq-hw: Register config_clks helper
OPP: Disallow "opp-hz" property without a corresponding clk

 drivers/cpufreq/qcom-cpufreq-hw.c | 34 ++++++++++++++++++++++++++++---
 drivers/opp/core.c                | 14 -------------
 drivers/opp/debugfs.c             |  2 +-
 3 files changed, 32 insertions(+), 18 deletions(-)
Signature · 1 line
2.31.1.272.g89b43f80a514