Skip to content

Commit eead184

Browse files
committed
cpufreq: CPPC: Pass structure instance by reference
Don't pass structure instance by value, pass it by reference instead. Tested-by: Vincent Guittot <[email protected]> Reviewed-by: Ionela Voinescu <[email protected]> Tested-by: Qian Cai <[email protected]> Signed-off-by: Viresh Kumar <[email protected]>
1 parent fe2535a commit eead184

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

drivers/cpufreq/cppc_cpufreq.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -375,18 +375,18 @@ static inline u64 get_delta(u64 t1, u64 t0)
375375
}
376376

377377
static int cppc_get_rate_from_fbctrs(struct cppc_cpudata *cpu_data,
378-
struct cppc_perf_fb_ctrs fb_ctrs_t0,
379-
struct cppc_perf_fb_ctrs fb_ctrs_t1)
378+
struct cppc_perf_fb_ctrs *fb_ctrs_t0,
379+
struct cppc_perf_fb_ctrs *fb_ctrs_t1)
380380
{
381381
u64 delta_reference, delta_delivered;
382382
u64 reference_perf, delivered_perf;
383383

384-
reference_perf = fb_ctrs_t0.reference_perf;
384+
reference_perf = fb_ctrs_t0->reference_perf;
385385

386-
delta_reference = get_delta(fb_ctrs_t1.reference,
387-
fb_ctrs_t0.reference);
388-
delta_delivered = get_delta(fb_ctrs_t1.delivered,
389-
fb_ctrs_t0.delivered);
386+
delta_reference = get_delta(fb_ctrs_t1->reference,
387+
fb_ctrs_t0->reference);
388+
delta_delivered = get_delta(fb_ctrs_t1->delivered,
389+
fb_ctrs_t0->delivered);
390390

391391
/* Check to avoid divide-by zero */
392392
if (delta_reference || delta_delivered)
@@ -417,7 +417,7 @@ static unsigned int cppc_cpufreq_get_rate(unsigned int cpu)
417417
if (ret)
418418
return ret;
419419

420-
return cppc_get_rate_from_fbctrs(cpu_data, fb_ctrs_t0, fb_ctrs_t1);
420+
return cppc_get_rate_from_fbctrs(cpu_data, &fb_ctrs_t0, &fb_ctrs_t1);
421421
}
422422

423423
static int cppc_cpufreq_set_boost(struct cpufreq_policy *policy, int state)

0 commit comments

Comments
 (0)