From: Colin Ian King <colin.king@canonical.com>
Date: Wed, 18 Oct 2017 12:10:38 +0000 (+0100)
Subject: hwmon: (asc7621) remove redundant assignment to newval
X-Git-Tag: v4.19~2260^2~5
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a3bdc5b5bd369fcab8306b80d2c740e332183d20;p=platform%2Fkernel%2Flinux-rpi.git

hwmon: (asc7621) remove redundant assignment to newval

The setting of newval to zero is redundant as the following if/else
stanzas will always update newval to a new value. Remove the
redundant setting, cleans up clang build warning:

drivers/hwmon/asc7621.c:582:2: warning: Value stored to 'newval' is
never read

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---

diff --git a/drivers/hwmon/asc7621.c b/drivers/hwmon/asc7621.c
index 4875e99..6d34c05 100644
--- a/drivers/hwmon/asc7621.c
+++ b/drivers/hwmon/asc7621.c
@@ -579,7 +579,6 @@ static ssize_t show_pwm_enable(struct device *dev,
 	mutex_unlock(&data->update_lock);
 
 	val = config | (altbit << 3);
-	newval = 0;
 
 	if (val == 3 || val >= 10)
 		newval = 255;