staging/cptm1217: fix checkpatch warnings about else clause
authorEvgeny Budilovsky <budevg@gmail.com>
Fri, 20 Jun 2014 08:22:14 +0000 (11:22 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 20 Jun 2014 15:28:01 +0000 (08:28 -0700)
Signed-off-by: Evgeny Budilovsky <budevg@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/cptm1217/clearpad_tm1217.c

index ca4c2c67dd880d4253909850a7811badcc24b7b9..edf9ff2ea25b32020161b92dc8f1df4edd4d217b 100644 (file)
@@ -147,12 +147,11 @@ static int cp_tm1217_read(struct cp_tm1217_device *ts,
        msleep(WAIT_FOR_RESPONSE);
        for (i = 0; i < MAX_RETRIES; i++) {
                retval = i2c_master_recv(ts->client, &req[1], size);
-               if (retval == size) {
+               if (retval == size)
                        break;
-               } else {
-                       msleep(INCREMENTAL_DELAY);
-                       dev_dbg(ts->dev, "cp_tm1217: Retry count is %d\n", i);
-               }
+
+               msleep(INCREMENTAL_DELAY);
+               dev_dbg(ts->dev, "cp_tm1217: Retry count is %d\n", i);
        }
        if (retval != size)
                dev_err(ts->dev, "cp_tm1217: Read from device failed\n");
@@ -288,11 +287,11 @@ static irqreturn_t cp_tm1217_sample_thread(int irq, void *handle)
        if (ts->thread_running == 1) {
                mutex_unlock(&ts->thread_mutex);
                return IRQ_HANDLED;
-       } else {
-               ts->thread_running = 1;
-               mutex_unlock(&ts->thread_mutex);
        }
 
+       ts->thread_running = 1;
+       mutex_unlock(&ts->thread_mutex);
+
        /* Mask the interrupts */
        retval = cp_tm1217_mask_interrupt(ts);