[PATCH v2 05/28] usb: dwc2: Fix brace usage

[John Youn]

* Remove braces for one-line statements
* Add missing braces where another arm in if-statement uses braces

Signed-off-by: John Youn <johny...@synopsys.com>
---
 drivers/usb/dwc2/gadget.c   | 10 ++++------
 drivers/usb/dwc2/platform.c | 12 ++++++------
 2 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c
index 2049afb1ffb1..dd839780da35 100644
--- a/drivers/usb/dwc2/gadget.c
+++ b/drivers/usb/dwc2/gadget.c
@@ -1644,9 +1644,8 @@ static int dwc2_hsotg_process_req_feature(struct 
dwc2_hsotg *hsotg,
                                }
 
                                /* If we have pending request, then start it */
-                               if (!ep->req) {
+                               if (!ep->req)
                                        dwc2_gadget_start_next_request(ep);
-                               }
                        }
 
                        break;
@@ -1938,9 +1937,8 @@ static void dwc2_hsotg_complete_request(struct dwc2_hsotg 
*hsotg,
         * so be careful when doing this.
         */
 
-       if (!hs_ep->req && result >= 0) {
+       if (!hs_ep->req && result >= 0)
                dwc2_gadget_start_next_request(hs_ep);
-       }
 }
 
 /*
@@ -4043,9 +4041,9 @@ static int dwc2_hsotg_ep_sethalt(struct usb_ep *ep, int 
value, bool now)
                epreg = DOEPCTL(index);
                epctl = dwc2_readl(hs->regs + epreg);
 
-               if (value)
+               if (value) {
                        epctl |= DXEPCTL_STALL;
-               else {
+               } else {
                        epctl &= ~DXEPCTL_STALL;
                        xfertype = epctl & DXEPCTL_EPTYPE_MASK;
                        if (xfertype == DXEPCTL_EPTYPE_BULK ||
diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c
index 649d7b9a714b..3f59a73de248 100644
--- a/drivers/usb/dwc2/platform.c
+++ b/drivers/usb/dwc2/platform.c
@@ -136,11 +136,11 @@ static int __dwc2_lowlevel_hw_enable(struct dwc2_hsotg 
*hsotg)
                        return ret;
        }
 
-       if (hsotg->uphy)
+       if (hsotg->uphy) {
                ret = usb_phy_init(hsotg->uphy);
-       else if (hsotg->plat && hsotg->plat->phy_init)
+       } else if (hsotg->plat && hsotg->plat->phy_init) {
                ret = hsotg->plat->phy_init(pdev, hsotg->plat->phy_type);
-       else {
+       } else {
                ret = phy_power_on(hsotg->phy);
                if (ret == 0)
                        ret = phy_init(hsotg->phy);
@@ -170,11 +170,11 @@ static int __dwc2_lowlevel_hw_disable(struct dwc2_hsotg 
*hsotg)
        struct platform_device *pdev = to_platform_device(hsotg->dev);
        int ret = 0;
 
-       if (hsotg->uphy)
+       if (hsotg->uphy) {
                usb_phy_shutdown(hsotg->uphy);
-       else if (hsotg->plat && hsotg->plat->phy_exit)
+       } else if (hsotg->plat && hsotg->plat->phy_exit) {
                ret = hsotg->plat->phy_exit(pdev, hsotg->plat->phy_type);
-       else {
+       } else {
                ret = phy_exit(hsotg->phy);
                if (ret == 0)
                        ret = phy_power_off(hsotg->phy);
-- 
2.11.0

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html