Input: ucb1400_ts - remove redundant variable penup
authorColin Ian King <colin.i.king@gmail.com>
Mon, 13 Dec 2021 05:20:31 +0000 (21:20 -0800)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Mon, 20 Dec 2021 07:55:25 +0000 (23:55 -0800)
Variable penup is assigned a value but penup is never read later, it
is redundant and can be removed.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20211205000525.153999-1-colin.i.king@gmail.com
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/ucb1400_ts.c

index e3f2c94..dfd3b35 100644 (file)
@@ -186,7 +186,6 @@ static irqreturn_t ucb1400_irq(int irqnr, void *devid)
 {
        struct ucb1400_ts *ucb = devid;
        unsigned int x, y, p;
-       bool penup;
 
        if (unlikely(irqnr != ucb->irq))
                return IRQ_NONE;
@@ -196,8 +195,7 @@ static irqreturn_t ucb1400_irq(int irqnr, void *devid)
        /* Start with a small delay before checking pendown state */
        msleep(UCB1400_TS_POLL_PERIOD);
 
-       while (!ucb->stopped && !(penup = ucb1400_ts_pen_up(ucb))) {
-
+       while (!ucb->stopped && !ucb1400_ts_pen_up(ucb)) {
                ucb1400_adc_enable(ucb->ac97);
                x = ucb1400_ts_read_xpos(ucb);
                y = ucb1400_ts_read_ypos(ucb);