USB: sierra.c: remove dbg() tracing calls

dbg() was used a lot a long time ago to trace code flow.  Now that we have
ftrace, this isn't needed at all, so remove these calls.

CC: Alan Stern <stern@rowland.harvard.edu>
CC: Rusty Russell <rusty@rustcorp.com.au>
CC: Johan Hovold <jhovold@gmail.com>
CC: Anton Samokhvalov <pg83@yandex.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

1 files changed, 4 insertions, 32 deletions

diff --git a/drivers/usb/serial/sierra.c b/drivers/usb/serial/sierra.c
index 8c8bf806f6fa..f298ddaac684 100644
--- a/drivers/usb/serial/sierra.c
+++ b/drivers/usb/serial/sierra.c
@@ -63,9 +63,7 @@ struct sierra_intf_private {
 
 static int sierra_set_power_state(struct usb_device *udev, __u16 swiState)
 {
-        int result;
-        dev_dbg(&udev->dev, "%s\n", __func__);
-        result = usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
+        return usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
                         SWIMS_USB_REQUEST_SetPower,     /* __u8 request      */
                         USB_TYPE_VENDOR,                /* __u8 request type */
                         swiState,                       /* __u16 value       */
@@ -73,14 +71,11 @@ static int sierra_set_power_state(struct usb_device *udev, __u16 swiState)
                         NULL,                           /* void *data        */
                         0,                              /* __u16 size        */
                         USB_CTRL_SET_TIMEOUT);          /* int timeout       */
-        return result;
 }
 
 static int sierra_vsc_set_nmea(struct usb_device *udev, __u16 enable)
 {
-        int result;
-        dev_dbg(&udev->dev, "%s\n", __func__);
-        result = usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
+        return usb_control_msg(udev, usb_sndctrlpipe(udev, 0),
                         SWIMS_USB_REQUEST_SetNmea,      /* __u8 request      */
                         USB_TYPE_VENDOR,                /* __u8 request type */
                         enable,                         /* __u16 value       */
@@ -88,7 +83,6 @@ static int sierra_vsc_set_nmea(struct usb_device *udev, __u16 enable)
                         NULL,                           /* void *data        */
                         0,                              /* __u16 size        */
                         USB_CTRL_SET_TIMEOUT);          /* int timeout       */
-        return result;
 }
 
 static int sierra_calc_num_ports(struct usb_serial *serial)
@@ -96,8 +90,6 @@ static int sierra_calc_num_ports(struct usb_serial *serial)
         int num_ports = 0;
         u8 ifnum, numendpoints;
 
-        dev_dbg(&serial->dev->dev, "%s\n", __func__);
-
         ifnum = serial->interface->cur_altsetting->desc.bInterfaceNumber;
         numendpoints = serial->interface->cur_altsetting->desc.bNumEndpoints;
 
@@ -150,7 +142,6 @@ static int sierra_calc_interface(struct usb_serial *serial)
         int interface;
         struct usb_interface *p_interface;
         struct usb_host_interface *p_host_interface;
-        dev_dbg(&serial->dev->dev, "%s\n", __func__);
 
         /* Get the interface structure pointer from the serial struct */
         p_interface = serial->interface;
@@ -175,9 +166,8 @@ static int sierra_probe(struct usb_serial *serial,
         u8 ifnum;
 
         udev = serial->dev;
-        dev_dbg(&udev->dev, "%s\n", __func__);
-
         ifnum = sierra_calc_interface(serial);
+
         /*
          * If this interface supports more than 1 alternate
          * select the 2nd one
@@ -344,8 +334,6 @@ static int sierra_send_setup(struct usb_serial_port *port)
         int do_send = 0;
         int retval;
 
-        dev_dbg(&port->dev, "%s\n", __func__);
-
         portdata = usb_get_serial_port_data(port);
 
         if (portdata->dtr_state)
@@ -393,7 +381,6 @@ static int sierra_send_setup(struct usb_serial_port *port)
 static void sierra_set_termios(struct tty_struct *tty,
                 struct usb_serial_port *port, struct ktermios *old_termios)
 {
-        dev_dbg(&port->dev, "%s\n", __func__);
         tty_termios_copy_hw(tty->termios, old_termios);
         sierra_send_setup(port);
 }
@@ -404,7 +391,6 @@ static int sierra_tiocmget(struct tty_struct *tty)
         unsigned int value;
         struct sierra_port_private *portdata;
 
-        dev_dbg(&port->dev, "%s\n", __func__);
         portdata = usb_get_serial_port_data(port);
 
         value = ((portdata->rts_state) ? TIOCM_RTS : 0) |
@@ -441,8 +427,7 @@ static void sierra_release_urb(struct urb *urb)
 {
         struct usb_serial_port *port;
         if (urb) {
-                port =  urb->context;
-                dev_dbg(&port->dev, "%s: %p\n", __func__, urb);
+                port = urb->context;
                 kfree(urb->transfer_buffer);
                 usb_free_urb(urb);
         }
@@ -455,7 +440,6 @@ static void sierra_outdat_callback(struct urb *urb)
         struct sierra_intf_private *intfdata;
         int status = urb->status;
 
-        dev_dbg(&port->dev, "%s - port %d\n", __func__, port->number);
         intfdata = port->serial->private;
 
         /* free up the transfer buffer, as usb_free_urb() does not do this */
@@ -598,8 +582,6 @@ static void sierra_indat_callback(struct urb *urb)
         endpoint = usb_pipeendpoint(urb->pipe);
         port = urb->context;
 
-        dev_dbg(&port->dev, "%s: %p\n", __func__, urb);
-
         if (status) {
                 dev_dbg(&port->dev, "%s: nonzero status: %d on"
                         " endpoint %02x\n", __func__, status, endpoint);
@@ -697,8 +679,6 @@ static int sierra_write_room(struct tty_struct *tty)
         struct sierra_port_private *portdata = usb_get_serial_port_data(port);
         unsigned long flags;
 
-        dev_dbg(&port->dev, "%s - port %d\n", __func__, port->number);
-
         /* try to give a good number back based on if we have any free urbs at
          * this point in time */
         spin_lock_irqsave(&portdata->lock, flags);
@@ -805,8 +785,6 @@ static void sierra_close(struct usb_serial_port *port)
         struct sierra_port_private *portdata;
         struct sierra_intf_private *intfdata = port->serial->private;
 
-
-        dev_dbg(&port->dev, "%s\n", __func__);
         portdata = usb_get_serial_port_data(port);
 
         portdata->rts_state = 0;
@@ -851,8 +829,6 @@ static int sierra_open(struct tty_struct *tty, struct usb_serial_port *port)
 
         portdata = usb_get_serial_port_data(port);
 
-        dev_dbg(&port->dev, "%s\n", __func__);
-
         /* Set some sane defaults */
         portdata->rts_state = 1;
         portdata->dtr_state = 1;
@@ -915,8 +891,6 @@ static int sierra_startup(struct usb_serial *serial)
         int i;
         u8 ifnum;
 
-        dev_dbg(&serial->dev->dev, "%s\n", __func__);
-
         /* Set Device mode to D0 */
         sierra_set_power_state(serial->dev, 0x0000);
 
@@ -977,8 +951,6 @@ static void sierra_release(struct usb_serial *serial)
         struct usb_serial_port *port;
         struct sierra_port_private *portdata;
 
-        dev_dbg(&serial->dev->dev, "%s\n", __func__);
-
         for (i = 0; i < serial->num_ports; ++i) {
                 port = serial->port[i];
                 if (!port)