Skip to contentSkip to conversation

SCSI sym53c8xx_2: make some code static

PATCH In review Adrian Bunk 2 participants 2 messages Nov 15, 2004 – Nov 16, 2004

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 SCSI sym53c8xx_2: make some code static

The patch below makes some needlessly global code static.

diffstat output:

 drivers/scsi/sym53c8xx_2/sym_glue.c |    6 +++---
 drivers/scsi/sym53c8xx_2/sym_glue.h |    1 -
 drivers/scsi/sym53c8xx_2/sym_hipd.c |    3 ++-
 drivers/scsi/sym53c8xx_2/sym_hipd.h |    1 -
 4 files changed, 5 insertions(+), 6 deletions(-)
drivers/scsi/sym53c8xx_2/sym_glue.h renamed from drivers/scsi/sym53c8xx_2/sym_glue.h.old +0 -1
@@ -550,7 +550,6 @@
 #define sym_cam_target_lun(ccb)	(ccb)->lun
 #define	sym_freeze_cam_ccb(ccb)	do { ; } while (0)
 void sym_xpt_done(hcb_p np, cam_ccb_p ccb);
-void sym_xpt_done2(hcb_p np, cam_ccb_p ccb, int cam_status);
 void sym_print_addr (ccb_p cp);
 void sym_xpt_async_bus_reset(hcb_p np);
 void sym_xpt_async_sent_bdr(hcb_p np, int target);
drivers/scsi/sym53c8xx_2/sym_glue.c renamed from drivers/scsi/sym53c8xx_2/sym_glue.c.old +3 -3
@@ -147,7 +147,7 @@
 }
 
 /* This lock protects only the memory allocation/free.  */
-spinlock_t sym53c8xx_lock = SPIN_LOCK_UNLOCKED;
+static spinlock_t sym53c8xx_lock = SPIN_LOCK_UNLOCKED;
 
 static struct scsi_transport_template *sym2_transport_template = NULL;
 
@@ -285,7 +285,7 @@
 	ccb->scsi_done(ccb);
 }
 
-void sym_xpt_done2(struct sym_hcb *np, struct scsi_cmnd *ccb, int cam_status)
+static void sym_xpt_done2(struct sym_hcb *np, struct scsi_cmnd *ccb, int cam_status)
 {
 	sym_set_cam_status(ccb, cam_status);
 	sym_xpt_done(np, ccb);
@@ -2010,7 +2010,7 @@
  * the preset SCSI ID (which may be zero) must be read in from
  * a special configuration space register of the 875.
  */
-void sym_config_pqs(struct pci_dev *pdev, struct sym_device *sym_dev)
+static void sym_config_pqs(struct pci_dev *pdev, struct sym_device *sym_dev)
 {
 	int slot;
 	u8 tmp;
drivers/scsi/sym53c8xx_2/sym_hipd.h renamed from drivers/scsi/sym53c8xx_2/sym_hipd.h.old +0 -1
@@ -1098,7 +1098,6 @@
 #endif
 void sym_start_up (hcb_p np, int reason);
 void sym_interrupt (hcb_p np);
-void sym_flush_comp_queue(hcb_p np, int cam_status);
 int sym_clear_tasks(hcb_p np, int cam_status, int target, int lun, int task);
 ccb_p sym_get_ccb (hcb_p np, u_char tn, u_char ln, u_char tag_order);
 void sym_free_ccb (hcb_p np, ccb_p cp);
drivers/scsi/sym53c8xx_2/sym_hipd.c renamed from drivers/scsi/sym53c8xx_2/sym_hipd.c.old +2 -1
@@ -55,6 +55,7 @@
 static void sym_complete_error (hcb_p np, ccb_p cp);
 static void sym_complete_ok (hcb_p np, ccb_p cp);
 static int sym_compute_residual(hcb_p np, ccb_p cp);
+static void sym_flush_comp_queue(hcb_p np, int cam_status);
 
 /*
  *  Returns the name of this driver.
@@ -3038,7 +3039,7 @@
  *  This function is called when all CCBs involved 
  *  in error handling/recovery have been reaped.
  */
-void sym_flush_comp_queue(hcb_p np, int cam_status)
+static void sym_flush_comp_queue(hcb_p np, int cam_status)
 {
 	SYM_QUEHEAD *qp;
 	ccb_p cp;
#1 lore raw
Quoted 1 line · Adrian Bunk

The patch below makes some needlessly global code static.

Thanks, I'll apply this.

Signature · 6 lines
"Next the statesmen will invent cheap lies, putting the blame upon 
the nation that is attacked, and every man will be glad of those
conscience-soothing falsities, and will diligently study them, and refuse
to examine any refutations of them; and thus he will by and by convince 
himself that the war is just, and will thank God for the better sleep 
he enjoys after this process of grotesque self-deception." -- Mark Twain