All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] staging: media: lirc: Remove multiple blank lines
@ 2016-09-24 16:45 Namrata A Shettar
  2016-09-25 10:04 ` Greg Kroah-Hartman
  0 siblings, 1 reply; 3+ messages in thread
From: Namrata A Shettar @ 2016-09-24 16:45 UTC (permalink / raw)
  To: Jarod Wilson, Mauro Carvalho Chehab, Greg Kroah-Hartman,
	Wolfram Sang, Maciek Borzecki, Arnd Bergmann, outreachy-kernel

Remove multiple blank lines to resolve checkpatch issue.

Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
---
 drivers/staging/media/lirc/lirc_sasem.c | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/drivers/staging/media/lirc/lirc_sasem.c b/drivers/staging/media/lirc/lirc_sasem.c
index 6ea1947..4678ae1 100644
--- a/drivers/staging/media/lirc/lirc_sasem.c
+++ b/drivers/staging/media/lirc/lirc_sasem.c
@@ -47,7 +47,6 @@
 #include <media/lirc.h>
 #include <media/lirc_dev.h>
 
-
 #define MOD_AUTHOR	"Oliver Stabel <oliver.stabel@gmx.de>, " \
 			"Tim Davies <tim@opensystems.net.au>"
 #define MOD_DESC	"USB Driver for Sasem Remote Controller V1.1"
@@ -86,7 +85,6 @@ static void ir_close(void *data);
 #define SASEM_DATA_BUF_SZ	32
 
 struct sasem_context {
-
 	struct usb_device *dev;
 	int vfd_isopen;			/* VFD port has been opened */
 	unsigned int vfd_contrast;	/* VFD contrast */
@@ -156,7 +154,6 @@ static int debug;
 
 
 /*** M O D U L E   C O D E ***/
-
 MODULE_AUTHOR(MOD_AUTHOR);
 MODULE_DESCRIPTION(MOD_DESC);
 MODULE_LICENSE("GPL");
@@ -186,7 +183,6 @@ static void deregister_from_lirc(struct sasem_context *context)
 	else
 		dev_info(&context->dev->dev,
 			 "Deregistered Sasem driver (minor:%d)\n", minor);
-
 }
 
 /**
@@ -297,7 +293,6 @@ static int vfd_close(struct inode *inode, struct file *file)
 		context->vfd_isopen = 0;
 		dev_info(&context->dev->dev, "VFD port closed\n");
 		if (!context->dev_present && !context->ir_isopen) {
-
 			/* Device disconnected before close and IR port is
 			 * not open. If IR port is open, context will be
 			 * deleted by ir_close. */
@@ -546,9 +541,7 @@ static void ir_close(void *data)
 		 * at disconnect time, so do it now.
 		 */
 		deregister_from_lirc(context);
-
 		if (!context->vfd_isopen) {
-
 			mutex_unlock(&context->ctx_lock);
 			delete_context(context);
 			return;
@@ -633,7 +626,6 @@ static void usb_rx_callback(struct urb *urb)
 		return;
 
 	switch (urb->status) {
-
 	case -ENOENT:		/* usbcore unlink successful! */
 		return;
 
@@ -651,8 +643,6 @@ static void usb_rx_callback(struct urb *urb)
 	usb_submit_urb(context->rx_urb, GFP_ATOMIC);
 }
 
-
-
 /**
  * Callback function for USB core API: Probe
  */
@@ -709,7 +699,6 @@ static int sasem_probe(struct usb_interface *interface,
 
 		} else if (!vfd_ep_found &&
 			usb_endpoint_is_int_out(ep)) {
-
 			tx_endpoint = ep;
 			vfd_ep_found = 1;
 			if (debug)
-- 
2.7.4



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] staging: media: lirc: Remove multiple blank lines
  2016-09-24 16:45 [PATCH] staging: media: lirc: Remove multiple blank lines Namrata A Shettar
@ 2016-09-25 10:04 ` Greg Kroah-Hartman
  2016-09-25 11:53   ` Namrata A Shettar
  0 siblings, 1 reply; 3+ messages in thread
From: Greg Kroah-Hartman @ 2016-09-25 10:04 UTC (permalink / raw)
  To: Namrata A Shettar
  Cc: Jarod Wilson, Mauro Carvalho Chehab, Wolfram Sang,
	Maciek Borzecki, Arnd Bergmann, outreachy-kernel

On Sat, Sep 24, 2016 at 10:15:30PM +0530, Namrata A Shettar wrote:
> Remove multiple blank lines to resolve checkpatch issue.
> 
> Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> ---
>  drivers/staging/media/lirc/lirc_sasem.c | 11 -----------
>  1 file changed, 11 deletions(-)

You sent another patch with this same subject: as part of a larger
series, what makes this one different?

thanks,

greg k-h


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] staging: media: lirc: Remove multiple blank lines
  2016-09-25 10:04 ` Greg Kroah-Hartman
@ 2016-09-25 11:53   ` Namrata A Shettar
  0 siblings, 0 replies; 3+ messages in thread
From: Namrata A Shettar @ 2016-09-25 11:53 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Jarod Wilson, Mauro Carvalho Chehab,
	Wolfram Sang, Maciek Borzecki, Arnd Bergmann, outreachy-kernel

On Sun, Sep 25, 2016 at 12:04:59PM +0200, Greg Kroah-Hartman wrote:
> On Sat, Sep 24, 2016 at 10:15:30PM +0530, Namrata A Shettar wrote:
> > Remove multiple blank lines to resolve checkpatch issue.
> > 
> > Signed-off-by: Namrata A Shettar <namrataashettar@gmail.com>
> > ---
> >  drivers/staging/media/lirc/lirc_sasem.c | 11 -----------
> >  1 file changed, 11 deletions(-)
> 
> You sent another patch with this same subject: as part of a larger
> series, what makes this one different?

This one I had mistakenly clubbed with [PATCH 6/6] of the previous
series. I have changed the subject line and resent this one as another
patch. 

Thanks,

namrata

> 
> thanks,
> 
> greg k-h


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-09-25 11:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-24 16:45 [PATCH] staging: media: lirc: Remove multiple blank lines Namrata A Shettar
2016-09-25 10:04 ` Greg Kroah-Hartman
2016-09-25 11:53   ` Namrata A Shettar

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.