On Wed, Jun 11, 2014 at 01:50:51PM -0700, Greg KH wrote: > On Wed, Jun 11, 2014 at 10:23:09PM +0200, mathias.engan@xxxxxxxxx wrote: > > From: Mathias Engan <mathias.engan@xxxxxxxxx> > > static int eucr_suspend(struct usb_interface *iface, pm_message_t message) > > { > > struct us_data *us = usb_get_intfdata(iface); > > + > > pr_info("--- eucr_suspend ---\n"); > > + > > Did checkpatch complain about this line not being here? Because that's > not what you said above in the patch description :( > Ah. No. It did not. However, in the previous, and most of the following functions there is a blank line after pr_info() so I added it so that eucr_suspend() would conform to the style of the other, similar, functions. Would you like me to resubmit or change the patch description? _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel