usb: core: hcd: make hcd->irq unsigned

There's really no point in having hcd->irq as a
signed integer when we consider the fact that
IRQ 0 means NO_IRQ. In order to avoid confusion,
make hcd->irq unsigned and fix users who were
passing -1 as the IRQ number to usb_add_hcd.

Tested-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

1 files changed, 4 insertions, 4 deletions

diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
index c939f5fdef9..a629ad86032 100644
--- a/drivers/usb/host/xhci.c
+++ b/drivers/usb/host/xhci.c
@@ -224,13 +224,13 @@ static void xhci_free_irq(struct xhci_hcd *xhci)
         int ret;
 
         /* return if using legacy interrupt */
-        if (xhci_to_hcd(xhci)->irq >= 0)
+        if (xhci_to_hcd(xhci)->irq > 0)
                 return;
 
         ret = xhci_free_msi(xhci);
         if (!ret)
                 return;
-        if (pdev->irq >= 0)
+        if (pdev->irq > 0)
                 free_irq(pdev->irq, xhci_to_hcd(xhci));
 
         return;
@@ -341,7 +341,7 @@ static int xhci_try_enable_msi(struct usb_hcd *hcd)
         /* unregister the legacy interrupt */
         if (hcd->irq)
                 free_irq(hcd->irq, hcd);
-        hcd->irq = -1;
+        hcd->irq = 0;
 
         ret = xhci_setup_msix(xhci);
         if (ret)
@@ -349,7 +349,7 @@ static int xhci_try_enable_msi(struct usb_hcd *hcd)
                 ret = xhci_setup_msi(xhci);
 
         if (!ret)
-                /* hcd->irq is -1, we have MSI */
+                /* hcd->irq is 0, we have MSI */
                 return 0;
 
         if (!pdev->irq) {