Implement the driver's set_wedge() method by adding the 'wedged' flag
to the 'struct musb_ep'.

Signed-off-by: Sergei Shtylyov <[email protected]>

---
The patch is against the recent Linus' tree.

This is probably 2.6.33 material...

 drivers/usb/musb/musb_gadget.c     |   21 +++++++++++++++++++--
 drivers/usb/musb/musb_gadget.h     |    2 ++
 drivers/usb/musb/musb_gadget_ep0.c |    6 +++++-
 3 files changed, 26 insertions(+), 3 deletions(-)

Index: linux-2.6/drivers/usb/musb/musb_gadget.c
===================================================================
--- linux-2.6.orig/drivers/usb/musb/musb_gadget.c
+++ linux-2.6/drivers/usb/musb/musb_gadget.c
@@ -965,7 +965,7 @@ static int musb_gadget_enable(struct usb
                musb_ep->dma = NULL;
 
        musb_ep->desc = desc;
-       musb_ep->busy = 0;
+       musb_ep->wedged = musb_ep->busy = 0;
        status = 0;
 
        pr_debug("%s periph: enabled %s for %s %s, %smaxpacket %d\n",
@@ -1262,7 +1262,8 @@ int musb_gadget_set_halt(struct usb_ep *
                                goto done;
                        }
                }
-       }
+       } else
+               musb_ep->wedged = 0;
 
        /* set/clear the stall and toggle bits */
        DBG(2, "%s: %s stall\n", ep->name, value ? "set" : "clear");
@@ -1301,6 +1302,21 @@ done:
        return status;
 }
 
+/*
+ * Sets the halt feature with the clear requests ignored
+ */
+int musb_gadget_set_wedge(struct usb_ep *ep)
+{
+       struct musb_ep          *musb_ep = to_musb_ep(ep);
+
+       if (!ep)
+               return -EINVAL;
+
+       musb_ep->wedged = 1;
+
+       return usb_ep_set_halt(ep);
+}
+
 static int musb_gadget_fifo_status(struct usb_ep *ep)
 {
        struct musb_ep          *musb_ep = to_musb_ep(ep);
@@ -1371,6 +1387,7 @@ static const struct usb_ep_ops musb_ep_o
        .queue          = musb_gadget_queue,
        .dequeue        = musb_gadget_dequeue,
        .set_halt       = musb_gadget_set_halt,
+       .set_wedge      = musb_gadget_set_wedge,
        .fifo_status    = musb_gadget_fifo_status,
        .fifo_flush     = musb_gadget_fifo_flush
 };
Index: linux-2.6/drivers/usb/musb/musb_gadget.h
===================================================================
--- linux-2.6.orig/drivers/usb/musb/musb_gadget.h
+++ linux-2.6/drivers/usb/musb/musb_gadget.h
@@ -75,6 +75,8 @@ struct musb_ep {
        /* later things are modified based on usage */
        struct list_head                req_list;
 
+       u8                              wedged;
+
        /* true if lock must be dropped but req_list may not be advanced */
        u8                              busy;
 };
Index: linux-2.6/drivers/usb/musb/musb_gadget_ep0.c
===================================================================
--- linux-2.6.orig/drivers/usb/musb/musb_gadget_ep0.c
+++ linux-2.6/drivers/usb/musb/musb_gadget_ep0.c
@@ -273,6 +273,11 @@ __acquires(musb->lock)
                                if (!musb_ep->desc)
                                        break;
 
+                               handled = 1;
+                               /* Ignore request if endpoint is wedged */
+                               if (musb_ep->wedged)
+                                       break;
+
                                /* REVISIT do it directly, no locking games */
                                spin_unlock(&musb->lock);
                                musb_gadget_set_halt(&musb_ep->end_point, 0);
@@ -280,7 +285,6 @@ __acquires(musb->lock)
 
                                /* select ep0 again */
                                musb_ep_select(mbase, 0);
-                               handled = 1;
                                } break;
                        default:
                                /* class, vendor, etc ... delegate */


_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to