[PATCH] USB: ohci-hcd fewer diagnostics

This gets rid of some potentially scarey messages I've recently
seen on disconnect, "bad hash" for a TD ... it's really a "no hash"
case, and this prevents the message in some cases where that's
not an issue.  Likely this is what Gary Gorgen noticed, but even
he had a different problem, this message shouldn't always appear.

It also slims down one other message, preventing it from appearing
in the routine "protocol stall" case (and cluttering logfiles).

drivers/usb/host/ohci-mem.c

@@ -115,8 +115,8 @@ td_free (struct ohci_hcd *hc, struct td *td)
 		prev = &(*prev)->td_hash;
 	if (*prev)
 		*prev = td->td_hash;
-	else
-		ohci_dbg (hc, "bad hash for td %p\n", td);
+	else if ((td->hwINFO & TD_DONE) != 0)
+		ohci_dbg (hc, "no hash for td %p\n", td);
 	pci_pool_free (hc->td_cache, td, td->td_dma);
 }
 

drivers/usb/host/ohci-q.c

@@ -783,13 +783,17 @@ ed_halted (struct ohci_hcd *ohci, struct td *td, int cc, struct td *rev)
 		ed->hwHeadP = next->hwNextTD | toggle;
 	}
 
-	/* help for troubleshooting: */
+	/* help for troubleshooting:  report anything that
+	 * looks odd ... that doesn't include protocol stalls
+	 * (or maybe some other things)
+	 */
+	if (cc != TD_CC_STALL || !usb_pipecontrol (urb->pipe))
 		ohci_dbg (ohci,
-		"urb %p usb-%s-%s ep-%d-%s cc %d --> status %d\n",
-		urb,
-		urb->dev->bus->bus_name, urb->dev->devpath,
+			"urb %p path %s ep%d%s %08x cc %d --> status %d\n",
+			urb, urb->dev->devpath,
 			usb_pipeendpoint (urb->pipe),
-		usb_pipein (urb->pipe) ? "IN" : "OUT",
+			usb_pipein (urb->pipe) ? "in" : "out",
+			le32_to_cpu (td->hwINFO),
 			cc, cc_to_error [cc]);
 
 	return rev;