From: Evgeny Budilovsky Date: Fri, 20 Jun 2014 08:22:14 +0000 (+0300) Subject: staging/cptm1217: fix checkpatch warnings about else clause X-Git-Tag: firefly_0821_release~176^2~3491^2~1670 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=55a4ea71ca4225f723cca1c44da569bd4de9d739;p=firefly-linux-kernel-4.4.55.git staging/cptm1217: fix checkpatch warnings about else clause Signed-off-by: Evgeny Budilovsky Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/cptm1217/clearpad_tm1217.c b/drivers/staging/cptm1217/clearpad_tm1217.c index ca4c2c67dd88..edf9ff2ea25b 100644 --- a/drivers/staging/cptm1217/clearpad_tm1217.c +++ b/drivers/staging/cptm1217/clearpad_tm1217.c @@ -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);