diff options
| author | Viresh Kumar <viresh.kumar@linaro.org> | 2026-02-23 11:05:59 +0530 |
|---|---|---|
| committer | Viresh Kumar <viresh.kumar@linaro.org> | 2026-02-23 11:05:59 +0530 |
| commit | 3d2398f44a2d48fb1c575a6e0bc6b38f3e689e22 (patch) | |
| tree | 912a3dda8a6def1609c8d096d15397e2ba51b51d /drivers/opp | |
| parent | e560083c0467f86b72aecac377b27bd1e7d16c49 (diff) | |
OPP: Move break out of scoped_guard in dev_pm_opp_xlate_required_opp()
The commit ff9c512041f2 ("OPP: Use mutex locking guards")
unintentionally made the for loop run longer than required.
scoped_guard() is implemented as a for loop. The break statement now
breaks out out the scoped_guard() and not out of the outer for loop.
The outer loop always iterates to completion.
Fix it.
Fixes: ff9c512041f2 ("OPP: Use mutex locking guards")
Reported-by: David Lechner <dlechner@baylibre.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Diffstat (limited to 'drivers/opp')
| -rw-r--r-- | drivers/opp/core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/opp/core.c b/drivers/opp/core.c index 866641666e41..da3f5eba4341 100644 --- a/drivers/opp/core.c +++ b/drivers/opp/core.c @@ -2742,8 +2742,8 @@ struct dev_pm_opp *dev_pm_opp_xlate_required_opp(struct opp_table *src_table, break; } } - break; } + break; } if (IS_ERR(dest_opp)) { |