diff options
| author | Lifeng Zheng <zhenglifeng1@huawei.com> | 2025-12-02 15:27:27 +0800 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2025-12-16 13:09:48 +0100 |
| commit | 78d83b293891c597cef773eb17d9cc02b386f21a (patch) | |
| tree | db9cb3bc6fdd79792108b9ccad30ad48e15d6ab8 /drivers | |
| parent | 77cf053b041fe13d1fdd2e572e16ee7776ff687d (diff) | |
cpufreq: cpufreq_boost_trigger_state() optimization
Optimize the error handling code in cpufreq_boost_trigger_state().
Signed-off-by: Lifeng Zheng <zhenglifeng1@huawei.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: Jie Zhan <zhanjie9@hisilicon.com>
[ rjw: Changelog edit ]
Link: https://patch.msgid.link/20251202072727.1368285-3-zhenglifeng1@huawei.com
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/cpufreq/cpufreq.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 8de9c94c097f..50dde2980f1b 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -2820,19 +2820,14 @@ static int cpufreq_boost_trigger_state(int state) continue; ret = policy_set_boost(policy, state); - if (ret) - goto err_reset_state; + if (unlikely(ret)) + break; } - if (ret) - goto err_reset_state; - cpus_read_unlock(); - return 0; - -err_reset_state: - cpus_read_unlock(); + if (likely(!ret)) + return 0; write_lock_irqsave(&cpufreq_driver_lock, flags); cpufreq_driver->boost_enabled = !state; |