Oliver Neukum <[email protected]> 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
