aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSachin Kamat <sachin.kamat@linaro.org>2012-08-22 06:19:01 -0400
committerFelipe Balbi <balbi@ti.com>2012-08-31 05:41:55 -0400
commit1660c8944652aac142e622d276c3750f735a568f (patch)
tree653992abb8932b627df79d6c9f1b3b1e55726dce
parentff24166c3996a343f3901a47bd5fef6181ac21f9 (diff)
usb: gadget: s3c2410_udc: Move assignment outside if condition
Fixes the following checkpatch errors: ERROR: do not use assignment in if condition Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by: Felipe Balbi <balbi@ti.com>
-rw-r--r--drivers/usb/gadget/s3c2410_udc.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/usb/gadget/s3c2410_udc.c b/drivers/usb/gadget/s3c2410_udc.c
index d9e3a21b3ddc..e895ea12b67e 100644
--- a/drivers/usb/gadget/s3c2410_udc.c
+++ b/drivers/usb/gadget/s3c2410_udc.c
@@ -1696,7 +1696,8 @@ static int s3c2410_udc_start(struct usb_gadget_driver *driver,
1696 udc->gadget.dev.driver = &driver->driver; 1696 udc->gadget.dev.driver = &driver->driver;
1697 1697
1698 /* Bind the driver */ 1698 /* Bind the driver */
1699 if ((retval = device_add(&udc->gadget.dev)) != 0) { 1699 retval = device_add(&udc->gadget.dev);
1700 if (retval) {
1700 dev_err(&udc->gadget.dev, "Error in device_add() : %d\n", retval); 1701 dev_err(&udc->gadget.dev, "Error in device_add() : %d\n", retval);
1701 goto register_error; 1702 goto register_error;
1702 } 1703 }
@@ -1704,7 +1705,8 @@ static int s3c2410_udc_start(struct usb_gadget_driver *driver,
1704 dprintk(DEBUG_NORMAL, "binding gadget driver '%s'\n", 1705 dprintk(DEBUG_NORMAL, "binding gadget driver '%s'\n",
1705 driver->driver.name); 1706 driver->driver.name);
1706 1707
1707 if ((retval = bind(&udc->gadget)) != 0) { 1708 retval = bind(&udc->gadget);
1709 if (retval) {
1708 device_del(&udc->gadget.dev); 1710 device_del(&udc->gadget.dev);
1709 goto register_error; 1711 goto register_error;
1710 } 1712 }