If we try to modprobe a second gadget driver before
rmmoding the first one, the reference for the first
gadget driver would get NULLed avoiding usb to change
gadget drivers later.

Cc: Greg Kroah-Hartman <[EMAIL PROTECTED]>
Cc: David Brownell <[EMAIL PROTECTED]>
Cc: Tony Lindgren <[EMAIL PROTECTED]>
Signed-off-by: Felipe Balbi <[EMAIL PROTECTED]>
---
 drivers/usb/musb/musb_gadget.c |   16 +++++++---------
 1 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/drivers/usb/musb/musb_gadget.c b/drivers/usb/musb/musb_gadget.c
index b3773f1..d6a802c 100644
--- a/drivers/usb/musb/musb_gadget.c
+++ b/drivers/usb/musb/musb_gadget.c
@@ -1710,17 +1710,15 @@ int usb_gadget_register_driver(struct usb_gadget_driver 
*driver)
 
        spin_unlock_irqrestore(&musb->lock, flags);
 
-       if (retval == 0)
+       if (retval == 0) {
                retval = driver->bind(&musb->g);
-       if (retval != 0) {
-               DBG(3, "bind to driver %s failed --> %d\n",
-                       driver->driver.name, retval);
-               musb->gadget_driver = NULL;
-               musb->g.dev.driver = NULL;
-       }
+               if (retval != 0) {
+                       DBG(3, "bind to driver %s failed --> %d\n",
+                                       driver->driver.name, retval);
+                       musb->gadget_driver = NULL;
+                       musb->g.dev.driver = NULL;
+               }
 
-       /* start peripheral and/or OTG engines */
-       if (retval == 0) {
                spin_lock_irqsave(&musb->lock, flags);
 
                /* REVISIT always use otg_set_peripheral(), handling
-- 
1.6.0.rc1.11.g1ce47

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to