[PATCH] input/serio:unneeded variable:delay

Subsystems: input (keyboard, mouse, joystick, touchscreen) drivers, the rest

STALE1800d

2 messages, 2 authors, 2021-09-04 · open the first message on its own page

[PATCH] input/serio:unneeded variable:delay

From: mateng <hidden>
Date: 2021-03-23 07:03:46

From: mateng <redacted>

delete unneeded variable 'delay'

Signed-off-by: mateng <redacted>
---
 drivers/input/serio/i8042.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/input/serio/i8042.c b/drivers/input/serio/i8042.c
index abae23a..145bda1 100644
--- a/drivers/input/serio/i8042.c
+++ b/drivers/input/serio/i8042.c
@@ -1126,7 +1126,6 @@ static void i8042_controller_reset(bool s2r_wants_reset)
 
 static long i8042_panic_blink(int state)
 {
-	long delay = 0;
 	char led;
 
 	led = (state) ? 0x01 | 0x04 : 0;
@@ -1142,7 +1141,7 @@ static long i8042_panic_blink(int state)
 	dbg("%02x -> i8042 (panic blink)\n", led);
 	i8042_write_data(led);
 	DELAY;
-	return delay;
+	return 0;
 }
 
 #undef DELAY
-- 
1.9.1

Re: [PATCH] input/serio:unneeded variable:delay

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2021-09-04 06:54:55

On Tue, Mar 23, 2021 at 03:02:50PM +0800, mateng wrote:
From: mateng <redacted>

delete unneeded variable 'delay'
Sorry, but this is bogus.
quoted hunk
Signed-off-by: mateng <redacted>
---
 drivers/input/serio/i8042.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/input/serio/i8042.c b/drivers/input/serio/i8042.c
index abae23a..145bda1 100644
--- a/drivers/input/serio/i8042.c
+++ b/drivers/input/serio/i8042.c
@@ -1126,7 +1126,6 @@ static void i8042_controller_reset(bool s2r_wants_reset)
 
 static long i8042_panic_blink(int state)
 {
-	long delay = 0;
 	char led;
 
 	led = (state) ? 0x01 | 0x04 : 0;
@@ -1142,7 +1141,7 @@ static long i8042_panic_blink(int state)
 	dbg("%02x -> i8042 (panic blink)\n", led);
 	i8042_write_data(led);
 	DELAY;
-	return delay;
+	return 0;
 }
 
 #undef DELAY
-- 
1.9.1
-- 
Dmitry
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help