Commit e587b8b2 authored by Oliver Neukum's avatar Oliver Neukum Committed by Sarah Sharp

xhci: make warnings greppable

This changes debug messages and warnings in xhci-ring.c
to be on a single line so grep can find them. grep must
have precedence over the 80 column limit.

[Sarah fixed two checkpatch.pl issues with split lines
introduced by this commit.]
Signed-off-by: default avatarOliver Neukum <oneukum@suse.de>
Signed-off-by: default avatarSarah Sharp <sarah.a.sharp@linux.intel.com>
parent 25cd2882
...@@ -1082,8 +1082,7 @@ static void xhci_handle_cmd_set_deq(struct xhci_hcd *xhci, int slot_id, ...@@ -1082,8 +1082,7 @@ static void xhci_handle_cmd_set_deq(struct xhci_hcd *xhci, int slot_id,
ep_ring = xhci_stream_id_to_ring(dev, ep_index, stream_id); ep_ring = xhci_stream_id_to_ring(dev, ep_index, stream_id);
if (!ep_ring) { if (!ep_ring) {
xhci_warn(xhci, "WARN Set TR deq ptr command for " xhci_warn(xhci, "WARN Set TR deq ptr command for freed stream ID %u\n",
"freed stream ID %u\n",
stream_id); stream_id);
/* XXX: Harmless??? */ /* XXX: Harmless??? */
dev->eps[ep_index].ep_state &= ~SET_DEQ_PENDING; dev->eps[ep_index].ep_state &= ~SET_DEQ_PENDING;
...@@ -1099,12 +1098,10 @@ static void xhci_handle_cmd_set_deq(struct xhci_hcd *xhci, int slot_id, ...@@ -1099,12 +1098,10 @@ static void xhci_handle_cmd_set_deq(struct xhci_hcd *xhci, int slot_id,
switch (cmd_comp_code) { switch (cmd_comp_code) {
case COMP_TRB_ERR: case COMP_TRB_ERR:
xhci_warn(xhci, "WARN Set TR Deq Ptr cmd invalid because " xhci_warn(xhci, "WARN Set TR Deq Ptr cmd invalid because of stream ID configuration\n");
"of stream ID configuration\n");
break; break;
case COMP_CTX_STATE: case COMP_CTX_STATE:
xhci_warn(xhci, "WARN Set TR Deq Ptr cmd failed due " xhci_warn(xhci, "WARN Set TR Deq Ptr cmd failed due to incorrect slot or ep state.\n");
"to incorrect slot or ep state.\n");
ep_state = le32_to_cpu(ep_ctx->ep_info); ep_state = le32_to_cpu(ep_ctx->ep_info);
ep_state &= EP_STATE_MASK; ep_state &= EP_STATE_MASK;
slot_state = le32_to_cpu(slot_ctx->dev_state); slot_state = le32_to_cpu(slot_ctx->dev_state);
...@@ -1114,13 +1111,12 @@ static void xhci_handle_cmd_set_deq(struct xhci_hcd *xhci, int slot_id, ...@@ -1114,13 +1111,12 @@ static void xhci_handle_cmd_set_deq(struct xhci_hcd *xhci, int slot_id,
slot_state, ep_state); slot_state, ep_state);
break; break;
case COMP_EBADSLT: case COMP_EBADSLT:
xhci_warn(xhci, "WARN Set TR Deq Ptr cmd failed because " xhci_warn(xhci, "WARN Set TR Deq Ptr cmd failed because slot %u was not enabled.\n",
"slot %u was not enabled.\n", slot_id); slot_id);
break; break;
default: default:
xhci_warn(xhci, "WARN Set TR Deq Ptr cmd with unknown " xhci_warn(xhci, "WARN Set TR Deq Ptr cmd with unknown completion code of %u.\n",
"completion code of %u.\n", cmd_comp_code);
cmd_comp_code);
break; break;
} }
/* OK what do we do now? The endpoint state is hosed, and we /* OK what do we do now? The endpoint state is hosed, and we
...@@ -1142,8 +1138,7 @@ static void xhci_handle_cmd_set_deq(struct xhci_hcd *xhci, int slot_id, ...@@ -1142,8 +1138,7 @@ static void xhci_handle_cmd_set_deq(struct xhci_hcd *xhci, int slot_id,
update_ring_for_set_deq_completion(xhci, dev, update_ring_for_set_deq_completion(xhci, dev,
ep_ring, ep_index); ep_ring, ep_index);
} else { } else {
xhci_warn(xhci, "Mismatch between completed Set TR Deq " xhci_warn(xhci, "Mismatch between completed Set TR Deq Ptr command & xHCI internal state.\n");
"Ptr command & xHCI internal state.\n");
xhci_warn(xhci, "ep deq seg = %p, deq ptr = %p\n", xhci_warn(xhci, "ep deq seg = %p, deq ptr = %p\n",
dev->eps[ep_index].queued_deq_seg, dev->eps[ep_index].queued_deq_seg,
dev->eps[ep_index].queued_deq_ptr); dev->eps[ep_index].queued_deq_ptr);
......
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