iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

opp: Allow dev_pm_opp_put_*() APIs to accept NULL opp_table

Browse Source 
This allows the callers to drop the unnecessary checks.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
This commit is contained in:
Viresh Kumar 2020-11-06 12:16:52 +05:30
parent e77dcb0b73
commit c7bf8758c9
1 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
drivers/opp/core.c
View File

@ -1660,6 +1660,9 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_set_supported_hw);
*/ */
void dev_pm_opp_put_supported_hw(struct opp_table *opp_table) void dev_pm_opp_put_supported_hw(struct opp_table *opp_table)
{ {
if (unlikely(!opp_table))
return;
/* Make sure there are no concurrent readers while updating opp_table */ /* Make sure there are no concurrent readers while updating opp_table */
WARN_ON(!list_empty(&opp_table->opp_list)); WARN_ON(!list_empty(&opp_table->opp_list));
@ -1716,6 +1719,9 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_set_prop_name);
*/ */
void dev_pm_opp_put_prop_name(struct opp_table *opp_table) void dev_pm_opp_put_prop_name(struct opp_table *opp_table)
{ {
if (unlikely(!opp_table))
return;
/* Make sure there are no concurrent readers while updating opp_table */ /* Make sure there are no concurrent readers while updating opp_table */
WARN_ON(!list_empty(&opp_table->opp_list)); WARN_ON(!list_empty(&opp_table->opp_list));
@ -1844,6 +1850,9 @@ void dev_pm_opp_put_regulators(struct opp_table *opp_table)
{ {
int i; int i;
if (unlikely(!opp_table))
return;
if (!opp_table->regulators) if (!opp_table->regulators)
goto put_opp_table; goto put_opp_table;
@ -1926,6 +1935,9 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_set_clkname);
*/ */
void dev_pm_opp_put_clkname(struct opp_table *opp_table) void dev_pm_opp_put_clkname(struct opp_table *opp_table)
{ {
if (unlikely(!opp_table))
return;
/* Make sure there are no concurrent readers while updating opp_table */ /* Make sure there are no concurrent readers while updating opp_table */
WARN_ON(!list_empty(&opp_table->opp_list)); WARN_ON(!list_empty(&opp_table->opp_list));
@ -1981,6 +1993,9 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_register_set_opp_helper);
*/ */
void dev_pm_opp_unregister_set_opp_helper(struct opp_table *opp_table) void dev_pm_opp_unregister_set_opp_helper(struct opp_table *opp_table)
{ {
if (unlikely(!opp_table))
return;
/* Make sure there are no concurrent readers while updating opp_table */ /* Make sure there are no concurrent readers while updating opp_table */
WARN_ON(!list_empty(&opp_table->opp_list)); WARN_ON(!list_empty(&opp_table->opp_list));
@ -2109,6 +2124,9 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_attach_genpd);
*/ */
void dev_pm_opp_detach_genpd(struct opp_table *opp_table) void dev_pm_opp_detach_genpd(struct opp_table *opp_table)
{ {
if (unlikely(!opp_table))
return;
/* /*
* Acquire genpd_virt_dev_lock to make sure virt_dev isn't getting * Acquire genpd_virt_dev_lock to make sure virt_dev isn't getting
* used in parallel. * used in parallel.