Replace the manual cpufreq_cpu_put() with __free(put_cpufreq_policy)
annotation for policy references. This reduces the risk of reference
counting mistakes and aligns the code with the latest kernel style.

No functional change intended.

Signed-off-by: Zihuan Zhang <zhangzih...@kylinos.cn>
---
 kernel/power/energy_model.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/kernel/power/energy_model.c b/kernel/power/energy_model.c
index ea7995a25780..4f91112c10bd 100644
--- a/kernel/power/energy_model.c
+++ b/kernel/power/energy_model.c
@@ -451,7 +451,7 @@ static void
 em_cpufreq_update_efficiencies(struct device *dev, struct em_perf_state *table)
 {
        struct em_perf_domain *pd = dev->em_pd;
-       struct cpufreq_policy *policy;
+       struct cpufreq_policy *policy __free(put_cpufreq_policy);
        int found = 0;
        int i, cpu;
 
@@ -479,8 +479,6 @@ em_cpufreq_update_efficiencies(struct device *dev, struct 
em_perf_state *table)
                        found++;
        }
 
-       cpufreq_cpu_put(policy);
-
        if (!found)
                return;
 
@@ -787,7 +785,7 @@ static void em_check_capacity_update(void)
 
        /* Check if CPUs capacity has changed than update EM */
        for_each_possible_cpu(cpu) {
-               struct cpufreq_policy *policy;
+               struct cpufreq_policy *policy __free(put_cpufreq_policy);
                struct em_perf_domain *pd;
                struct device *dev;
 
@@ -801,7 +799,6 @@ static void em_check_capacity_update(void)
                                              msecs_to_jiffies(1000));
                        break;
                }
-               cpufreq_cpu_put(policy);
 
                dev = get_cpu_device(cpu);
                pd = em_pd_get(dev);
-- 
2.25.1

Reply via email to