drivers/scsi/aic7xxx/: possible cleanups
Subsystems (2)
PATCH 1/1 drivers/scsi/aic7xxx/: possible cleanups
This patch contains the following possible cleanups:
- make needlessly global code static
- #if 0 the following unused global functions:
- aic79xx_core.c: ahd_print_scb
- aic79xx_core.c: ahd_suspend
- aic79xx_core.c: ahd_resume
- aic79xx_core.c: ahd_dump_scbs
- aic79xx_osm.c: ahd_softc_comp
Signed-off-by Adrian Bunk <[email protected]>
---
This patch was already sent on:
- 10 Jan 2006
drivers/scsi/aic7xxx/aic79xx.h | 53 ----- drivers/scsi/aic7xxx/aic79xx_core.c | 262 +++++++++++++++---------- drivers/scsi/aic7xxx/aic79xx_inline.h | 30 -- drivers/scsi/aic7xxx/aic79xx_osm.c | 4 drivers/scsi/aic7xxx/aic79xx_osm.h | 6 drivers/scsi/aic7xxx/aic79xx_osm_pci.c | 2 drivers/scsi/aic7xxx/aic79xx_pci.c | 7 drivers/scsi/aic7xxx/aic79xx_proc.c | 2 drivers/scsi/aic7xxx/aic7xxx.h | 2 drivers/scsi/aic7xxx/aic7xxx_osm.c | 2 drivers/scsi/aic7xxx/aic7xxx_osm.h | 2 drivers/scsi/aic7xxx/aic7xxx_osm_pci.c | 2 drivers/scsi/aic7xxx/aic7xxx_pci.c | 4 13 files changed, 178 insertions(+), 200 deletions(-)
drivers/scsi/aic7xxx/aic79xx_inline.h renamed from drivers/scsi/aic7xxx/aic79xx_inline.h~drivers-scsi-aic7xxx-possible-cleanups +0 -30
@@ -418,10 +418,6 @@ ahd_targetcmd_offset(struct ahd_softc *a }/*********************** Miscelaneous Support Functions ***********************/ -static __inlinevoid ahd_complete_scb (struct ahd_softc *ahd, -struct scb *scb); -static __inlinevoid ahd_update_residual (struct ahd_softc *ahd, -struct scb *scb);static __inlinestruct ahd_initiator_tinfo * ahd_fetch_transinfo(struct ahd_softc *ahd, char channel, u_int our_id,
@@ -467,32 +463,6 @@ static __inline uint32_t ahd_get_sense_bufaddr(struct ahd_softc *ahd, struct scb *scb); -static __inlinevoid -ahd_complete_scb (struct ahd_softc *ahd,struct scb *scb) -{ -uint32_t sgptr; - - sgptr =ahd_le32toh (scb->hscb->sgptr); -if ((sgptr &SG_STATUS_VALID ) !=0 ) -ahd_handle_scb_status (ahd, scb); -else -ahd_done (ahd, scb); -} - -/* -* Determine whether the sequencer reported a residual -* for this SCB/transaction. -*/ -static __inlinevoid -ahd_update_residual (struct ahd_softc *ahd,struct scb *scb) -{ -uint32_t sgptr; - - sgptr =ahd_le32toh (scb->hscb->sgptr); -if ((sgptr &SG_STATUS_VALID ) !=0 ) -ahd_calc_residual (ahd, scb); -} -/* * Return pointers to the transfer negotiation information * for the specified our_id/remote_id pair.
drivers/scsi/aic7xxx/aic79xx_osm_pci.c renamed from drivers/scsi/aic7xxx/aic79xx_osm_pci.c~drivers-scsi-aic7xxx-possible-cleanups +1 -1
@@ -82,7 +82,7 @@ static struct pci_device_id ahd_linux_pcMODULE_DEVICE_TABLE (pci, ahd_linux_pci_id_table); -struct pci_driver aic79xx_pci_driver = { +static struct pci_driver aic79xx_pci_driver = { .name ="aic79xx" , .probe = ahd_linux_pci_dev_probe, .remove = ahd_linux_pci_dev_remove,
drivers/scsi/aic7xxx/aic79xx_pci.c renamed from drivers/scsi/aic7xxx/aic79xx_pci.c~drivers-scsi-aic7xxx-possible-cleanups +4 -3
@@ -99,7 +99,7 @@ static ahd_device_setup_t ahd_aic7901A_sstatic ahd_device_setup_t ahd_aic7902_setup;static ahd_device_setup_t ahd_aic790X_setup; -struct ahd_pci_identity ahd_pci_ident_table [] = +static struct ahd_pci_identity ahd_pci_ident_table [] = {/* aic7901 based controllers */ {
@@ -196,7 +196,7 @@ struct ahd_pci_identity ahd_pci_ident_ta } }; -const u_int ahd_num_pci_devs =NUM_ELEMENTS (ahd_pci_ident_table); +static const u_int ahd_num_pci_devs =NUM_ELEMENTS (ahd_pci_ident_table);#define DEVCONFIG 0x40#define PCIXINITPAT 0x0000E000ul
@@ -240,6 +240,7 @@ static int ahd_check_extport(struct ahd_static void ahd_configure_termination (struct ahd_softc *ahd,u_int adapter_control);static void ahd_pci_split_intr (struct ahd_softc *ahd,u_int intstat); +static void ahd_pci_intr (struct ahd_softc *ahd);struct ahd_pci_identity *ahd_find_pci_device (ahd_dev_softc_t pci)
@@ -759,7 +760,7 @@ static const char *pci_status_strings[] "%s: Address or Write Phase Parity Error Detected in %s.\n" }; -void +static void ahd_pci_intr (struct ahd_softc *ahd) {uint8_t pci_status[8 ];
drivers/scsi/aic7xxx/aic7xxx.h renamed from drivers/scsi/aic7xxx/aic7xxx.h~drivers-scsi-aic7xxx-possible-cleanups +0 -2
@@ -1136,8 +1136,6 @@ struct ahc_pci_identity {char *name;ahc_device_setup_t *setup; }; -extern struct ahc_pci_identity ahc_pci_ident_table[]; -extern const u_int ahc_num_pci_devs;/***************************** VL/EISA Declarations ***************************/ struct aic7770_identity {
drivers/scsi/aic7xxx/aic7xxx_osm.c renamed from drivers/scsi/aic7xxx/aic7xxx_osm.c~drivers-scsi-aic7xxx-possible-cleanups +1 -1
@@ -334,7 +334,7 @@ static uint32_t aic7xxx_seltime;* force all outstanding transactions to be serviced prior to a new * transaction. */ -uint32_t aic7xxx_periodic_otag; +static uint32_t aic7xxx_periodic_otag;/* * Module information and settable options.
drivers/scsi/aic7xxx/aic7xxx_osm.h renamed from drivers/scsi/aic7xxx/aic7xxx_osm.h~drivers-scsi-aic7xxx-possible-cleanups +0 -2
@@ -565,8 +565,6 @@ ahc_unlock(struct ahc_softc *ahc, unsign#define PCIR_SUBVEND_0 0x2c#define PCIR_SUBDEV_0 0x2e -extern struct pci_driver aic7xxx_pci_driver; - typedef enum { AHC_POWER_STATE_D0,
drivers/scsi/aic7xxx/aic7xxx_osm_pci.c renamed from drivers/scsi/aic7xxx/aic7xxx_osm_pci.c~drivers-scsi-aic7xxx-possible-cleanups +1 -1
@@ -130,7 +130,7 @@ static struct pci_device_id ahc_linux_pcMODULE_DEVICE_TABLE (pci, ahc_linux_pci_id_table); -struct pci_driver aic7xxx_pci_driver = { +static struct pci_driver aic7xxx_pci_driver = { .name ="aic7xxx" , .probe = ahc_linux_pci_dev_probe, .remove = ahc_linux_pci_dev_remove,
drivers/scsi/aic7xxx/aic7xxx_pci.c renamed from drivers/scsi/aic7xxx/aic7xxx_pci.c~drivers-scsi-aic7xxx-possible-cleanups +2 -2
@@ -164,7 +164,7 @@ static ahc_device_setup_t ahc_aha394XX_sstatic ahc_device_setup_t ahc_aha494XX_setup;static ahc_device_setup_t ahc_aha398XX_setup; -struct ahc_pci_identity ahc_pci_ident_table [] = +static struct ahc_pci_identity ahc_pci_ident_table [] = {/* aic7850 based controllers */ {
@@ -549,7 +549,7 @@ struct ahc_pci_identity ahc_pci_ident_ta } }; -const u_int ahc_num_pci_devs =NUM_ELEMENTS (ahc_pci_ident_table); +static const u_int ahc_num_pci_devs =NUM_ELEMENTS (ahc_pci_ident_table);#define AHC_394X_SLOT_CHANNEL_A 4#define AHC_394X_SLOT_CHANNEL_B 5
_
drivers/scsi/aic7xxx/aic79xx_osm.h renamed from drivers/scsi/aic7xxx/aic79xx_osm.h.old +0 -6
@@ -526,9 +526,6 @@int pos; }; -void ahd_format_transinfo (struct info_str *info, -struct ahd_transinfo *tinfo); -/******************************** Locking *************************************/ static __inlinevoid ahd_lockinit (struct ahd_softc *ahd)
@@ -602,8 +599,6 @@#define PCIXM_STATUS_MAXCRDS 0x1C00/* Maximum Cumulative Read Size */ #define PCIXM_STATUS_RCVDSCEM 0x2000/* Received a Split Comp w/Error msg */ -extern struct pci_driver aic79xx_pci_driver; - typedef enum { AHD_POWER_STATE_D0,
@@ -884,7 +879,6 @@irqreturn_t ahd_linux_isr (int irq,void *dev_id,struct pt_regs * regs);void ahd_platform_flushwork (struct ahd_softc *ahd); -int ahd_softc_comp (struct ahd_softc *,struct ahd_softc *);void ahd_done (struct ahd_softc *,struct scb *);void ahd_send_async (struct ahd_softc *,char channel,u_int target,u_int lun,ac_code ,void *);
drivers/scsi/aic7xxx/aic79xx_osm.c renamed from drivers/scsi/aic7xxx/aic79xx_osm.c.old +3 -1
@@ -312,7 +312,7 @@* force all outstanding transactions to be serviced prior to a new * transaction. */ -uint32_t aic79xx_periodic_otag; +static uint32_t aic79xx_periodic_otag;/* * Module information and settable options.
@@ -794,6 +794,7 @@return (0 ); } +#if 0 /********************* Platform Dependent Functions ***************************/ /* * Compare "left hand" softc with "right hand" softc, returning:
@@ -847,6 +848,7 @@ value = rahd->channel - lahd->channel; return (value); } +#endif /* 0 */ static void ahd_linux_setup_iocell_info (u_long index,int instance,int targ,int32_t value)
drivers/scsi/aic7xxx/aic79xx_proc.c renamed from drivers/scsi/aic7xxx/aic79xx_proc.c.old +1 -1
@@ -138,7 +138,7 @@return (len); } -void +static void ahd_format_transinfo (struct info_str *info,struct ahd_transinfo *tinfo) {u_int speed;
drivers/scsi/aic7xxx/aic79xx.h renamed from drivers/scsi/aic7xxx/aic79xx.h.old +0 -53
130 lines not on this page · raw message
drivers/scsi/aic7xxx/aic79xx_core.c renamed from drivers/scsi/aic7xxx/aic79xx_core.c.old +165 -97
476 lines not on this page · raw message