USB: remove use of err() in drivers/usb/serial

err() is going away, so switch to dev_err() or printk() if it's really
needed.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

1 files changed, 67 insertions, 46 deletions

diff --git a/drivers/usb/serial/whiteheat.c b/drivers/usb/serial/whiteheat.c
index 39f7f8d291d6..5335d3211c07 100644
--- a/drivers/usb/serial/whiteheat.c
+++ b/drivers/usb/serial/whiteheat.c
@@ -303,12 +303,15 @@ static int whiteheat_firmware_download(struct usb_serial *serial,
 
         if (request_ihex_firmware(&firmware_fw, "whiteheat.fw",
                                   &serial->dev->dev)) {
-                err("%s - request \"whiteheat.fw\" failed", __func__);
+                dev_err(&serial->dev->dev,
+                        "%s - request \"whiteheat.fw\" failed\n", __func__);
                 goto out;
         }
         if (request_ihex_firmware(&loader_fw, "whiteheat_loader.fw",
                              &serial->dev->dev)) {
-                err("%s - request \"whiteheat_loader.fw\" failed", __func__);
+                dev_err(&serial->dev->dev,
+                        "%s - request \"whiteheat_loader.fw\" failed\n",
+                        __func__);
                 goto out;
         }
         ret = 0;
@@ -320,9 +323,10 @@ static int whiteheat_firmware_download(struct usb_serial *serial,
                                               (unsigned char *)record->data,
                                               be16_to_cpu(record->len), 0xa0);
                 if (response < 0) {
-                        err("%s - ezusb_writememory failed for loader (%d %04X %p %d)",
-                            __func__, response, be32_to_cpu(record->addr),
-                            record->data, be16_to_cpu(record->len));
+                        dev_err(&serial->dev->dev, "%s - ezusb_writememory "
+                                "failed for loader (%d %04X %p %d)\n",
+                                __func__, response, be32_to_cpu(record->addr),
+                                record->data, be16_to_cpu(record->len));
                         break;
                 }
                 record = ihex_next_binrec(record);
@@ -338,9 +342,11 @@ static int whiteheat_firmware_download(struct usb_serial *serial,
                                               (unsigned char *)record->data,
                                               be16_to_cpu(record->len), 0xa3);
                 if (response < 0) {
-                        err("%s - ezusb_writememory failed for first firmware step (%d %04X %p %d)", 
-                            __func__, response, be32_to_cpu(record->addr),
-                            record->data, be16_to_cpu(record->len));
+                        dev_err(&serial->dev->dev, "%s - ezusb_writememory "
+                                "failed for first firmware step "
+                                "(%d %04X %p %d)\n", __func__, response,
+                                be32_to_cpu(record->addr), record->data,
+                                be16_to_cpu(record->len));
                         break;
                 }
                 ++record;
@@ -354,9 +360,11 @@ static int whiteheat_firmware_download(struct usb_serial *serial,
                                               (unsigned char *)record->data,
                                               be16_to_cpu(record->len), 0xa0);
                 if (response < 0) {
-                        err("%s - ezusb_writememory failed for second firmware step (%d %04X %p %d)", 
-                            __func__, response, be32_to_cpu(record->addr),
-                            record->data, be16_to_cpu(record->len));
+                        dev_err(&serial->dev->dev, "%s - ezusb_writememory "
+                                "failed for second firmware step "
+                                "(%d %04X %p %d)\n", __func__, response,
+                                be32_to_cpu(record->addr), record->data,
+                                be16_to_cpu(record->len));
                         break;
                 }
                 ++record;
@@ -421,12 +429,12 @@ static int whiteheat_attach(struct usb_serial *serial)
         ret = usb_bulk_msg(serial->dev, pipe, command, 2,
                                                 &alen, COMMAND_TIMEOUT_MS);
         if (ret) {
-                err("%s: Couldn't send command [%d]",
-                                serial->type->description, ret);
+                dev_err(&serial->dev->dev, "%s: Couldn't send command [%d]\n",
+                        serial->type->description, ret);
                 goto no_firmware;
         } else if (alen != 2) {
-                err("%s: Send command incomplete [%d]",
-                                serial->type->description, alen);
+                dev_err(&serial->dev->dev, "%s: Send command incomplete [%d]\n",
+                        serial->type->description, alen);
                 goto no_firmware;
         }
 
@@ -437,16 +445,16 @@ static int whiteheat_attach(struct usb_serial *serial)
         ret = usb_bulk_msg(serial->dev, pipe, result,
                         sizeof(*hw_info) + 1, &alen, COMMAND_TIMEOUT_MS);
         if (ret) {
-                err("%s: Couldn't get results [%d]",
-                                serial->type->description, ret);
+                dev_err(&serial->dev->dev, "%s: Couldn't get results [%d]\n",
+                        serial->type->description, ret);
                 goto no_firmware;
         } else if (alen != sizeof(*hw_info) + 1) {
-                err("%s: Get results incomplete [%d]",
-                                serial->type->description, alen);
+                dev_err(&serial->dev->dev, "%s: Get results incomplete [%d]\n",
+                        serial->type->description, alen);
                 goto no_firmware;
         } else if (result[0] != command[0]) {
-                err("%s: Command failed [%d]",
-                                serial->type->description, result[0]);
+                dev_err(&serial->dev->dev, "%s: Command failed [%d]\n",
+                        serial->type->description, result[0]);
                 goto no_firmware;
         }
 
@@ -461,8 +469,9 @@ static int whiteheat_attach(struct usb_serial *serial)
 
                 info = kmalloc(sizeof(struct whiteheat_private), GFP_KERNEL);
                 if (info == NULL) {
-                        err("%s: Out of memory for port structures\n",
-                                        serial->type->description);
+                        dev_err(&port->dev,
+                                "%s: Out of memory for port structures\n",
+                                serial->type->description);
                         goto no_private;
                 }
 
@@ -482,18 +491,20 @@ static int whiteheat_attach(struct usb_serial *serial)
                 for (j = 0; j < urb_pool_size; j++) {
                         urb = usb_alloc_urb(0, GFP_KERNEL);
                         if (!urb) {
-                                err("No free urbs available");
+                                dev_err(&port->dev, "No free urbs available\n");
                                 goto no_rx_urb;
                         }
                         buf_size = port->read_urb->transfer_buffer_length;
                         urb->transfer_buffer = kmalloc(buf_size, GFP_KERNEL);
                         if (!urb->transfer_buffer) {
-                                err("Couldn't allocate urb buffer");
+                                dev_err(&port->dev,
+                                        "Couldn't allocate urb buffer\n");
                                 goto no_rx_buf;
                         }
                         wrap = kmalloc(sizeof(*wrap), GFP_KERNEL);
                         if (!wrap) {
-                                err("Couldn't allocate urb wrapper");
+                                dev_err(&port->dev,
+                                        "Couldn't allocate urb wrapper\n");
                                 goto no_rx_wrap;
                         }
                         usb_fill_bulk_urb(urb, serial->dev,
@@ -506,18 +517,20 @@ static int whiteheat_attach(struct usb_serial *serial)
 
                         urb = usb_alloc_urb(0, GFP_KERNEL);
                         if (!urb) {
-                                err("No free urbs available");
+                                dev_err(&port->dev, "No free urbs available\n");
                                 goto no_tx_urb;
                         }
                         buf_size = port->write_urb->transfer_buffer_length;
                         urb->transfer_buffer = kmalloc(buf_size, GFP_KERNEL);
                         if (!urb->transfer_buffer) {
-                                err("Couldn't allocate urb buffer");
+                                dev_err(&port->dev,
+                                        "Couldn't allocate urb buffer\n");
                                 goto no_tx_buf;
                         }
                         wrap = kmalloc(sizeof(*wrap), GFP_KERNEL);
                         if (!wrap) {
-                                err("Couldn't allocate urb wrapper");
+                                dev_err(&port->dev,
+                                        "Couldn't allocate urb wrapper\n");
                                 goto no_tx_wrap;
                         }
                         usb_fill_bulk_urb(urb, serial->dev,
@@ -535,8 +548,9 @@ static int whiteheat_attach(struct usb_serial *serial)
         command_info = kmalloc(sizeof(struct whiteheat_command_private),
                                                                 GFP_KERNEL);
         if (command_info == NULL) {
-                err("%s: Out of memory for port structures\n",
-                                        serial->type->description);
+                dev_err(&serial->dev->dev,
+                        "%s: Out of memory for port structures\n",
+                        serial->type->description);
                 goto no_command_private;
         }
 
@@ -553,12 +567,15 @@ static int whiteheat_attach(struct usb_serial *serial)
 
 no_firmware:
         /* Firmware likely not running */
-        err("%s: Unable to retrieve firmware version, try replugging\n",
-                                        serial->type->description);
-        err("%s: If the firmware is not running (status led not blinking)\n",
-                                        serial->type->description);
-        err("%s: please contact support@connecttech.com\n",
-                                        serial->type->description);
+        dev_err(&serial->dev->dev,
+                "%s: Unable to retrieve firmware version, try replugging\n",
+                serial->type->description);
+        dev_err(&serial->dev->dev,
+                "%s: If the firmware is not running (status led not blinking)\n",
+                serial->type->description);
+        dev_err(&serial->dev->dev,
+                "%s: please contact support@connecttech.com\n",
+                serial->type->description);
         kfree(result);
         return -ENODEV;
 
@@ -681,8 +698,9 @@ static int whiteheat_open(struct tty_struct *tty,
         /* Start reading from the device */
         retval = start_port_read(port);
         if (retval) {
-                err("%s - failed submitting read urb, error %d",
-                                __func__, retval);
+                dev_err(&port->dev,
+                        "%s - failed submitting read urb, error %d\n",
+                        __func__, retval);
                 firm_close(port);
                 stop_command_port(port->serial);
                 goto exit;
@@ -807,8 +825,9 @@ static int whiteheat_write(struct tty_struct *tty,
                 urb->transfer_buffer_length = bytes;
                 result = usb_submit_urb(urb, GFP_ATOMIC);
                 if (result) {
-                        err("%s - failed submitting write urb, error %d",
-                                                        __func__, result);
+                        dev_err(&port->dev,
+                                "%s - failed submitting write urb, error %d\n",
+                                __func__, result);
                         sent = result;
                         spin_lock_irqsave(&info->lock, flags);
                         list_add(tmp, &info->tx_urbs_free);
@@ -1076,7 +1095,7 @@ static void whiteheat_read_callback(struct urb *urb)
         wrap = urb_to_wrap(urb, &info->rx_urbs_submitted);
         if (!wrap) {
                 spin_unlock(&info->lock);
-                err("%s - Not my urb!", __func__);
+                dev_err(&port->dev, "%s - Not my urb!\n", __func__);
                 return;
         }
         list_del(&wrap->list);
@@ -1120,7 +1139,7 @@ static void whiteheat_write_callback(struct urb *urb)
         wrap = urb_to_wrap(urb, &info->tx_urbs_submitted);
         if (!wrap) {
                 spin_unlock(&info->lock);
-                err("%s - Not my urb!", __func__);
+                dev_err(&port->dev, "%s - Not my urb!\n", __func__);
                 return;
         }
         list_move(&wrap->list, &info->tx_urbs_free);
@@ -1384,8 +1403,9 @@ static int start_command_port(struct usb_serial *serial)
                 command_port->read_urb->dev = serial->dev;
                 retval = usb_submit_urb(command_port->read_urb, GFP_KERNEL);
                 if (retval) {
-                        err("%s - failed submitting read urb, error %d",
-                                                        __func__, retval);
+                        dev_err(&serial->dev->dev,
+                                "%s - failed submitting read urb, error %d\n",
+                                __func__, retval);
                         goto exit;
                 }
         }
@@ -1523,7 +1543,8 @@ static void rx_data_softint(struct work_struct *work)
                 urb->dev = port->serial->dev;
                 result = usb_submit_urb(urb, GFP_ATOMIC);
                 if (result) {
-                        err("%s - failed resubmitting read urb, error %d",
+                        dev_err(&port->dev,
+                                "%s - failed resubmitting read urb, error %d\n",
                                 __func__, result);
                         spin_lock_irqsave(&info->lock, flags);
                         list_add(tmp, &info->rx_urbs_free);