[2.6 patch] drivers/scsi/gdth.c: cleanups

From: Adrian Bunk (bunk_at_stusta.de)
Date: 02/28/05

  • Next message: Adrian Bunk: "[-mm patch] drivers/scsi/ch.c: make a struct static"
    Date:	Mon, 28 Feb 2005 19:02:46 +0100
    To: achim_leubner@adaptec.com
    
    

    This patch contains the following cleanups:
    - make some needlessly global functions static
    - remove one more kernel 2.2 #ifdef

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

    ---
     drivers/scsi/gdth.c |   24 ++++++++++++------------
     drivers/scsi/gdth.h |   41 -----------------------------------------
     2 files changed, 12 insertions(+), 53 deletions(-)
    --- linux-2.6.11-rc4-mm1-full/drivers/scsi/gdth.h.old	2005-02-28 18:25:09.000000000 +0100
    +++ linux-2.6.11-rc4-mm1-full/drivers/scsi/gdth.h	2005-02-28 18:31:38.000000000 +0100
    @@ -1029,51 +1029,10 @@
     
     /* function prototyping */
     
    -int gdth_detect(Scsi_Host_Template *);
    -int gdth_release(struct Scsi_Host *);
    -int gdth_queuecommand(Scsi_Cmnd *,void (*done)(Scsi_Cmnd *));
    -const char *gdth_info(struct Scsi_Host *);
    -
     #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,0)
    -int gdth_bios_param(struct scsi_device *,struct block_device *,sector_t,int *);
     int gdth_proc_info(struct Scsi_Host *, char *,char **,off_t,int,int);
    -#elif LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0)
    -int gdth_bios_param(Disk *,kdev_t,int *);
    -int gdth_proc_info(char *,char **,off_t,int,int,int);
     #else
    -int gdth_bios_param(Disk *,kdev_t,int *);
    -extern struct proc_dir_entry proc_scsi_gdth;
     int gdth_proc_info(char *,char **,off_t,int,int,int);
    -int gdth_abort(Scsi_Cmnd *);
    -int gdth_reset(Scsi_Cmnd *,unsigned int); 
    -#define GDTH { proc_dir:        &proc_scsi_gdth,                 \
    -               proc_info:       gdth_proc_info,                  \
    -               name:            "GDT SCSI Disk Array Controller",\
    -               detect:          gdth_detect,                     \
    -               release:         gdth_release,                    \
    -               info:            gdth_info,                       \
    -               command:         NULL,                            \
    -               queuecommand:    gdth_queuecommand,               \
    -               eh_abort_handler: gdth_eh_abort,                  \
    -               eh_device_reset_handler: gdth_eh_device_reset,    \
    -               eh_bus_reset_handler: gdth_eh_bus_reset,          \
    -               eh_host_reset_handler: gdth_eh_host_reset,        \
    -               abort:           gdth_abort,                      \
    -               reset:           gdth_reset,                      \
    -               bios_param:      gdth_bios_param,                 \
    -               can_queue:       GDTH_MAXCMDS,                    \
    -               this_id:         -1,                              \
    -               sg_tablesize:    GDTH_MAXSG,                      \
    -               cmd_per_lun:     GDTH_MAXC_P_L,                   \
    -               present:         0,                               \
    -               unchecked_isa_dma: 1,                             \
    -               use_clustering:  ENABLE_CLUSTERING,               \
    -               use_new_eh_code: 1       /* use new error code */ }    
     #endif
     
    -int gdth_eh_abort(Scsi_Cmnd *scp);
    -int gdth_eh_device_reset(Scsi_Cmnd *scp);
    -int gdth_eh_bus_reset(Scsi_Cmnd *scp);
    -int gdth_eh_host_reset(Scsi_Cmnd *scp);
    -
     #endif
    --- linux-2.6.11-rc4-mm1-full/drivers/scsi/gdth.c.old	2005-02-28 18:25:50.000000000 +0100
    +++ linux-2.6.11-rc4-mm1-full/drivers/scsi/gdth.c	2005-02-28 18:44:14.000000000 +0100
    @@ -4034,7 +4034,7 @@
     }
     
     #ifdef GDTH_STATISTICS
    -void gdth_timeout(ulong data)
    +static void gdth_timeout(ulong data)
     {
         ulong32 i;
         Scsi_Cmnd *nscp;
    @@ -4062,7 +4062,7 @@
     }
     #endif
     
    -void __init internal_setup(char *str,int *ints)
    +static void __init internal_setup(char *str,int *ints)
     {
         int i, argc;
         char *cur_str, *argv;
    @@ -4153,7 +4153,7 @@
         return 1;
     }
     
    -int __init gdth_detect(Scsi_Host_Template *shtp)
    +static int __init gdth_detect(Scsi_Host_Template *shtp)
     {
         struct Scsi_Host *shp;
         gdth_pci_str pcistr[MAXHA];
    @@ -4604,7 +4604,7 @@
     }
     
     
    -int gdth_release(struct Scsi_Host *shp)
    +static int gdth_release(struct Scsi_Host *shp)
     {
         int hanum;
         gdth_ha_str *ha;
    @@ -4691,7 +4691,7 @@
         return("");
     }
     
    -const char *gdth_info(struct Scsi_Host *shp)
    +static const char *gdth_info(struct Scsi_Host *shp)
     {
         int hanum;
         gdth_ha_str *ha;
    @@ -4704,19 +4704,19 @@
     }
     
     /* new error handling */
    -int gdth_eh_abort(Scsi_Cmnd *scp)
    +static int gdth_eh_abort(Scsi_Cmnd *scp)
     {
         TRACE2(("gdth_eh_abort()\n"));
         return FAILED;
     }
     
    -int gdth_eh_device_reset(Scsi_Cmnd *scp)
    +static int gdth_eh_device_reset(Scsi_Cmnd *scp)
     {
         TRACE2(("gdth_eh_device_reset()\n"));
         return FAILED;
     }
     
    -int gdth_eh_bus_reset(Scsi_Cmnd *scp)
    +static int gdth_eh_bus_reset(Scsi_Cmnd *scp)
     {
         int i, hanum;
         gdth_ha_str *ha;
    @@ -4770,7 +4770,7 @@
         return SUCCESS;
     }
     
    -int gdth_eh_host_reset(Scsi_Cmnd *scp)
    +static int gdth_eh_host_reset(Scsi_Cmnd *scp)
     {
         TRACE2(("gdth_eh_host_reset()\n"));
         return FAILED;
    @@ -4778,9 +4778,9 @@
     
     
     #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,0)
    -int gdth_bios_param(struct scsi_device *sdev,struct block_device *bdev,sector_t cap,int *ip)
    +static int gdth_bios_param(struct scsi_device *sdev,struct block_device *bdev,sector_t cap,int *ip)
     #else
    -int gdth_bios_param(Disk *disk,kdev_t dev,int *ip)
    +static int gdth_bios_param(Disk *disk,kdev_t dev,int *ip)
     #endif
     {
         unchar b, t;
    @@ -4818,7 +4818,7 @@
     }
     
     
    -int gdth_queuecommand(Scsi_Cmnd *scp,void (*done)(Scsi_Cmnd *))
    +static int gdth_queuecommand(Scsi_Cmnd *scp,void (*done)(Scsi_Cmnd *))
     {
         int hanum;
         int priority;
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at  http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at  http://www.tux.org/lkml/
    

  • Next message: Adrian Bunk: "[-mm patch] drivers/scsi/ch.c: make a struct static"

    Relevant Pages