[2.6 patch] SCSI t128.c: remove an unused function

[2.6 patch] SCSI t128.c: remove an unused function

[Adrian Bunk]

The patch below removes the unused function t128_setup.

Please review whether it's correct.


diffstat output:
 drivers/scsi/t128.c |   28 ----------------------------
 1 files changed, 28 deletions(-)


Signed-off-by: Adrian Bunk <bunk@stusta.de>

--- linux-2.6.10-rc1-mm5-full/drivers/scsi/t128.c.old	2004-11-14 01:31:48.000000000 +0100
+++ linux-2.6.10-rc1-mm5-full/drivers/scsi/t128.c	2004-11-14 01:37:27.000000000 +0100
@@ -154,34 +154,6 @@
 
 #define NO_SIGNATURES (sizeof (signatures) /  sizeof (struct signature))
 
-/*
- * Function : t128_setup(char *str, int *ints)
- *
- * Purpose : LILO command line initialization of the overrides array,
- * 
- * Inputs : str - unused, ints - array of integer parameters with ints[0]
- *	equal to the number of ints.
- *
- */
-
-void __init t128_setup(char *str, int *ints){
-    static int commandline_current = 0;
-    int i;
-    if (ints[0] != 2) 
-	printk("t128_setup : usage t128=address,irq\n");
-    else 
-	if (commandline_current < NO_OVERRIDES) {
-	    overrides[commandline_current].address = ints[1];
-	    overrides[commandline_current].irq = ints[2];
-	    for (i = 0; i < NO_BASES; ++i)
-		if (bases[i].address == ints[1]) {
-		    bases[i].noauto = 1;
-		    break;
-		}
-	    ++commandline_current;
-	}
-}
-
 /* 
  * Function : int t128_detect(Scsi_Host_Template * tpnt)
  *

Re: [2.6 patch] SCSI t128.c: remove an unused function

[Alan Cox]

On Llu, 2004-11-15 at 02:38, Adrian Bunk wrote:
> The patch below removes the unused function t128_setup.
> 
> Please review whether it's correct.

Its wrong. The fix is to make the setup function get called, IFF you can
find anyone with a t128 any more

Re: [2.6 patch] SCSI t128.c: remove an unused function

[Adrian Bunk]

On Mon, Nov 15, 2004 at 02:40:39PM +0000, Alan Cox wrote:
> On Llu, 2004-11-15 at 02:38, Adrian Bunk wrote:
> > The patch below removes the unused function t128_setup.
> > 
> > Please review whether it's correct.
> 
> Its wrong. The fix is to make the setup function get called, IFF you can
> find anyone with a t128 any more

Ah, it seems your t128 fix which did this in 2.4.17-pre7 is (like your 
dtc cleanup in the same patch) among the fixes not yet forward-ported 
from 2.4 to 2.6 ...

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

Re: [2.6 patch] SCSI t128.c: remove an unused function

[Alan Cox]

On Llu, 2004-11-15 at 17:03, Adrian Bunk wrote:
> On Mon, Nov 15, 2004 at 02:40:39PM +0000, Alan Cox wrote:
> > On Llu, 2004-11-15 at 02:38, Adrian Bunk wrote:
> > > The patch below removes the unused function t128_setup.
> > > 
> > > Please review whether it's correct.
> > 
> > Its wrong. The fix is to make the setup function get called, IFF you can
> > find anyone with a t128 any more
> 
> Ah, it seems your t128 fix which did this in 2.4.17-pre7 is (like your 
> dtc cleanup in the same patch) among the fixes not yet forward-ported 
> from 2.4 to 2.6 ...

Yep. Those probably want propogating so they don't get lost. Not that
5380's work in 2.6 with all the scsi changes as far as I can tell