USB: legotower: remove unneeded tracing macros
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 26 Jun 2013 23:30:39 +0000 (16:30 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 23 Jul 2013 23:10:02 +0000 (16:10 -0700)
Remove the unneeded tracing macros in this driver.  The kernel has a
built-in trace function that can be used if this is really still needed.

Cc: Juergen Stuber <starblue@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/misc/legousbtower.c

index 80894791c02003fb8ee628e8ed722411bb174ee9..93c95d09f04eb3931e3a29272b6e76fcb9691cc3 100644 (file)
@@ -318,8 +318,6 @@ static inline void lego_usb_tower_debug_data (int level, const char *function, i
  */
 static inline void tower_delete (struct lego_usb_tower *dev)
 {
-       dbg(2, "%s: enter", __func__);
-
        tower_abort_transfers (dev);
 
        /* free data structures */
@@ -329,8 +327,6 @@ static inline void tower_delete (struct lego_usb_tower *dev)
        kfree (dev->interrupt_in_buffer);
        kfree (dev->interrupt_out_buffer);
        kfree (dev);
-
-       dbg(2, "%s: leave", __func__);
 }
 
 
@@ -346,8 +342,6 @@ static int tower_open (struct inode *inode, struct file *file)
        struct tower_reset_reply reset_reply;
        int result;
 
-       dbg(2, "%s: enter", __func__);
-
        nonseekable_open(inode, file);
        subminor = iminor(inode);
 
@@ -435,8 +429,6 @@ unlock_exit:
        mutex_unlock(&dev->lock);
 
 exit:
-       dbg(2, "%s: leave, return value %d ", __func__, retval);
-
        return retval;
 }
 
@@ -448,8 +440,6 @@ static int tower_release (struct inode *inode, struct file *file)
        struct lego_usb_tower *dev;
        int retval = 0;
 
-       dbg(2, "%s: enter", __func__);
-
        dev = file->private_data;
 
        if (dev == NULL) {
@@ -491,7 +481,6 @@ unlock_exit:
 exit:
        mutex_unlock(&open_disc_mutex);
 exit_nolock:
-       dbg(2, "%s: leave, return value %d", __func__, retval);
        return retval;
 }
 
@@ -502,11 +491,9 @@ exit_nolock:
  */
 static void tower_abort_transfers (struct lego_usb_tower *dev)
 {
-       dbg(2, "%s: enter", __func__);
-
        if (dev == NULL) {
                dbg(1, "%s: dev is null", __func__);
-               goto exit;
+               return;
        }
 
        /* shutdown transfer */
@@ -518,9 +505,6 @@ static void tower_abort_transfers (struct lego_usb_tower *dev)
        }
        if (dev->interrupt_out_busy && dev->udev)
                usb_kill_urb(dev->interrupt_out_urb);
-
-exit:
-       dbg(2, "%s: leave", __func__);
 }
 
 
@@ -553,8 +537,6 @@ static unsigned int tower_poll (struct file *file, poll_table *wait)
        struct lego_usb_tower *dev;
        unsigned int mask = 0;
 
-       dbg(2, "%s: enter", __func__);
-
        dev = file->private_data;
 
        if (!dev->udev)
@@ -571,8 +553,6 @@ static unsigned int tower_poll (struct file *file, poll_table *wait)
                mask |= POLLOUT | POLLWRNORM;
        }
 
-       dbg(2, "%s: leave, mask = %d", __func__, mask);
-
        return mask;
 }
 
@@ -597,8 +577,6 @@ static ssize_t tower_read (struct file *file, char __user *buffer, size_t count,
        int retval = 0;
        unsigned long timeout = 0;
 
-       dbg(2, "%s: enter, count = %Zd", __func__, count);
-
        dev = file->private_data;
 
        /* lock this object */
@@ -672,7 +650,6 @@ unlock_exit:
        mutex_unlock(&dev->lock);
 
 exit:
-       dbg(2, "%s: leave, return value %d", __func__, retval);
        return retval;
 }
 
@@ -686,8 +663,6 @@ static ssize_t tower_write (struct file *file, const char __user *buffer, size_t
        size_t bytes_to_write;
        int retval = 0;
 
-       dbg(2, "%s: enter, count = %Zd", __func__, count);
-
        dev = file->private_data;
 
        /* lock this object */
@@ -757,8 +732,6 @@ unlock_exit:
        mutex_unlock(&dev->lock);
 
 exit:
-       dbg(2, "%s: leave, return value %d", __func__, retval);
-
        return retval;
 }
 
@@ -772,8 +745,6 @@ static void tower_interrupt_in_callback (struct urb *urb)
        int status = urb->status;
        int retval;
 
-       dbg(4, "%s: enter, status %d", __func__, status);
-
        lego_usb_tower_debug_data(5, __func__, urb->actual_length, urb->transfer_buffer);
 
        if (status) {
@@ -817,7 +788,6 @@ exit:
        wake_up_interruptible (&dev->read_wait);
 
        lego_usb_tower_debug_data(5, __func__, urb->actual_length, urb->transfer_buffer);
-       dbg(4, "%s: leave, status %d", __func__, status);
 }
 
 
@@ -829,7 +799,6 @@ static void tower_interrupt_out_callback (struct urb *urb)
        struct lego_usb_tower *dev = urb->context;
        int status = urb->status;
 
-       dbg(4, "%s: enter, status %d", __func__, status);
        lego_usb_tower_debug_data(5, __func__, urb->actual_length, urb->transfer_buffer);
 
        /* sync/async unlink faults aren't errors */
@@ -844,7 +813,6 @@ static void tower_interrupt_out_callback (struct urb *urb)
        wake_up_interruptible(&dev->write_wait);
 
        lego_usb_tower_debug_data(5, __func__, urb->actual_length, urb->transfer_buffer);
-       dbg(4, "%s: leave, status %d", __func__, status);
 }
 
 
@@ -866,8 +834,6 @@ static int tower_probe (struct usb_interface *interface, const struct usb_device
        int retval = -ENOMEM;
        int result;
 
-       dbg(2, "%s: enter", __func__);
-
        /* allocate memory for our device state and initialize it */
 
        dev = kmalloc (sizeof(struct lego_usb_tower), GFP_KERNEL);
@@ -993,8 +959,6 @@ static int tower_probe (struct usb_interface *interface, const struct usb_device
 
 
 exit:
-       dbg(2, "%s: leave, return value 0x%.8lx (dev)", __func__, (long) dev);
-
        return retval;
 
 error:
@@ -1013,8 +977,6 @@ static void tower_disconnect (struct usb_interface *interface)
        struct lego_usb_tower *dev;
        int minor;
 
-       dbg(2, "%s: enter", __func__);
-
        dev = usb_get_intfdata (interface);
        mutex_lock(&open_disc_mutex);
        usb_set_intfdata (interface, NULL);
@@ -1041,8 +1003,6 @@ static void tower_disconnect (struct usb_interface *interface)
 
        dev_info(&interface->dev, "LEGO USB Tower #%d now disconnected\n",
                 (minor - LEGO_USB_TOWER_MINOR_BASE));
-
-       dbg(2, "%s: leave", __func__);
 }
 
 module_usb_driver(tower_driver);