diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c index 17340864a540..df6d5096f5b4 100644 --- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -1513,10 +1513,6 @@ static void dwc3_gadget_ep_skip_trbs(struct dwc3_ep *dep, struct dwc3_request *r { int i; - /* If req->trb is not set, then the request has not started */ - if (!req->trb) - return; - /* * If request was already started, this means we had to * stop the transfer. With that we also need to ignore @@ -1607,8 +1603,6 @@ int __dwc3_gadget_ep_set_halt(struct dwc3_ep *dep, int value, int protocol) { struct dwc3_gadget_ep_cmd_params params; struct dwc3 *dwc = dep->dwc; - struct dwc3_request *req; - struct dwc3_request *tmp; int ret; if (usb_endpoint_xfer_isoc(dep->endpoint.desc)) { @@ -1645,37 +1639,13 @@ int __dwc3_gadget_ep_set_halt(struct dwc3_ep *dep, int value, int protocol) else dep->flags |= DWC3_EP_STALL; } else { - /* - * Don't issue CLEAR_STALL command to control endpoints. The - * controller automatically clears the STALL when it receives - * the SETUP token. - */ - if (dep->number <= 1) { - dep->flags &= ~(DWC3_EP_STALL | DWC3_EP_WEDGE); - return 0; - } ret = dwc3_send_clear_stall_ep_cmd(dep); - if (ret) { + if (ret) dev_err(dwc->dev, "failed to clear STALL on %s\n", dep->name); - return ret; - } - - dep->flags &= ~(DWC3_EP_STALL | DWC3_EP_WEDGE); - - dwc3_stop_active_transfer(dep, true, true); - - list_for_each_entry_safe(req, tmp, &dep->started_list, list) - dwc3_gadget_move_cancelled_request(req); - - list_for_each_entry_safe(req, tmp, &dep->pending_list, list) - dwc3_gadget_move_cancelled_request(req); - - if (!(dep->flags & DWC3_EP_END_TRANSFER_PENDING)) { - dep->flags &= ~DWC3_EP_DELAY_START; - dwc3_gadget_ep_cleanup_cancelled_requests(dep); - } + else + dep->flags &= ~(DWC3_EP_STALL | DWC3_EP_WEDGE); } return ret;