misc drivers/block/cciss* cleanups (fwd)
PATCH 1/1 misc drivers/block/cciss* cleanups (fwd)
The patch forwarded below still applies and compiles against 2.6.10-mm2.
Please apply.
----- Forwarded message from Adrian Bunk <[email protected]> -----
Date: Mon, 29 Nov 2004 13:26:27 +0100
From: Adrian Bunk <[email protected]>
To: [email protected], [email protected]
Subject: [2.6 patch] misc drivers/block/cciss* cleanups
The patch below contains the following changes:
- make some needlessly global code static
- fold init_cciss_module into init_cciss (there's no need for two separate functions)
- remove the unused global function cciss_scsi_info
drivers/block/cciss.c | 18 +++++++----------- drivers/block/cciss_scsi.c | 35 ++++------------------------------- drivers/block/cciss_scsi.h | 1 - 3 files changed, 11 insertions(+), 43 deletions(-)
drivers/block/cciss.c renamed from drivers/block/cciss.c.old +7 -11
@@ -61,7 +61,7 @@#include <linux/cciss_ioctl.h> /* define the PCI info for the cards we can control */ -const struct pci_device_id cciss_pci_device_id[] = { +static const struct pci_device_id cciss_pci_device_id[] = { {PCI_VENDOR_ID_COMPAQ ,PCI_DEVICE_ID_COMPAQ_CISS ,0x0E11 ,0x4070 ,0 ,0 ,0 }, {PCI_VENDOR_ID_COMPAQ ,PCI_DEVICE_ID_COMPAQ_CISSB ,
@@ -2908,21 +2908,17 @@* This is it. Register the PCI driver information for the cards we control * the OS will call our registered routines when it finds one of our cards. */ -int __init cciss_init (void ) +static int __init init_cciss (void ) { +register_cciss_ioctl32 (); + printk(KERN_INFO DRIVER_NAME "\n"); /* Register for our PCI devices */ return pci_module_init(&cciss_pci_driver); } -static int __init init_cciss_module (void ) -{ -register_cciss_ioctl32 (); -return (cciss_init ()); -} - -static void __exit cleanup_cciss_module(void) +static void __exit cleanup_cciss(void) {int i;
@@ -2941,5 +2937,5 @@remove_proc_entry ("cciss" , proc_root_driver); } -module_init (init_cciss_module); -module_exit (cleanup_cciss_module); +module_init (init_cciss); +module_exit (cleanup_cciss);
drivers/block/cciss_scsi.c renamed from drivers/block/cciss_scsi.c.old +4 -31
@@ -53,9 +53,7 @@ int cmd_type); -const char *cciss_scsi_info (struct Scsi_Host *sa); - -int cciss_scsi_proc_info( +static int cciss_scsi_proc_info( struct Scsi_Host *sh, char *buffer, /* data buffer */ char **start, /* where data in buffer starts */
@@ -63,7 +61,7 @@int length,/* length of data in buffer */ int func); /* 0 == read, 1 == write */ -int cciss_scsi_queue_command (struct scsi_cmnd *cmd, +static int cciss_scsi_queue_command (struct scsi_cmnd *cmd,void (* done)(struct scsi_cmnd *));static struct cciss_scsi_hba_t ccissscsi[MAX_CTLR ] = {
@@ -712,8 +710,6 @@return 1 ; } -static void __exit cleanup_cciss_module(void); - static void cciss_unmap_one(struct pci_dev *pdev, CommandList_struct *cp,
@@ -1114,7 +1110,7 @@ } -int +static int cciss_scsi_proc_info(struct Scsi_Host *sh, char *buffer, /* data buffer */ char **start, /* where data in buffer starts */
@@ -1149,29 +1145,6 @@ buffer, length); } -/* this is via the generic proc support */ -const char * -cciss_scsi_info (struct Scsi_Host *sa) -{ -static char buf[300 ]; -ctlr_info_t *ci; - -/* probably need to work on putting a bit more info in here... */ -/* this is output via the /proc filesystem. */ - - ci = (ctlr_info_t *) sa->hostdata[0 ]; - -sprintf (buf,"%s %c%c%c%c\n" , - ci->product_name, - ci->firm_ver[0 ], - ci->firm_ver[1 ], - ci->firm_ver[2 ], - ci->firm_ver[3 ]); - -return buf; -} - -/* cciss_scatter_gather takes a struct scsi_cmnd, (cmd), and does the pci dma mapping and fills in the scatter gather entries of the cciss command, cp. */
@@ -1225,7 +1198,7 @@ } -int +static int cciss_scsi_queue_command (struct scsi_cmnd *cmd,void (* done)(struct scsi_cmnd *)) {ctlr_info_t **c;
drivers/block/cciss_scsi.h renamed from drivers/block/cciss_scsi.h.old +0 -1
@@ -39,7 +39,6 @@#define SCSI_CCISS_CAN_QUEUE 2/* -info: cciss_scsi_info, \ Note, cmd_per_lun could give us some trouble, so I'm setting it very low. Likewise, SCSI_CCISS_CAN_QUEUE is set very conservatively.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
----- End forwarded message -----