Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a91e5519 authored by qctecmdr's avatar qctecmdr Committed by Gerrit - the friendly Code Review server
Browse files

Merge "usb: dwc3: gadget: Remove unnecessary checks"

parents b84282ae 5d1c8912
Loading
Loading
Loading
Loading
+5 −8
Original line number Original line Diff line number Diff line
@@ -1703,9 +1703,6 @@ static int dwc3_gadget_ep_dequeue(struct usb_ep *ep,
			/* wait until it is processed */
			/* wait until it is processed */
			dwc3_stop_active_transfer(dep, true, true);
			dwc3_stop_active_transfer(dep, true, true);


			if (!r->trb)
				goto out0;

			/*
			/*
			 * Remove any started request if the transfer is
			 * Remove any started request if the transfer is
			 * cancelled.
			 * cancelled.
@@ -1713,10 +1710,7 @@ static int dwc3_gadget_ep_dequeue(struct usb_ep *ep,
			list_for_each_entry_safe(r, t, &dep->started_list, list)
			list_for_each_entry_safe(r, t, &dep->started_list, list)
				dwc3_gadget_move_cancelled_request(r);
				dwc3_gadget_move_cancelled_request(r);


			if (dep->flags & DWC3_EP_TRANSFER_STARTED)
			goto out0;
			goto out0;
			else
				goto out1;
		}
		}
		dev_err(dwc->dev, "request %pK was not queued to %s\n",
		dev_err(dwc->dev, "request %pK was not queued to %s\n",
				request, ep->name);
				request, ep->name);
@@ -1724,7 +1718,6 @@ static int dwc3_gadget_ep_dequeue(struct usb_ep *ep,
		goto out0;
		goto out0;
	}
	}


out1:
	dbg_ep_dequeue(dep->number, req);
	dbg_ep_dequeue(dep->number, req);
	dwc3_gadget_giveback(dep, req, -ECONNRESET);
	dwc3_gadget_giveback(dep, req, -ECONNRESET);


@@ -2932,9 +2925,13 @@ static void dwc3_gadget_endpoint_transfer_in_progress(struct dwc3_ep *dep,


	dwc3_gadget_ep_cleanup_completed_requests(dep, event, status);
	dwc3_gadget_ep_cleanup_completed_requests(dep, event, status);


	if (dep->flags & DWC3_EP_END_TRANSFER_PENDING)
		goto out;

	if (stop)
	if (stop)
		dwc3_stop_active_transfer(dep, true, true);
		dwc3_stop_active_transfer(dep, true, true);


out:
	/*
	/*
	 * WORKAROUND: This is the 2nd half of U1/U2 -> U0 workaround.
	 * WORKAROUND: This is the 2nd half of U1/U2 -> U0 workaround.
	 * See dwc3_gadget_linksts_change_interrupt() for 1st half.
	 * See dwc3_gadget_linksts_change_interrupt() for 1st half.