[PATCH v3 06/12] staging: ced1401: ced_ioc.c fix checkpatch warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Fix checkpatch warning "else is not generally useful after a break or return" in
file ced_ioc.c, function ced_wait_event()

Signed-off-by: Luca Ellero <luca.ellero@xxxxxxxxxxxxxxxx>
---
 drivers/staging/ced1401/ced_ioc.c |   65 +++++++++++++++++++------------------
 1 file changed, 33 insertions(+), 32 deletions(-)

diff --git a/drivers/staging/ced1401/ced_ioc.c b/drivers/staging/ced1401/ced_ioc.c
index cb075af..0a40246 100644
--- a/drivers/staging/ced1401/ced_ioc.c
+++ b/drivers/staging/ced1401/ced_ioc.c
@@ -887,48 +887,49 @@ int ced_set_event(struct ced_data *ced, struct transfer_event __user *ute)
 int ced_wait_event(struct ced_data *ced, int area, int time_out)
 {
 	int ret;
+	int wait;
+	struct transarea *ta;
 
 	if ((unsigned)area >= MAX_TRANSAREAS)
 		return U14ERR_BADAREA;
-	else {
-		int wait;
-		struct transarea *ta = &ced->trans_def[area];
 
-		 /* convert timeout to jiffies */
-		time_out = (time_out * HZ + 999) / 1000;
+	ta = &ced->trans_def[area];
 
-		/* We cannot wait holding the mutex, but we check the flags  */
-		/* while holding it. This may well be pointless as another   */
-		/* thread could get in between releasing it and the wait     */
-		/* call. However, this would have to clear the wake_up flag. */
-		/* However, the !ta->used may help us in this case.	     */
+	 /* convert timeout to jiffies */
+	time_out = (time_out * HZ + 999) / 1000;
 
-		/* make sure we have no competitor */
-		mutex_lock(&ced->io_mutex);
-		if (!ta->used || !ta->event_sz) /* check something to */
-						  /* wait for...        */
-			return U14ERR_NOTSET;	/*  ...else we do nothing */
-		mutex_unlock(&ced->io_mutex);
+	/* We cannot wait holding the mutex, but we check the flags  */
+	/* while holding it. This may well be pointless as another   */
+	/* thread could get in between releasing it and the wait     */
+	/* call. However, this would have to clear the wake_up flag. */
+	/* However, the !ta->used may help us in this case.	     */
 
-		if (time_out)
-			wait = wait_event_interruptible_timeout(ta->event,
-								ta->wake_up ||
-								!ta->used,
-								time_out);
-		else
-			wait = wait_event_interruptible(ta->event,
+	/* make sure we have no competitor */
+	mutex_lock(&ced->io_mutex);
+	if (!ta->used || !ta->event_sz) /* check something to */
+					  /* wait for...        */
+		return U14ERR_NOTSET;	/*  ...else we do nothing */
+	mutex_unlock(&ced->io_mutex);
+
+	if (time_out)
+		wait = wait_event_interruptible_timeout(ta->event,
 							ta->wake_up ||
-							!ta->used);
+							!ta->used,
+							time_out);
+	else
+		wait = wait_event_interruptible(ta->event,
+						ta->wake_up ||
+						!ta->used);
+
+	if (wait)
+		ret = -ERESTARTSYS; /* oops - we have had a SIGNAL */
+	else
+		ret = ta->wake_up; /* else the wakeup count */
 
-		if (wait)
-			ret = -ERESTARTSYS; /* oops - we have had a SIGNAL */
-		else
-			ret = ta->wake_up; /* else the wakeup count */
+	spin_lock_irq(&ced->staged_lock);
+	ta->wake_up = 0;	/*  clear the flag */
+	spin_unlock_irq(&ced->staged_lock);
 
-		spin_lock_irq(&ced->staged_lock);
-		ta->wake_up = 0;	/*  clear the flag */
-		spin_unlock_irq(&ced->staged_lock);
-	}
 	return ret;
 }
 
-- 
1.7.10.4

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux