summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2018-07-11 09:09:10 +0100
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>2018-07-18 13:29:11 +0300
commit309dca51c902d10267bbb120a8a2f3efc3491728 (patch)
tree671ebddbc09636c48c1da027596c6e4a835fa33f
parente78fd3c3eead7cb71c4b3bbcefbaa26a9345c0ee (diff)
downloadlinux-stable-309dca51c902d10267bbb120a8a2f3efc3491728.tar.gz
linux-stable-309dca51c902d10267bbb120a8a2f3efc3491728.tar.bz2
linux-stable-309dca51c902d10267bbb120a8a2f3efc3491728.zip
platform/x86: intel_ips: remove redundant variables slope and offset
Variables slope and offset are being assigned but are never used hence they are redundant and can be removed. Cleans up clang warnings: warning: variable 'slope' set but not used [-Wunused-but-set-variable] warning: variable 'offset' set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
-rw-r--r--drivers/platform/x86/intel_ips.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c
index 014fc1634a3d..c5ece7ef08c6 100644
--- a/drivers/platform/x86/intel_ips.c
+++ b/drivers/platform/x86/intel_ips.c
@@ -858,10 +858,7 @@ static u16 read_mgtv(struct ips_driver *ips)
static u16 read_ptv(struct ips_driver *ips)
{
- u16 val, slope, offset;
-
- slope = (ips->pta_val & PTA_SLOPE_MASK) >> PTA_SLOPE_SHIFT;
- offset = ips->pta_val & PTA_OFFSET_MASK;
+ u16 val;
val = thm_readw(THM_PTV) & PTV_MASK;