usb: gadget: ci_udc: implement usb_ep_ops dequeue callback
authorPeng Fan <Peng.Fan@freescale.com>
Fri, 28 Aug 2015 01:20:30 +0000 (09:20 +0800)
committerMarek Vasut <marex@denx.de>
Sat, 12 Sep 2015 18:24:42 +0000 (20:24 +0200)
Implement endpoint dequeue callback function.

Without this function, uboot will hang when executing fastboot comamnd.
See following flow:
"fastboot_tx_write_str->fastboot_tx_write->usb_ep_dequeue->ep->ops->dequeue"
without implement ci_udc dequeue function, ep->ops->dequeue is NULL, then
uboot will hang.

Tested on mx6qsabresd board with fastboot enabled.

Signed-off-by: Peng Fan <Peng.Fan@freescale.com>
Tested-by: Fabio Estevam <fabio.estevam@freescale.com>
Cc: "Ɓukasz Majewski" <l.majewski@samsung.com>
Cc: Marek Vasut <marex@denx.de>
drivers/usb/gadget/ci_udc.c

index 3e8eb8799f4e0227fc2fa24c3d9716175d786d5e..c0d23a444bf251bb7af90eb48c8153ad25c8b4ff 100644 (file)
@@ -87,6 +87,7 @@ static int ci_ep_enable(struct usb_ep *ep,
 static int ci_ep_disable(struct usb_ep *ep);
 static int ci_ep_queue(struct usb_ep *ep,
                struct usb_request *req, gfp_t gfp_flags);
+static int ci_ep_dequeue(struct usb_ep *ep, struct usb_request *req);
 static struct usb_request *
 ci_ep_alloc_request(struct usb_ep *ep, unsigned int gfp_flags);
 static void ci_ep_free_request(struct usb_ep *ep, struct usb_request *_req);
@@ -99,6 +100,7 @@ static struct usb_ep_ops ci_ep_ops = {
        .enable         = ci_ep_enable,
        .disable        = ci_ep_disable,
        .queue          = ci_ep_queue,
+       .dequeue        = ci_ep_dequeue,
        .alloc_request  = ci_ep_alloc_request,
        .free_request   = ci_ep_free_request,
 };
@@ -525,6 +527,30 @@ static void ci_ep_submit_next_request(struct ci_ep *ci_ep)
        writel(bit, &udc->epprime);
 }
 
+static int ci_ep_dequeue(struct usb_ep *_ep, struct usb_request *_req)
+{
+       struct ci_ep *ci_ep = container_of(_ep, struct ci_ep, ep);
+       struct ci_req *ci_req;
+
+       list_for_each_entry(ci_req, &ci_ep->queue, queue) {
+               if (&ci_req->req == _req)
+                       break;
+       }
+
+       if (&ci_req->req != _req)
+               return -EINVAL;
+
+       list_del_init(&ci_req->queue);
+
+       if (ci_req->req.status == -EINPROGRESS) {
+               ci_req->req.status = -ECONNRESET;
+               if (ci_req->req.complete)
+                       ci_req->req.complete(_ep, _req);
+       }
+
+       return 0;
+}
+
 static int ci_ep_queue(struct usb_ep *ep,
                struct usb_request *req, gfp_t gfp_flags)
 {