USB: serial: kl5kusb105.c: remove dbg() usage
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 14 Sep 2012 16:47:43 +0000 (09:47 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 14 Sep 2012 16:47:43 +0000 (09:47 -0700)
dbg() was a very old USB-serial-specific macro.
This patch removes it from being used in the
driver and uses dev_dbg() instead.

CC: Oliver Neukum <oliver@neukum.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/serial/kl5kusb105.c

index 5bed59c..73e0a23 100644 (file)
@@ -239,7 +239,9 @@ static int klsi_105_startup(struct usb_serial *serial)
                priv = kmalloc(sizeof(struct klsi_105_private),
                                                   GFP_KERNEL);
                if (!priv) {
-                       dbg("%skmalloc for klsi_105_private failed.", __func__);
+                       dev_dbg(&serial->interface->dev,
+                               "%s - kmalloc for klsi_105_private failed.\n",
+                               __func__);
                        i--;
                        goto err_cleanup;
                }
@@ -344,14 +346,14 @@ static int  klsi_105_open(struct tty_struct *tty, struct usb_serial_port *port)
                dev_err(&port->dev, "Enabling read failed (error = %d)\n", rc);
                retval = rc;
        } else
-               dbg("%s - enabled reading", __func__);
+               dev_dbg(&port->dev, "%s - enabled reading\n", __func__);
 
        rc = klsi_105_get_line_state(port, &line_state);
        if (rc >= 0) {
                spin_lock_irqsave(&priv->lock, flags);
                priv->line_state = line_state;
                spin_unlock_irqrestore(&priv->lock, flags);
-               dbg("%s - read line state 0x%lx", __func__, line_state);
+               dev_dbg(&port->dev, "%s - read line state 0x%lx\n", __func__, line_state);
                retval = 0;
        } else
                retval = rc;
@@ -421,7 +423,7 @@ static void klsi_105_process_read_urb(struct urb *urb)
                return;
 
        if (urb->actual_length <= KLSI_HDR_LEN) {
-               dbg("%s - malformed packet", __func__);
+               dev_dbg(&port->dev, "%s - malformed packet\n", __func__);
                return;
        }
 
@@ -431,7 +433,7 @@ static void klsi_105_process_read_urb(struct urb *urb)
 
        len = get_unaligned_le16(data);
        if (len > urb->actual_length - KLSI_HDR_LEN) {
-               dbg("%s - packet length mismatch", __func__);
+               dev_dbg(&port->dev, "%s - packet length mismatch\n", __func__);
                len = urb->actual_length - KLSI_HDR_LEN;
        }
 
@@ -445,6 +447,7 @@ static void klsi_105_set_termios(struct tty_struct *tty,
                                 struct ktermios *old_termios)
 {
        struct klsi_105_private *priv = usb_get_serial_port_data(port);
+       struct device *dev = &port->dev;
        unsigned int iflag = tty->termios->c_iflag;
        unsigned int old_iflag = old_termios->c_iflag;
        unsigned int cflag = tty->termios->c_cflag;
@@ -455,8 +458,7 @@ static void klsi_105_set_termios(struct tty_struct *tty,
 
        cfg = kmalloc(sizeof(*cfg), GFP_KERNEL);
        if (!cfg) {
-               dev_err(&port->dev, "%s - out of memory for config buffer.\n",
-                               __func__);
+               dev_err(dev, "%s - out of memory for config buffer.\n", __func__);
                return;
        }
 
@@ -471,7 +473,7 @@ static void klsi_105_set_termios(struct tty_struct *tty,
        if ((cflag & CBAUD) != (old_cflag & CBAUD)) {
                /* reassert DTR and (maybe) RTS on transition from B0 */
                if ((old_cflag & CBAUD) == B0) {
-                       dbg("%s: baud was B0", __func__);
+                       dev_dbg(dev, "%s: baud was B0\n", __func__);
 #if 0
                        priv->control_state |= TIOCM_DTR;
                        /* don't set RTS if using hardware flow control */
@@ -509,14 +511,13 @@ static void klsi_105_set_termios(struct tty_struct *tty,
                priv->cfg.baudrate = kl5kusb105a_sio_b115200;
                break;
        default:
-               dbg("KLSI USB->Serial converter:"
-                   " unsupported baudrate request, using default of 9600");
-                       priv->cfg.baudrate = kl5kusb105a_sio_b9600;
+               dev_dbg(dev, "KLSI USB->Serial converter: unsupported baudrate request, using default of 9600");
+               priv->cfg.baudrate = kl5kusb105a_sio_b9600;
                baud = 9600;
                break;
        }
        if ((cflag & CBAUD) == B0) {
-               dbg("%s: baud is B0", __func__);
+               dev_dbg(dev, "%s: baud is B0\n", __func__);
                /* Drop RTS and DTR */
                /* maybe this should be simulated by sending read
                 * disable and read enable messages?
@@ -533,11 +534,11 @@ static void klsi_105_set_termios(struct tty_struct *tty,
                /* set the number of data bits */
                switch (cflag & CSIZE) {
                case CS5:
-                       dbg("%s - 5 bits/byte not supported", __func__);
+                       dev_dbg(dev, "%s - 5 bits/byte not supported\n", __func__);
                        spin_unlock_irqrestore(&priv->lock, flags);
                        goto err;
                case CS6:
-                       dbg("%s - 6 bits/byte not supported", __func__);
+                       dev_dbg(dev, "%s - 6 bits/byte not supported\n", __func__);
                        spin_unlock_irqrestore(&priv->lock, flags);
                        goto err;
                case CS7:
@@ -547,8 +548,7 @@ static void klsi_105_set_termios(struct tty_struct *tty,
                        priv->cfg.databits = kl5kusb105a_dtb_8;
                        break;
                default:
-                       dev_err(&port->dev,
-                               "CSIZE was not CS5-CS8, using default of 8\n");
+                       dev_err(dev, "CSIZE was not CS5-CS8, using default of 8\n");
                        priv->cfg.databits = kl5kusb105a_dtb_8;
                        break;
                }
@@ -616,7 +616,7 @@ static void mct_u232_break_ctl(struct tty_struct *tty, int break_state)
                                (struct mct_u232_private *)port->private;
        unsigned char lcr = priv->last_lcr;
 
-       dbg("%sstate=%d", __func__, break_state);
+       dev_dbg(&port->dev, "%s - state=%d\n", __func__, break_state);
 
        /* LOCKING */
        if (break_state)
@@ -645,7 +645,7 @@ static int klsi_105_tiocmget(struct tty_struct *tty)
        spin_lock_irqsave(&priv->lock, flags);
        priv->line_state = line_state;
        spin_unlock_irqrestore(&priv->lock, flags);
-       dbg("%s - read line state 0x%lx", __func__, line_state);
+       dev_dbg(&port->dev, "%s - read line state 0x%lx\n", __func__, line_state);
        return (int)line_state;
 }