On Thu, Jun 12, 2014 at 09:28:56AM +0200, mathias.engan@xxxxxxxxx wrote:
> From: Mathias Engan <mathias.engan@xxxxxxxxx>
>
> Add blank line after declarations in eucr_suspend and eucr_resume
> as reported by checkpatch.
>
> Signed-off-by: Mathias Engan <mathias.engan@xxxxxxxxx>
> ---
> drivers/staging/keucr/usb.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/staging/keucr/usb.c b/drivers/staging/keucr/usb.c
> index 12ebde7..dbbd63f 100644
> --- a/drivers/staging/keucr/usb.c
> +++ b/drivers/staging/keucr/usb.c
> @@ -37,6 +37,7 @@ MODULE_DEVICE_TABLE(usb, eucr_usb_ids);
> 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");
> /* Wait until no command is running */
> mutex_lock(&us->dev_mutex);
> @@ -53,6 +54,7 @@ static int eucr_resume(struct usb_interface *iface)
> u8 tmp = 0;
>
> struct us_data *us = usb_get_intfdata(iface);
Shouldn't you remove the blank line above this variable as well?
thanks,
greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
