public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon
@ 2013-08-04 17:58 kumargauravgupta3
  2013-08-04 18:08 ` Dan Carpenter
  0 siblings, 1 reply; 4+ messages in thread
From: kumargauravgupta3 @ 2013-08-04 17:58 UTC (permalink / raw)
  To: arnaud.patard, gregkh, aaro.koskinen, dan.carpenter
  Cc: linux-kernel, kernel-janitors

From: Kumar Gaurav <kumar@AnandLinux.(none)>

diff --git a/drivers/staging/xgifb/vb_setmode.h b/drivers/staging/xgifb/vb_setmode.h
index 2c0a31c..679a822 100644
--- a/drivers/staging/xgifb/vb_setmode.h
+++ b/drivers/staging/xgifb/vb_setmode.h
@@ -13,7 +13,7 @@ extern void XGI_GetVBType(struct vb_device_info *);
 extern void XGI_SenseCRT1(struct vb_device_info *);
 extern unsigned char XGISetModeNew(struct xgifb_video_info *xgifb_info,
 				   struct xgi_hw_device_info *HwDeviceExtension,
-				   unsigned short ModeNo) ;
+				   unsigned short ModeNo);
 
 extern unsigned char XGI_SearchModeID(unsigned short ModeNo,
 				      unsigned short *ModeIdIndex,
-- 
1.7.9.5


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

* Re: [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon
  2013-08-04 17:58 [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon kumargauravgupta3
@ 2013-08-04 18:08 ` Dan Carpenter
  0 siblings, 0 replies; 4+ messages in thread
From: Dan Carpenter @ 2013-08-04 18:08 UTC (permalink / raw)
  To: kumargauravgupta3
  Cc: arnaud.patard, gregkh, aaro.koskinen, linux-kernel,
	kernel-janitors

On Sun, Aug 04, 2013 at 11:28:21PM +0530, kumargauravgupta3@gmail.com wrote:
> From: Kumar Gaurav <kumar@AnandLinux.(none)>

Gar...  Sorry, when you sent this to kj earlier, I should have
pointed out that there is no Signed-off-by line and the From line
is not correct or needed.

Signed-off-by is intended like a signing a legal document, it should
match the email you are using to send the patch.

My bad.  Can you resend.

regards,
dan carpenter


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

* [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon
@ 2013-08-04 18:41 Kumar Gaurav
  2013-08-04 18:56 ` Aaro Koskinen
  0 siblings, 1 reply; 4+ messages in thread
From: Kumar Gaurav @ 2013-08-04 18:41 UTC (permalink / raw)
  To: arnaud.patard, gregkh, aaro.koskinen, dan.carpenter
  Cc: linux-kernel, kernel-janitors, Kumar Gaurav

Fixed issue by removing Space before semicolon

Signed-off-by: Kumar Gaurav <kumargauravgupta3@gmail.com>
---
 drivers/staging/xgifb/vb_setmode.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/xgifb/vb_setmode.h b/drivers/staging/xgifb/vb_setmode.h
index 2c0a31c..679a822 100644
--- a/drivers/staging/xgifb/vb_setmode.h
+++ b/drivers/staging/xgifb/vb_setmode.h
@@ -13,7 +13,7 @@ extern void XGI_GetVBType(struct vb_device_info *);
 extern void XGI_SenseCRT1(struct vb_device_info *);
 extern unsigned char XGISetModeNew(struct xgifb_video_info *xgifb_info,
 				   struct xgi_hw_device_info *HwDeviceExtension,
-				   unsigned short ModeNo) ;
+				   unsigned short ModeNo);
 
 extern unsigned char XGI_SearchModeID(unsigned short ModeNo,
 				      unsigned short *ModeIdIndex,
-- 
1.7.9.5


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

* Re: [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon
  2013-08-04 18:41 Kumar Gaurav
@ 2013-08-04 18:56 ` Aaro Koskinen
  0 siblings, 0 replies; 4+ messages in thread
From: Aaro Koskinen @ 2013-08-04 18:56 UTC (permalink / raw)
  To: Kumar Gaurav
  Cc: arnaud.patard, gregkh, dan.carpenter, linux-kernel,
	kernel-janitors

On Mon, Aug 05, 2013 at 12:11:02AM +0530, Kumar Gaurav wrote:
> Fixed issue by removing Space before semicolon
> 
> Signed-off-by: Kumar Gaurav <kumargauravgupta3@gmail.com>
> ---
>  drivers/staging/xgifb/vb_setmode.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/xgifb/vb_setmode.h b/drivers/staging/xgifb/vb_setmode.h
> index 2c0a31c..679a822 100644
> --- a/drivers/staging/xgifb/vb_setmode.h
> +++ b/drivers/staging/xgifb/vb_setmode.h
> @@ -13,7 +13,7 @@ extern void XGI_GetVBType(struct vb_device_info *);
>  extern void XGI_SenseCRT1(struct vb_device_info *);
>  extern unsigned char XGISetModeNew(struct xgifb_video_info *xgifb_info,
>  				   struct xgi_hw_device_info *HwDeviceExtension,
> -				   unsigned short ModeNo) ;
> +				   unsigned short ModeNo);

There's a similar issue in vb_table.h, line 1287. Maybe fix that also
at the same time?

Thanks,

A.

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

end of thread, other threads:[~2013-08-04 18:56 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-04 17:58 [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon kumargauravgupta3
2013-08-04 18:08 ` Dan Carpenter
  -- strict thread matches above, loose matches on Subject: below --
2013-08-04 18:41 Kumar Gaurav
2013-08-04 18:56 ` Aaro Koskinen

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