Re: [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon

2013-08-04 Thread Aaro Koskinen
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 
> ---
>  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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging:xgifb:Fixed a warning of Space before semicolon

2013-08-04 Thread Kumar Gaurav
Fixed issue by removing Space before semicolon

Signed-off-by: Kumar Gaurav 
---
 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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon

2013-08-04 Thread Dan Carpenter
On Sun, Aug 04, 2013 at 11:28:21PM +0530, kumargauravgup...@gmail.com wrote:
> From: Kumar Gaurav 

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging:xgifb:Fixed a warning of Space before semicolon

2013-08-04 Thread kumargauravgupta3
From: Kumar Gaurav 

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging:xgifb:Fixed a warning of Space before semicolon

2013-08-04 Thread kumargauravgupta3
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

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon

2013-08-04 Thread Dan Carpenter
On Sun, Aug 04, 2013 at 11:28:21PM +0530, kumargauravgup...@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

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] Staging:xgifb:Fixed a warning of Space before semicolon

2013-08-04 Thread Kumar Gaurav
Fixed issue by removing Space before semicolon

Signed-off-by: Kumar Gaurav kumargauravgup...@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

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] Staging:xgifb:Fixed a warning of Space before semicolon

2013-08-04 Thread Aaro Koskinen
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 kumargauravgup...@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.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/