On Sat, Apr 18, 2009 at 10:55:06PM +0930, Tony Cook wrote: > @@ -493,7 +493,7 @@ > spin_unlock(&mos7840_port->pool_lock); > if (result) { > dev_err(&urb->dev->dev, > - "%s - Error %d submitting interrupt urb\n", > + "%s - Error %d submitting interrupt urb", > __func__, result); > } > } No, the dev_* functions need the trailing \n characters. Don't worry, I edited your patch so you don't have to resend it again :) thanks, greg k-h -- 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