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

[gregkh@xxxxxxxxxxxxxxxxxxx]

On Fri, May 01, 2015 at 10:28:00AM +0100, Alan Cox wrote:
> On Fri, 2015-05-01 at 09:41 +0200, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
> > On Thu, Apr 30, 2015 at 11:25:48PM +0100, One Thousand Gnomes wrote:
> > > On Thu, 30 Apr 2015 16:14:06 +0200
> > > "gregkh@xxxxxxxxxxxxxxxxxxx" <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > > 
> > > > On Thu, Apr 23, 2015 at 04:09:28PM +0100, Alan Cox wrote:
> > > > > On Thu, 2015-04-23 at 13:43 +0000, 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).
> > > > 
> > > > I think it's now time to delete these, want me to do that for 4.2?  I
> > > > can queue that up in my tree now, so that we don't see any more cleanup
> > > > patches being made for them?
> > > 
> > > Yeah I think it can go....
> > 
> > I was about to delete it, but what about drivers/scsi/dpt/dpti_i2o.* ?
> > Should that be removed as well?
> 
> 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 implemented in the staging i2o core.

thanks,

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