Oliver Neukum <oneukum@xxxxxxxx> writes:
> @@ -360,22 +361,22 @@ void usb_serial_generic_read_bulk_callback(struct urb *urb)
>
> dev_dbg(&port->dev, "%s - urb %d, len %d\n", __func__, i,
> urb->actual_length);
> - switch (urb->status) {
> + switch (status) {
> case 0:
> break;
> case -ENOENT:
> case -ECONNRESET:
> case -ESHUTDOWN:
> dev_dbg(&port->dev, "%s - urb stopped: %d\n",
> - __func__, urb->status);
> + __func__, status);
__func__ is redundant in dev_dbg(). Removing it will allow you to
unbreak these lines, making the code look a lot nicer.
Bjørn
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
