[PATCH] Input: adi: remove redundant variable z

Colin Ian King posted 1 patch 4 years, 3 months ago
drivers/input/joystick/adi.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH] Input: adi: remove redundant variable z
Posted by Colin Ian King 4 years, 3 months ago
Variable z is being assigned a value that is never read, the
variable is redundant and can be removed.

Cleans up clang scan build warning:
drivers/input/joystick/adi.c:139:6: warning: Although the
value stored to 'z' is used in the enclosing expression,
the value is never actually read from 'z' [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
 drivers/input/joystick/adi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/input/joystick/adi.c b/drivers/input/joystick/adi.c
index 592c95b87f54..e10d57bf1180 100644
--- a/drivers/input/joystick/adi.c
+++ b/drivers/input/joystick/adi.c
@@ -123,7 +123,7 @@ static void adi_read_packet(struct adi_port *port)
 {
 	struct adi *adi = port->adi;
 	struct gameport *gameport = port->gameport;
-	unsigned char u, v, w, x, z;
+	unsigned char u, v, w, x;
 	int t[2], s[2], i;
 	unsigned long flags;
 
@@ -136,7 +136,7 @@ static void adi_read_packet(struct adi_port *port)
 	local_irq_save(flags);
 
 	gameport_trigger(gameport);
-	v = z = gameport_read(gameport);
+	v = gameport_read(gameport);
 
 	do {
 		u = v;
-- 
2.35.1
Re: [PATCH] Input: adi: remove redundant variable z
Posted by Dmitry Torokhov 4 years, 3 months ago
On Fri, Mar 18, 2022 at 12:23:18AM +0000, Colin Ian King wrote:
> Variable z is being assigned a value that is never read, the
> variable is redundant and can be removed.
> 
> Cleans up clang scan build warning:
> drivers/input/joystick/adi.c:139:6: warning: Although the
> value stored to 'z' is used in the enclosing expression,
> the value is never actually read from 'z' [deadcode.DeadStores]
> 
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>

Applied, thank you.

-- 
Dmitry