Re: [patch 02/25] drivers/scsi/advansys.c: cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2007-05-23 at 16:33 -0700, Andrew Morton wrote:
> On Wed, 23 May 2007 18:21:31 -0500
> James Bottomley <James.Bottomley@xxxxxxxxxxxx> wrote:
> 
> > On Wed, 2007-05-23 at 14:41 -0700, akpm@xxxxxxxxxxxxxxxxxxxx wrote:
> > > From: Adrian Bunk <bunk@xxxxxxxxx>
> > > 
> > > - remove the unneeded advansys.h
> > > - remove the unused advansys_setup()
> > 
> > This isn't quite right, though, is it?  advansys_setup() should be used.
> > 
> > It looks like there was a bogus conversion away from init/main.c which
> > left the function dangling.  Isn't the correct thing to do to wire it up
> > in advansys_detect() which is where it should have been in the first
> > place.
> > 
> 
> Even going back to linux-2.4.2 I can find no caller to advansys_setup(). 
> Or was its call buried in the magical template macros?

If I remember rightly, the bootsetups[] array was a 2.0 thing it became
kernel_param in 2.1.75.  It was gone by 2.4.0, so this removal thing
must have been in the 2.3 timeframe.

Still ... it can be put back .. it looks basically sound.

James




-
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux