summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2018-01-18 11:22:11 -0800
committerDmitry Torokhov <dmitry.torokhov@gmail.com>2018-01-18 11:39:49 -0800
commitdeb106beb7efebd43a98d0fa84b3709a862da20c (patch)
tree2978c55dd62a24d578098e686cdb4ff2c0d088ed
parent0de45027257bbae71a78240af4903d08447a669c (diff)
downloadop-kernel-dev-deb106beb7efebd43a98d0fa84b3709a862da20c.zip
op-kernel-dev-deb106beb7efebd43a98d0fa84b3709a862da20c.tar.gz
Input: cyapa - remove redundant assignment to 'pwr_cmd'
The variable pwr_cmd is being assigned to cyapa->suspend_power_mode twice, once during the declaration and once after taking an interruptible mutex lock. Remove the redundant first assignment since the value is never read and it is outside the mutex lock. Cleans up clang warning: drivers/input/mouse/cyapa.c:743:5: warning: Value stored to 'pwr_cmd' during its initialization is never read Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
-rw-r--r--drivers/input/mouse/cyapa.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/input/mouse/cyapa.c b/drivers/input/mouse/cyapa.c
index fd8865c..dfd3873 100644
--- a/drivers/input/mouse/cyapa.c
+++ b/drivers/input/mouse/cyapa.c
@@ -740,7 +740,7 @@ static ssize_t cyapa_show_suspend_scanrate(struct device *dev,
char *buf)
{
struct cyapa *cyapa = dev_get_drvdata(dev);
- u8 pwr_cmd = cyapa->suspend_power_mode;
+ u8 pwr_cmd;
u16 sleep_time;
int len;
int error;
OpenPOWER on IntegriCloud