From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Date: Sat, 7 May 2016 00:24:48 +0000 (+0200)
Subject: intel_pstate: Clean up intel_pstate_get()
X-Git-Tag: v4.7-rc2~10^2^2~14
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f96fd0c86f081c27a0ba5efadfc13f612b4071d3;p=platform%2Fkernel%2Flinux-exynos.git

intel_pstate: Clean up intel_pstate_get()

intel_pstate_get() contains a local variable that's initialized but
never used and it can be written in fewer lines of code, so clean
it up.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
---

diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
index 3addb22..cde1a4e 100644
--- a/drivers/cpufreq/intel_pstate.c
+++ b/drivers/cpufreq/intel_pstate.c
@@ -1404,14 +1404,9 @@ static int intel_pstate_init_cpu(unsigned int cpunum)
 
 static unsigned int intel_pstate_get(unsigned int cpu_num)
 {
-	struct sample *sample;
-	struct cpudata *cpu;
+	struct cpudata *cpu = all_cpu_data[cpu_num];
 
-	cpu = all_cpu_data[cpu_num];
-	if (!cpu)
-		return 0;
-	sample = &cpu->sample;
-	return get_avg_frequency(cpu);
+	return cpu ? get_avg_frequency(cpu) : 0;
 }
 
 static void intel_pstate_set_update_util_hook(unsigned int cpu_num)