xhci: Add bus number to some debug messages
authorMathias Nyman <mathias.nyman@linux.intel.com>
Fri, 20 Aug 2021 12:35:03 +0000 (15:35 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 26 Aug 2021 11:06:03 +0000 (13:06 +0200)
As we register two usb buses for each xHC, and systems with several
hosts are more and more common it is getting hard to follow the
flow of debug messages without knowing which bus they belong to

Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
Link: https://lore.kernel.org/r/20210820123503.2605901-7-mathias.nyman@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/xhci-hub.c
drivers/usb/host/xhci-ring.c
drivers/usb/host/xhci.c

index 151e93c..a3f875e 100644 (file)
@@ -1667,7 +1667,8 @@ int xhci_hub_status_data(struct usb_hcd *hcd, char *buf)
                        status = 1;
        }
        if (!status && !reset_change) {
-               xhci_dbg(xhci, "%s: stopping port polling.\n", __func__);
+               xhci_dbg(xhci, "%s: stopping usb%d port polling\n",
+                        __func__, hcd->self.busnum);
                clear_bit(HCD_FLAG_POLL_RH, &hcd->flags);
        }
        spin_unlock_irqrestore(&xhci->lock, flags);
@@ -1699,7 +1700,8 @@ int xhci_bus_suspend(struct usb_hcd *hcd)
                if (bus_state->resuming_ports ||        /* USB2 */
                    bus_state->port_remote_wakeup) {    /* USB3 */
                        spin_unlock_irqrestore(&xhci->lock, flags);
-                       xhci_dbg(xhci, "suspend failed because a port is resuming\n");
+                       xhci_dbg(xhci, "usb%d bus suspend to fail because a port is resuming\n",
+                                hcd->self.busnum);
                        return -EBUSY;
                }
        }
index 8be4ba3..e676749 100644 (file)
@@ -2031,7 +2031,8 @@ cleanup:
         * bits are still set.  When an event occurs, switch over to
         * polling to avoid losing status changes.
         */
-       xhci_dbg(xhci, "%s: starting port polling.\n", __func__);
+       xhci_dbg(xhci, "%s: starting usb%d port polling.\n",
+                __func__, hcd->self.busnum);
        set_bit(HCD_FLAG_POLL_RH, &hcd->flags);
        spin_unlock(&xhci->lock);
        /* Pass this up to the core */
index 18a203c..f3dabd0 100644 (file)
@@ -993,7 +993,8 @@ int xhci_suspend(struct xhci_hcd *xhci, bool do_wakeup)
        xhci_dbc_suspend(xhci);
 
        /* Don't poll the roothubs on bus suspend. */
-       xhci_dbg(xhci, "%s: stopping port polling.\n", __func__);
+       xhci_dbg(xhci, "%s: stopping usb%d port polling.\n",
+                __func__, hcd->self.busnum);
        clear_bit(HCD_FLAG_POLL_RH, &hcd->flags);
        del_timer_sync(&hcd->rh_timer);
        clear_bit(HCD_FLAG_POLL_RH, &xhci->shared_hcd->flags);
@@ -1257,7 +1258,8 @@ int xhci_resume(struct xhci_hcd *xhci, bool hibernated)
                usb_asmedia_modifyflowcontrol(to_pci_dev(hcd->self.controller));
 
        /* Re-enable port polling. */
-       xhci_dbg(xhci, "%s: starting port polling.\n", __func__);
+       xhci_dbg(xhci, "%s: starting usb%d port polling.\n",
+                __func__, hcd->self.busnum);
        set_bit(HCD_FLAG_POLL_RH, &xhci->shared_hcd->flags);
        usb_hcd_poll_rh_status(xhci->shared_hcd);
        set_bit(HCD_FLAG_POLL_RH, &hcd->flags);