USB: isp1362: remove _DBG() usage
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 28 Jun 2013 18:33:00 +0000 (11:33 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 23 Jul 2013 23:35:17 +0000 (16:35 -0700)
If you want a debug call, just make it, so move to using the
already-there DBG() call.  No need to make things more complex than they
really need to be.

Acked-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/isp1362-hcd.c
drivers/usb/host/isp1362.h

index c747ba35f7319501eabaeb8e91b4c128ab04b084..428d184ceb339390bd0de08f21dc7a9034fd7f18 100644 (file)
@@ -1573,12 +1573,12 @@ static int isp1362_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
                DBG(0, "ClearHubFeature: ");
                switch (wValue) {
                case C_HUB_OVER_CURRENT:
-                       _DBG(0, "C_HUB_OVER_CURRENT\n");
+                       DBG(0, "C_HUB_OVER_CURRENT\n");
                        spin_lock_irqsave(&isp1362_hcd->lock, flags);
                        isp1362_write_reg32(isp1362_hcd, HCRHSTATUS, RH_HS_OCIC);
                        spin_unlock_irqrestore(&isp1362_hcd->lock, flags);
                case C_HUB_LOCAL_POWER:
-                       _DBG(0, "C_HUB_LOCAL_POWER\n");
+                       DBG(0, "C_HUB_LOCAL_POWER\n");
                        break;
                default:
                        goto error;
@@ -1589,7 +1589,7 @@ static int isp1362_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
                switch (wValue) {
                case C_HUB_OVER_CURRENT:
                case C_HUB_LOCAL_POWER:
-                       _DBG(0, "C_HUB_OVER_CURRENT or C_HUB_LOCAL_POWER\n");
+                       DBG(0, "C_HUB_OVER_CURRENT or C_HUB_LOCAL_POWER\n");
                        break;
                default:
                        goto error;
@@ -1620,36 +1620,36 @@ static int isp1362_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
 
                switch (wValue) {
                case USB_PORT_FEAT_ENABLE:
-                       _DBG(0, "USB_PORT_FEAT_ENABLE\n");
+                       DBG(0, "USB_PORT_FEAT_ENABLE\n");
                        tmp = RH_PS_CCS;
                        break;
                case USB_PORT_FEAT_C_ENABLE:
-                       _DBG(0, "USB_PORT_FEAT_C_ENABLE\n");
+                       DBG(0, "USB_PORT_FEAT_C_ENABLE\n");
                        tmp = RH_PS_PESC;
                        break;
                case USB_PORT_FEAT_SUSPEND:
-                       _DBG(0, "USB_PORT_FEAT_SUSPEND\n");
+                       DBG(0, "USB_PORT_FEAT_SUSPEND\n");
                        tmp = RH_PS_POCI;
                        break;
                case USB_PORT_FEAT_C_SUSPEND:
-                       _DBG(0, "USB_PORT_FEAT_C_SUSPEND\n");
+                       DBG(0, "USB_PORT_FEAT_C_SUSPEND\n");
                        tmp = RH_PS_PSSC;
                        break;
                case USB_PORT_FEAT_POWER:
-                       _DBG(0, "USB_PORT_FEAT_POWER\n");
+                       DBG(0, "USB_PORT_FEAT_POWER\n");
                        tmp = RH_PS_LSDA;
 
                        break;
                case USB_PORT_FEAT_C_CONNECTION:
-                       _DBG(0, "USB_PORT_FEAT_C_CONNECTION\n");
+                       DBG(0, "USB_PORT_FEAT_C_CONNECTION\n");
                        tmp = RH_PS_CSC;
                        break;
                case USB_PORT_FEAT_C_OVER_CURRENT:
-                       _DBG(0, "USB_PORT_FEAT_C_OVER_CURRENT\n");
+                       DBG(0, "USB_PORT_FEAT_C_OVER_CURRENT\n");
                        tmp = RH_PS_OCIC;
                        break;
                case USB_PORT_FEAT_C_RESET:
-                       _DBG(0, "USB_PORT_FEAT_C_RESET\n");
+                       DBG(0, "USB_PORT_FEAT_C_RESET\n");
                        tmp = RH_PS_PRSC;
                        break;
                default:
@@ -1669,7 +1669,7 @@ static int isp1362_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
                wIndex--;
                switch (wValue) {
                case USB_PORT_FEAT_SUSPEND:
-                       _DBG(0, "USB_PORT_FEAT_SUSPEND\n");
+                       DBG(0, "USB_PORT_FEAT_SUSPEND\n");
                        spin_lock_irqsave(&isp1362_hcd->lock, flags);
                        isp1362_write_reg32(isp1362_hcd, HCRHPORT1 + wIndex, RH_PS_PSS);
                        isp1362_hcd->rhport[wIndex] =
@@ -1677,7 +1677,7 @@ static int isp1362_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
                        spin_unlock_irqrestore(&isp1362_hcd->lock, flags);
                        break;
                case USB_PORT_FEAT_POWER:
-                       _DBG(0, "USB_PORT_FEAT_POWER\n");
+                       DBG(0, "USB_PORT_FEAT_POWER\n");
                        spin_lock_irqsave(&isp1362_hcd->lock, flags);
                        isp1362_write_reg32(isp1362_hcd, HCRHPORT1 + wIndex, RH_PS_PPS);
                        isp1362_hcd->rhport[wIndex] =
@@ -1685,7 +1685,7 @@ static int isp1362_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
                        spin_unlock_irqrestore(&isp1362_hcd->lock, flags);
                        break;
                case USB_PORT_FEAT_RESET:
-                       _DBG(0, "USB_PORT_FEAT_RESET\n");
+                       DBG(0, "USB_PORT_FEAT_RESET\n");
                        spin_lock_irqsave(&isp1362_hcd->lock, flags);
 
                        t1 = jiffies + msecs_to_jiffies(USB_RESET_WIDTH);
@@ -1719,7 +1719,7 @@ static int isp1362_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
        default:
  error:
                /* "protocol stall" on error */
-               _DBG(0, "PROTOCOL STALL\n");
+               DBG(0, "PROTOCOL STALL\n");
                retval = -EPIPE;
        }
 
index e0f6f922052d139e18bbaa14b9bf978ec40a8b96..d36a9086890039f25ba28e9c30ae3b28b28b0941 100644 (file)
@@ -588,14 +588,8 @@ static inline struct usb_hcd *isp1362_hcd_to_hcd(struct isp1362_hcd *isp1362_hcd
                if (dbg_level > level) \
                        pr_debug(fmt); \
        } while (0)
-#define _DBG(level, fmt...)    \
-       do { \
-               if (dbg_level > level) \
-                       printk(fmt); \
-       } while (0)
 #else
 #define DBG(fmt...)            do {} while (0)
-#define _DBG DBG
 #endif
 
 #ifdef VERBOSE