Commit 925e4610 authored by Finn Thain's avatar Finn Thain Committed by Christoph Hellwig

ncr5380: Fix compiler warnings and __setup options

Some __setup() options mentioned in Documentation/scsi don't work because
a few lines of code went missing sometime since Linux 2.4. Fix the options
and thus fix some compiler warnings for both the non-modular case,

    CC      drivers/scsi/dtc.o
drivers/scsi/dtc.c:176:20: warning: 'dtc_setup' defined but not used [-Wunused-function]

and the modular case,

    CC [M]  drivers/scsi/pas16.o
drivers/scsi/pas16.c:335:20: warning: 'pas16_setup' defined but not used [-Wunused-function]
    CC [M]  drivers/scsi/t128.o
drivers/scsi/t128.c:147:20: warning: 't128_setup' defined but not used [-Wunused-function]
Signed-off-by: default avatarFinn Thain <fthain@telegraphics.com.au>
Reviewed-by: default avatarHannes Reinecke <hare@suse.de>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent acfc8cad
...@@ -173,10 +173,13 @@ static const struct signature { ...@@ -173,10 +173,13 @@ static const struct signature {
* *
*/ */
static void __init dtc_setup(char *str, int *ints) static int __init dtc_setup(char *str)
{ {
static int commandline_current = 0; static int commandline_current = 0;
int i; int i;
int ints[10];
get_options(str, ARRAY_SIZE(ints), ints);
if (ints[0] != 2) if (ints[0] != 2)
printk("dtc_setup: usage dtc=address,irq\n"); printk("dtc_setup: usage dtc=address,irq\n");
else if (commandline_current < NO_OVERRIDES) { else if (commandline_current < NO_OVERRIDES) {
...@@ -189,7 +192,10 @@ static void __init dtc_setup(char *str, int *ints) ...@@ -189,7 +192,10 @@ static void __init dtc_setup(char *str, int *ints)
} }
++commandline_current; ++commandline_current;
} }
return 1;
} }
__setup("dtc=", dtc_setup);
#endif #endif
/* /*
......
...@@ -337,6 +337,7 @@ static int __init ...@@ -337,6 +337,7 @@ static int __init
} }
#ifndef MODULE
/* /*
* Function : pas16_setup(char *str, int *ints) * Function : pas16_setup(char *str, int *ints)
* *
...@@ -347,10 +348,13 @@ static int __init ...@@ -347,10 +348,13 @@ static int __init
* *
*/ */
void __init pas16_setup(char *str, int *ints) static int __init pas16_setup(char *str)
{ {
static int commandline_current = 0; static int commandline_current = 0;
int i; int i;
int ints[10];
get_options(str, ARRAY_SIZE(ints), ints);
if (ints[0] != 2) if (ints[0] != 2)
printk("pas16_setup : usage pas16=io_port,irq\n"); printk("pas16_setup : usage pas16=io_port,irq\n");
else else
...@@ -364,8 +368,12 @@ void __init pas16_setup(char *str, int *ints) ...@@ -364,8 +368,12 @@ void __init pas16_setup(char *str, int *ints)
} }
++commandline_current; ++commandline_current;
} }
return 1;
} }
__setup("pas16=", pas16_setup);
#endif
/* /*
* Function : int pas16_detect(struct scsi_host_template * tpnt) * Function : int pas16_detect(struct scsi_host_template * tpnt)
* *
......
...@@ -148,6 +148,7 @@ static struct signature { ...@@ -148,6 +148,7 @@ static struct signature {
#define NO_SIGNATURES ARRAY_SIZE(signatures) #define NO_SIGNATURES ARRAY_SIZE(signatures)
#ifndef MODULE
/* /*
* Function : t128_setup(char *str, int *ints) * Function : t128_setup(char *str, int *ints)
* *
...@@ -158,9 +159,13 @@ static struct signature { ...@@ -158,9 +159,13 @@ static struct signature {
* *
*/ */
void __init t128_setup(char *str, int *ints){ static int __init t128_setup(char *str)
{
static int commandline_current = 0; static int commandline_current = 0;
int i; int i;
int ints[10];
get_options(str, ARRAY_SIZE(ints), ints);
if (ints[0] != 2) if (ints[0] != 2)
printk("t128_setup : usage t128=address,irq\n"); printk("t128_setup : usage t128=address,irq\n");
else else
...@@ -174,8 +179,12 @@ void __init t128_setup(char *str, int *ints){ ...@@ -174,8 +179,12 @@ void __init t128_setup(char *str, int *ints){
} }
++commandline_current; ++commandline_current;
} }
return 1;
} }
__setup("t128=", t128_setup);
#endif
/* /*
* Function : int t128_detect(struct scsi_host_template * tpnt) * Function : int t128_detect(struct scsi_host_template * tpnt)
* *
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment