Skip to content

Commit

Permalink
usb: dwc3: ep0: don't use ep0in for transfers
Browse files Browse the repository at this point in the history
In "usb: dwc3: remove special status request handling in ep0" I simplied
a few things and used the generic API for the status transfers. The bug
I introcuded here is that we queue now requests to dep[1] but we don't
clear that list in the stall+start case.
Actually we don't need to use dep[1] at all. We only did in the past to
talk to the correct endpoint (i.e. in or out). This is now take care of
in a diffent place within the ep0 code. So we could queue the in
transfers to dep[0] and don't use dep[1] at all.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Felipe Balbi <balbi@ti.com>
  • Loading branch information
Sebastian Andrzej Siewior authored and Felipe Balbi committed Dec 12, 2011
1 parent 1e7618d commit c2da2ff
Showing 1 changed file with 8 additions and 10 deletions.
18 changes: 8 additions & 10 deletions drivers/usb/dwc3/ep0.c
Original file line number Diff line number Diff line change
Expand Up @@ -190,8 +190,7 @@ int dwc3_gadget_ep0_queue(struct usb_ep *ep, struct usb_request *request,
}

/* we share one TRB for ep0/1 */
if (!list_empty(&dwc->eps[0]->request_list) ||
!list_empty(&dwc->eps[1]->request_list)) {
if (!list_empty(&dep->request_list)) {
ret = -EBUSY;
goto out;
}
Expand All @@ -213,8 +212,8 @@ static void dwc3_ep0_stall_and_restart(struct dwc3 *dwc)
struct dwc3_ep *dep = dwc->eps[0];

/* stall is always issued on EP0 */
__dwc3_gadget_ep_set_halt(dwc->eps[0], 1);
dwc->eps[0]->flags = DWC3_EP_ENABLED;
__dwc3_gadget_ep_set_halt(dep, 1);
dep->flags = DWC3_EP_ENABLED;

if (!list_empty(&dep->request_list)) {
struct dwc3_request *req;
Expand Down Expand Up @@ -300,7 +299,7 @@ static int dwc3_ep0_handle_status(struct dwc3 *dwc, struct usb_ctrlrequest *ctrl
dwc->ep0_usb_req.length = sizeof(*response_pkt);
dwc->ep0_usb_req.dma = dwc->setup_buf_addr;
dwc->ep0_usb_req.complete = dwc3_ep0_status_cmpl;
return usb_ep_queue(&dwc->eps[1]->endpoint, &dwc->ep0_usb_req,
return usb_ep_queue(&dwc->eps[0]->endpoint, &dwc->ep0_usb_req,
GFP_ATOMIC);
}

Expand Down Expand Up @@ -552,22 +551,21 @@ static void dwc3_ep0_complete_data(struct dwc3 *dwc,
struct dwc3_request *r = NULL;
struct usb_request *ur;
struct dwc3_trb trb;
struct dwc3_ep *dep;
struct dwc3_ep *ep0;
u32 transferred;
u8 epnum;

epnum = event->endpoint_number;
dep = dwc->eps[epnum];
ep0 = dwc->eps[0];

dwc->ep0_next_event = DWC3_EP0_NRDY_STATUS;

r = next_request(&dwc->eps[0]->request_list);
r = next_request(&ep0->request_list);
ur = &r->request;

dwc3_trb_to_nat(dwc->ep0_trb, &trb);

if (dwc->ep0_bounced) {
struct dwc3_ep *ep0 = dwc->eps[0];

transferred = min_t(u32, ur->length,
ep0->endpoint.maxpacket - trb.length);
Expand All @@ -588,7 +586,7 @@ static void dwc3_ep0_complete_data(struct dwc3 *dwc,
* seems to be case when req.length > maxpacket. Could it be?
*/
if (r)
dwc3_gadget_giveback(dep, r, 0);
dwc3_gadget_giveback(ep0, r, 0);
}
}

Expand Down

0 comments on commit c2da2ff

Please sign in to comment.