[patch 1/3] staging: dwc2: cleanups in dwc2_hcd_qh_add()

[Dan Carpenter]

I've made the success and error paths clearer and pulled some code in
one indent level.

Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>

diff --git a/drivers/staging/dwc2/hcd_queue.c b/drivers/staging/dwc2/hcd_queue.c
index f200f1f6e1c6..ad9136e9d820 100644
--- a/drivers/staging/dwc2/hcd_queue.c
+++ b/drivers/staging/dwc2/hcd_queue.c
@@ -609,7 +609,7 @@ static void dwc2_deschedule_periodic(struct dwc2_hsotg 
*hsotg,
  */
 int dwc2_hcd_qh_add(struct dwc2_hsotg *hsotg, struct dwc2_qh *qh)
 {
-       int status = 0;
+       int status;
        u32 intr_mask;
 
        if (dbg_qh(qh))
@@ -617,7 +617,7 @@ int dwc2_hcd_qh_add(struct dwc2_hsotg *hsotg, struct 
dwc2_qh *qh)
 
        if (!list_empty(&qh->qh_list_entry))
                /* QH already in a schedule */
-               return status;
+               return 0;
 
        /* Add the new QH to the appropriate schedule */
        if (dwc2_qh_is_non_per(qh)) {
@@ -626,17 +626,17 @@ int dwc2_hcd_qh_add(struct dwc2_hsotg *hsotg, struct 
dwc2_qh *qh)
                              &hsotg->non_periodic_sched_inactive);
        } else {
                status = dwc2_schedule_periodic(hsotg, qh);
-               if (status == 0) {
-                       if (!hsotg->periodic_qh_count) {
-                               intr_mask = readl(hsotg->regs + GINTMSK);
-                               intr_mask |= GINTSTS_SOF;
-                               writel(intr_mask, hsotg->regs + GINTMSK);
-                       }
-                       hsotg->periodic_qh_count++;
+               if (status)
+                       return status;
+               if (!hsotg->periodic_qh_count) {
+                       intr_mask = readl(hsotg->regs + GINTMSK);
+                       intr_mask |= GINTSTS_SOF;
+                       writel(intr_mask, hsotg->regs + GINTMSK);
                }
+               hsotg->periodic_qh_count++;
        }
 
-       return status;
+       return 0;
 }
 
 /**
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel