linux-stable.git/drivers/opp/core.c, branch v5.8.4

opp: Put opp table in dev_pm_opp_set_rate() if _set_opp_bw() fails

2020-08-26T09:49:00+00:00

commit d4ec88d205583ac4f9482cf3e89128589bd881d2 upstream.

We get the opp_table pointer at the top of the function and so we should
put the pointer at the end of the function like all other exit paths
from this function do.

Cc: v5.8+  # v5.8+
Fixes: b00e667a6d8b ("opp: Remove bandwidth votes when target_freq is zero")
Reviewed-by: Rajendra Nayak 
Signed-off-by: Stephen Boyd 
[ Viresh: Split the patch into two ]
Signed-off-by: Viresh Kumar 
Signed-off-by: Greg Kroah-Hartman

opp: Put opp table in dev_pm_opp_set_rate() for empty tables

2020-08-26T09:49:00+00:00

commit 8979ef70850eb469e1094279259d1ef393ffe85f upstream.

We get the opp_table pointer at the top of the function and so we should
put the pointer at the end of the function like all other exit paths
from this function do.

Cc: v5.7+  # v5.7+
Fixes: aca48b61f963 ("opp: Manage empty OPP tables with clk handle")
Reviewed-by: Rajendra Nayak 
Signed-off-by: Stephen Boyd 
[ Viresh: Split the patch into two ]
Signed-off-by: Viresh Kumar 
Signed-off-by: Greg Kroah-Hartman

opp: Enable resources again if they were disabled earlier

2020-08-26T09:49:00+00:00

commit a4501bac0e553bed117b7e1b166d49731caf7260 upstream.

dev_pm_opp_set_rate() can now be called with freq = 0 in order
to either drop performance or bandwidth votes or to disable
regulators on platforms which support them.

In such cases, a subsequent call to dev_pm_opp_set_rate() with
the same frequency ends up returning early because 'old_freq == freq'

Instead make it fall through and put back the dropped performance
and bandwidth votes and/or enable back the regulators.

Cc: v5.3+  # v5.3+
Fixes: cd7ea582866f ("opp: Make dev_pm_opp_set_rate() handle freq = 0 to drop performance votes")
Reported-by: Sajida Bhanu 
Reviewed-by: Sibi Sankar 
Reported-by: Matthias Kaehlcke 
Tested-by: Matthias Kaehlcke 
Reviewed-by: Stephen Boyd 
Signed-off-by: Rajendra Nayak 
[ Viresh: Don't skip clk_set_rate() and massaged changelog ]
Signed-off-by: Viresh Kumar 
Signed-off-by: Greg Kroah-Hartman

opp: Remove bandwidth votes when target_freq is zero

2020-05-29T04:45:12+00:00

We already drop several votes when target_freq is set to zero, drop
bandwidth votes as well.

Reported-by: Sibi Sankar 
Reviewed-by: Georgi Djakov 
Tested-by: Georgi Djakov 
Reviewed-by: Sibi Sankar 
Tested-by: Sibi Sankar 
Signed-off-by: Viresh Kumar

opp: core: add regulators enable and disable

2020-05-29T04:45:12+00:00

Add enable regulators to dev_pm_opp_set_regulators() and disable
regulators to dev_pm_opp_put_regulators(). Even if bootloader
leaves regulators enabled, they should be enabled in kernel in
order to increase the reference count.

Tested-by: Marek Szyprowski 
Acked-by: Clément Péron 
Tested-by: Clément Péron 
Signed-off-by: Kamil Konieczny 
[ Viresh: Enable the regulator only after it is programmed and add a
	  flag to track its status. ]
Signed-off-by: Viresh Kumar

opp: Reorder the code for !target_freq case

2020-05-29T04:45:12+00:00

Reorder the code a bit to make it more readable. Add additional comment
as well.

Tested-by: Marek Szyprowski 
Acked-by: Clément Péron 
Tested-by: Clément Péron 
Signed-off-by: Viresh Kumar

opp: Update the bandwidth on OPP frequency changes

2020-05-29T04:45:10+00:00

If the OPP bandwidth values are populated, we want to switch also the
interconnect bandwidth in addition to frequency and voltage.

Signed-off-by: Georgi Djakov 
Reviewed-by: Matthias Kaehlcke 
Reviewed-by: Sibi Sankar 
Signed-off-by: Viresh Kumar

opp: Add support for parsing interconnect bandwidth

2020-05-29T04:45:08+00:00

The OPP bindings now support bandwidth values, so add support to parse it
from device tree and store it into the new dev_pm_opp_icc_bw struct, which
is part of the dev_pm_opp.

Signed-off-by: Georgi Djakov 
Reviewed-by: Matthias Kaehlcke 
[ Viresh: Create _read_bw() and use it, renamed _of_find_icc_paths() to
	  dev_pm_opp_of_find_icc_paths(), exported it and made opp_table
	  argument optional. Also drop the depends on from Kconfig. ]
Signed-off-by: Viresh Kumar

OPP: Add helpers for reading the binding properties

2020-05-13T06:18:13+00:00

The opp-hz DT property is not mandatory and we may use another property
as a key in the OPP table. Add helper functions to simplify the reading
and comparing the keys.

Signed-off-by: Saravana Kannan 
Signed-off-by: Georgi Djakov 
Reviewed-by: Matthias Kaehlcke 
Reviewed-by: Sibi Sankar 
[ Viresh: Removed an unnecessary comment ]
Signed-off-by: Viresh Kumar

opp: Manage empty OPP tables with clk handle

2020-04-13T10:44:55+00:00

With OPP core now supporting DVFS for IO devices, we have instances of
IO devices (same IP block) which require an OPP on some platforms/SoCs
while just needing to scale the clock on some others.

In order to avoid conditional code in every driver which supports such
devices (to check for availability of OPPs and then deciding to do
either dev_pm_opp_set_rate() or clk_set_rate()) add support to manage
empty OPP tables with a clk handle.

This makes dev_pm_opp_set_rate() equivalent of a clk_set_rate() for
devices with just a clk and no OPPs specified, and makes
dev_pm_opp_set_rate(0) bail out without throwing an error.

Signed-off-by: Rajendra Nayak 
Signed-off-by: Viresh Kumar