Commit da8c1c46 authored by Augusto Mecking Caringi's avatar Augusto Mecking Caringi Committed by Zhang Rui

thermal/intel_powerclamp: Remove set-but-not-used variables

In poll_pkg_cstate() function, the variables jiffies_last and
jiffies_now are set but never used.

This has been detected by building the driver with W=1:

drivers/thermal/intel_powerclamp.c: In function ‘poll_pkg_cstate’:
drivers/thermal/intel_powerclamp.c:464:23: warning: variable
‘jiffies_last’ set but not used [-Wunused-but-set-variable]
static unsigned long jiffies_last;
                       ^
Signed-off-by: default avatarAugusto Mecking Caringi <augustocaringi@gmail.com>
Acked-by: default avatarJacob Pan <jacob.jun.pan@linux.intel.com>
Signed-off-by: default avatarZhang Rui <rui.zhang@intel.com>
parent d5adbfcd
...@@ -461,16 +461,13 @@ static void poll_pkg_cstate(struct work_struct *dummy) ...@@ -461,16 +461,13 @@ static void poll_pkg_cstate(struct work_struct *dummy)
{ {
static u64 msr_last; static u64 msr_last;
static u64 tsc_last; static u64 tsc_last;
static unsigned long jiffies_last;
u64 msr_now; u64 msr_now;
unsigned long jiffies_now;
u64 tsc_now; u64 tsc_now;
u64 val64; u64 val64;
msr_now = pkg_state_counter(); msr_now = pkg_state_counter();
tsc_now = rdtsc(); tsc_now = rdtsc();
jiffies_now = jiffies;
/* calculate pkg cstate vs tsc ratio */ /* calculate pkg cstate vs tsc ratio */
if (!msr_last || !tsc_last) if (!msr_last || !tsc_last)
...@@ -485,7 +482,6 @@ static void poll_pkg_cstate(struct work_struct *dummy) ...@@ -485,7 +482,6 @@ static void poll_pkg_cstate(struct work_struct *dummy)
/* update record */ /* update record */
msr_last = msr_now; msr_last = msr_now;
jiffies_last = jiffies_now;
tsc_last = tsc_now; tsc_last = tsc_now;
if (true == clamping) if (true == clamping)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment