From 102245fbed0b903e107880cb39a43bd8df757180 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Mon, 26 Oct 2015 19:48:25 +0100 Subject: libpayload: Avoid confusing usb debug output in dwc2 driver enqueue_packet already runs start_ep_transfer, which enqueues the next job. It's pretty much guaranteed that the port will look busy. BUG=none BRANCH=none TEST=no spurious ep 0-0 busy messages Change-Id: I9cbfa7b51dd37564262295ddbcdd0755da40c05b Signed-off-by: Patrick Georgi Original-Commit-Id: 8997dbd78dc363334f4e22eaa61f25de1449ffba Original-Change-Id: I8a39713fc1d6f16b80284e0f21dc95685716a9b7 Original-Signed-off-by: Patrick Georgi Original-Reviewed-on: https://chromium-review.googlesource.com/308763 Original-Commit-Ready: Patrick Georgi Original-Tested-by: Patrick Georgi Original-Reviewed-by: Furquan Shaikh Original-Reviewed-by: yunzhi li Reviewed-on: http://review.coreboot.org/12259 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- payloads/libpayload/drivers/udc/dwc2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/payloads/libpayload/drivers/udc/dwc2.c b/payloads/libpayload/drivers/udc/dwc2.c index 03691e79c6..32d6a5db9e 100644 --- a/payloads/libpayload/drivers/udc/dwc2.c +++ b/payloads/libpayload/drivers/udc/dwc2.c @@ -519,8 +519,8 @@ static void complete_ep_transfer(struct usbdev_ctrl *this, int endpoint, if (endpoint == 0 && job->xfered_length == 0) dwc2_enqueue_packet(this, 0, 0, p->setup_buf, 8, 0, 0); - - start_ep_transfer(p, endpoint, in_dir); + else + start_ep_transfer(p, endpoint, in_dir); } static void dwc2_outep_intr(struct usbdev_ctrl *this, dwc2_ep_t *ep) -- cgit v1.2.3