2021-07-30 14:40:50

by Rafael J. Wysocki

[permalink] [raw]
Subject: [PATCH v1 2/2] cpuidle: teo: Rename two local variables in teo_select()

From: Rafael J. Wysocki <[email protected]>

Rename two local variables in teo_select() so that their names better
reflect their purpose.

No functional impact.

Signed-off-by: Rafael J. Wysocki <[email protected]>
---
drivers/cpuidle/governors/teo.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)

Index: linux-pm/drivers/cpuidle/governors/teo.c
===================================================================
--- linux-pm.orig/drivers/cpuidle/governors/teo.c
+++ linux-pm/drivers/cpuidle/governors/teo.c
@@ -382,8 +382,8 @@ static int teo_select(struct cpuidle_dri
alt_intercepts = 2 * idx_intercept_sum > cpu_data->total - idx_hit_sum;
alt_recent = idx_recent_sum > NR_RECENT / 2;
if (alt_recent || alt_intercepts) {
- s64 last_enabled_span_ns = duration_ns;
- int last_enabled_idx = idx;
+ s64 first_suitable_span_ns = duration_ns;
+ int first_suitable_idx = idx;

/*
* Look for the deepest idle state whose target residency had
@@ -419,8 +419,8 @@ static int teo_select(struct cpuidle_dri
* disabled, so take the first enabled
* deeper state with suitable time span.
*/
- idx = last_enabled_idx;
- duration_ns = last_enabled_span_ns;
+ idx = first_suitable_idx;
+ duration_ns = first_suitable_span_ns;
}
break;
}
@@ -434,14 +434,14 @@ static int teo_select(struct cpuidle_dri
* alternative candidate state has been found,
* it may still turn out to be a better choice.
*/
- if (last_enabled_idx != idx)
+ if (first_suitable_idx != idx)
continue;

break;
}

- last_enabled_span_ns = span_ns;
- last_enabled_idx = i;
+ first_suitable_span_ns = span_ns;
+ first_suitable_idx = i;
}
}