Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-05-01 Thread gre...@linuxfoundation.org
On Fri, May 01, 2015 at 10:28:00AM +0100, Alan Cox wrote: On Fri, 2015-05-01 at 09:41 +0200, gre...@linuxfoundation.org wrote: On Thu, Apr 30, 2015 at 11:25:48PM +0100, One Thousand Gnomes wrote: On Thu, 30 Apr 2015 16:14:06 +0200 gre...@linuxfoundation.org gre...@linuxfoundation.org

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-05-01 Thread Dan Carpenter
On Thu, Apr 30, 2015 at 10:05:00PM +0530, hari prasath wrote: Greg I am not sure if this needs to go into next release. I send this patch as checkpatch was complaining about it. May be the owners of this driver can only decide upon it. Alan cox also mentioned the same point that its

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-05-01 Thread Alan Cox
Possibly but that ought to go via staging and really is one for the SCSI folks to call. The dpt_i2o was a bit more common than i2o proper. But if the staging i2o core is removed, doesn't that mean that this driver will stop working? It uses code in uapi i2o.h, which I'm guessing is

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-05-01 Thread gre...@linuxfoundation.org
On Thu, Apr 30, 2015 at 11:25:48PM +0100, One Thousand Gnomes wrote: On Thu, 30 Apr 2015 16:14:06 +0200 gre...@linuxfoundation.org gre...@linuxfoundation.org wrote: On Thu, Apr 23, 2015 at 04:09:28PM +0100, Alan Cox wrote: On Thu, 2015-04-23 at 13:43 +, Gujulan Elango, Hari Prasath

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-05-01 Thread Dan Carpenter
On Fri, May 01, 2015 at 10:18:14AM +0300, Dan Carpenter wrote: On Thu, Apr 30, 2015 at 10:05:00PM +0530, hari prasath wrote: Greg I am not sure if this needs to go into next release. I send this patch as checkpatch was complaining about it. May be the owners of this driver can only

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-05-01 Thread Alan Cox
On Fri, 2015-05-01 at 09:41 +0200, gre...@linuxfoundation.org wrote: On Thu, Apr 30, 2015 at 11:25:48PM +0100, One Thousand Gnomes wrote: On Thu, 30 Apr 2015 16:14:06 +0200 gre...@linuxfoundation.org gre...@linuxfoundation.org wrote: On Thu, Apr 23, 2015 at 04:09:28PM +0100, Alan Cox

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-04-30 Thread gre...@linuxfoundation.org
On Thu, Apr 23, 2015 at 04:09:28PM +0100, Alan Cox wrote: On Thu, 2015-04-23 at 13:43 +, Gujulan Elango, Hari Prasath (H.) wrote: This patch removes unwanted semicolon around close braces of code blocks The i2o driver moved into staging ready to be deleted unless someone steps up

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-04-30 Thread hari prasath
On 30 April 2015 at 19:44, gre...@linuxfoundation.org gre...@linuxfoundation.org wrote: On Thu, Apr 23, 2015 at 04:09:28PM +0100, Alan Cox wrote: On Thu, 2015-04-23 at 13:43 +, Gujulan Elango, Hari Prasath (H.) wrote: This patch removes unwanted semicolon around close braces of code

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-04-30 Thread One Thousand Gnomes
On Thu, 30 Apr 2015 16:14:06 +0200 gre...@linuxfoundation.org gre...@linuxfoundation.org wrote: On Thu, Apr 23, 2015 at 04:09:28PM +0100, Alan Cox wrote: On Thu, 2015-04-23 at 13:43 +, Gujulan Elango, Hari Prasath (H.) wrote: This patch removes unwanted semicolon around close braces

Re: [PATCH] staging: i2o: Remove unwanted semicolon

2015-04-23 Thread Alan Cox
On Thu, 2015-04-23 at 13:43 +, Gujulan Elango, Hari Prasath (H.) wrote: This patch removes unwanted semicolon around close braces of code blocks The i2o driver moved into staging ready to be deleted unless someone steps up with hardware willing to maintain it (which is rather unlikely).

[PATCH] staging: i2o: Remove unwanted semicolon

2015-04-23 Thread Gujulan Elango, Hari Prasath (H.)
This patch removes unwanted semicolon around close braces of code blocks Signed-off-by: Hari Prasath hguju...@visteon.com --- drivers/staging/i2o/iop.c | 44 ++-- 1 file changed, 22 insertions(+), 22 deletions(-) diff --git a/drivers/staging/i2o/iop.c