Commit bbfc3420 authored by Thomas Pugliese's avatar Thomas Pugliese Committed by Greg Kroah-Hartman

usb: wusbcore: add more info to debug prints in urb_unlink path

Add more info to debug prints in urb_unlink path
Signed-off-by: default avatarThomas Pugliese <thomas.pugliese@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent b374487e
...@@ -373,10 +373,10 @@ static unsigned __wa_xfer_is_done(struct wa_xfer *xfer) ...@@ -373,10 +373,10 @@ static unsigned __wa_xfer_is_done(struct wa_xfer *xfer)
seg->result); seg->result);
goto out; goto out;
case WA_SEG_ABORTED: case WA_SEG_ABORTED:
dev_dbg(dev, "xfer %p ID %08X#%u ABORTED: result %d\n", xfer->result = seg->result;
xfer, wa_xfer_id(xfer), seg->index, dev_dbg(dev, "xfer %p ID %08X#%u: ABORTED result %zu(0x%08zX)\n",
urb->status); xfer, wa_xfer_id(xfer), seg->index, seg->result,
xfer->result = urb->status; seg->result);
goto out; goto out;
default: default:
dev_warn(dev, "xfer %p ID %08X#%u: is_done bad state %d\n", dev_warn(dev, "xfer %p ID %08X#%u: is_done bad state %d\n",
...@@ -1568,7 +1568,8 @@ static int wa_urb_enqueue_b(struct wa_xfer *xfer) ...@@ -1568,7 +1568,8 @@ static int wa_urb_enqueue_b(struct wa_xfer *xfer)
wusb_dev = __wusb_dev_get_by_usb_dev(wusbhc, urb->dev); wusb_dev = __wusb_dev_get_by_usb_dev(wusbhc, urb->dev);
if (wusb_dev == NULL) { if (wusb_dev == NULL) {
mutex_unlock(&wusbhc->mutex); mutex_unlock(&wusbhc->mutex);
pr_err("%s: error wusb dev gone\n", __func__); dev_err(&(urb->dev->dev), "%s: error wusb dev gone\n",
__func__);
goto error_dev_gone; goto error_dev_gone;
} }
mutex_unlock(&wusbhc->mutex); mutex_unlock(&wusbhc->mutex);
...@@ -1577,18 +1578,18 @@ static int wa_urb_enqueue_b(struct wa_xfer *xfer) ...@@ -1577,18 +1578,18 @@ static int wa_urb_enqueue_b(struct wa_xfer *xfer)
xfer->wusb_dev = wusb_dev; xfer->wusb_dev = wusb_dev;
result = urb->status; result = urb->status;
if (urb->status != -EINPROGRESS) { if (urb->status != -EINPROGRESS) {
pr_err("%s: error_dequeued\n", __func__); dev_err(&(urb->dev->dev), "%s: error_dequeued\n", __func__);
goto error_dequeued; goto error_dequeued;
} }
result = __wa_xfer_setup(xfer, urb); result = __wa_xfer_setup(xfer, urb);
if (result < 0) { if (result < 0) {
pr_err("%s: error_xfer_setup\n", __func__); dev_err(&(urb->dev->dev), "%s: error_xfer_setup\n", __func__);
goto error_xfer_setup; goto error_xfer_setup;
} }
result = __wa_xfer_submit(xfer); result = __wa_xfer_submit(xfer);
if (result < 0) { if (result < 0) {
pr_err("%s: error_xfer_submit\n", __func__); dev_err(&(urb->dev->dev), "%s: error_xfer_submit\n", __func__);
goto error_xfer_submit; goto error_xfer_submit;
} }
spin_unlock_irqrestore(&xfer->lock, flags); spin_unlock_irqrestore(&xfer->lock, flags);
...@@ -1844,8 +1845,8 @@ int wa_urb_dequeue(struct wahc *wa, struct urb *urb, int status) ...@@ -1844,8 +1845,8 @@ int wa_urb_dequeue(struct wahc *wa, struct urb *urb, int status)
pr_debug("%s: DEQUEUE xfer id 0x%08X\n", __func__, wa_xfer_id(xfer)); pr_debug("%s: DEQUEUE xfer id 0x%08X\n", __func__, wa_xfer_id(xfer));
rpipe = xfer->ep->hcpriv; rpipe = xfer->ep->hcpriv;
if (rpipe == NULL) { if (rpipe == NULL) {
pr_debug("%s: xfer id 0x%08X has no RPIPE. %s", pr_debug("%s: xfer %p id 0x%08X has no RPIPE. %s",
__func__, wa_xfer_id(xfer), __func__, xfer, wa_xfer_id(xfer),
"Probably already aborted.\n" ); "Probably already aborted.\n" );
goto out_unlock; goto out_unlock;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment