public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Staging: comedi: fix blank line coding style in comedi_bond.c
@ 2014-12-18  2:35 Frederic Jacob
  2014-12-18  2:57 ` Greg KH
  0 siblings, 1 reply; 3+ messages in thread
From: Frederic Jacob @ 2014-12-18  2:35 UTC (permalink / raw)
  To: gregkh; +Cc: linux-kernel, Frederic Jacob

This is a patch to the comedi_bond.c file that fixes up a blank line after declaration warning found by the checkpatch.pl tool
Signed-off-by: Frederic Jacob <frederic.jacob.78@gmail.com>
---
This is my first patch that I send. It is for the Eudyptula chalange task 10.
 drivers/staging/comedi/drivers/comedi_bond.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/staging/comedi/drivers/comedi_bond.c b/drivers/staging/comedi/drivers/comedi_bond.c
index 85b2f4a..221d381 100644
--- a/drivers/staging/comedi/drivers/comedi_bond.c
+++ b/drivers/staging/comedi/drivers/comedi_bond.c
@@ -261,6 +261,7 @@ static int do_dev_config(struct comedi_device *dev, struct comedi_devconfig *it)
 			{
 				/* Append dev:subdev to devpriv->name */
 				char buf[20];
+
 				snprintf(buf, sizeof(buf), "%u:%u ",
 					 bdev->minor, bdev->subdev);
 				strlcat(devpriv->name, buf,
-- 
1.9.1


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

* Re: [PATCH] Staging: comedi: fix blank line coding style in comedi_bond.c
  2014-12-18  2:35 [PATCH] Staging: comedi: fix blank line coding style in comedi_bond.c Frederic Jacob
@ 2014-12-18  2:57 ` Greg KH
  2014-12-18  5:15   ` [PATCHv2] " Frederic Jacob
  0 siblings, 1 reply; 3+ messages in thread
From: Greg KH @ 2014-12-18  2:57 UTC (permalink / raw)
  To: Frederic Jacob; +Cc: linux-kernel

On Wed, Dec 17, 2014 at 09:35:41PM -0500, Frederic Jacob wrote:
> This is a patch to the comedi_bond.c file that fixes up a blank line after declaration warning found by the checkpatch.pl tool

Please line-wrap your line properly.


> Signed-off-by: Frederic Jacob <frederic.jacob.78@gmail.com>

And provide an empty line before the signed-off-by: line.

Can you fix that up and resend?

thanks,

greg k-h

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

* [PATCHv2] Staging: comedi: fix blank line coding style in comedi_bond.c
  2014-12-18  2:57 ` Greg KH
@ 2014-12-18  5:15   ` Frederic Jacob
  0 siblings, 0 replies; 3+ messages in thread
From: Frederic Jacob @ 2014-12-18  5:15 UTC (permalink / raw)
  To: gregkh; +Cc: linux-kernel, Frederic Jacob

 This is a patch to the comedi_bond.c file that fixes up a blank line after
 declaration warning found by the checkpatch.pl tool 

Signed-off-by: Frederic Jacob <frederic.jacob.78@gmail.com>
---
v2: Fix description line-wrap and add a line before Signed-off-by
 drivers/staging/comedi/drivers/comedi_bond.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/staging/comedi/drivers/comedi_bond.c b/drivers/staging/comedi/drivers/comedi_bond.c
index 85b2f4a..221d381 100644
--- a/drivers/staging/comedi/drivers/comedi_bond.c
+++ b/drivers/staging/comedi/drivers/comedi_bond.c
@@ -261,6 +261,7 @@ static int do_dev_config(struct comedi_device *dev, struct comedi_devconfig *it)
 			{
 				/* Append dev:subdev to devpriv->name */
 				char buf[20];
+
 				snprintf(buf, sizeof(buf), "%u:%u ",
 					 bdev->minor, bdev->subdev);
 				strlcat(devpriv->name, buf,
-- 
1.9.1


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

end of thread, other threads:[~2014-12-18  5:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-18  2:35 [PATCH] Staging: comedi: fix blank line coding style in comedi_bond.c Frederic Jacob
2014-12-18  2:57 ` Greg KH
2014-12-18  5:15   ` [PATCHv2] " Frederic Jacob

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox