This patch contains the following possible cleanups: - make needlessly global code static - #if 0 the following unused global functions: - qla_sup.c: qla24xx_get_flash_manufacturer - qla_sup.c: qla24xx_write_flash_data Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx> --- This patch was already sent on: - 20 Nov 2005 drivers/scsi/qla2xxx/qla_os.c | 14 +++++++------- drivers/scsi/qla2xxx/qla_sup.c | 8 ++++++-- 2 files changed, 13 insertions(+), 9 deletions(-) --- linux-2.6.15-rc1-mm2-full/drivers/scsi/qla2xxx/qla_os.c.old 2005-11-20 19:56:43.000000000 +0100 +++ linux-2.6.15-rc1-mm2-full/drivers/scsi/qla2xxx/qla_os.c 2005-11-20 20:04:14.000000000 +0100 @@ -54,7 +54,7 @@ MODULE_PARM_DESC(ql2xloginretrycount, "Specify an alternate value for the NVRAM login retry count."); -int ql2xfwloadbin=1; +static int ql2xfwloadbin=1; module_param(ql2xfwloadbin, int, S_IRUGO|S_IRUSR); MODULE_PARM_DESC(ql2xfwloadbin, "Load ISP2xxx firmware image via hotplug."); @@ -266,7 +266,7 @@ return str; } -char * +static char * qla2x00_fw_version_str(struct scsi_qla_host *ha, char *str) { char un_str[10]; @@ -304,7 +304,7 @@ return (str); } -char * +static char * qla24xx_fw_version_str(struct scsi_qla_host *ha, char *str) { sprintf(str, "%d.%02d.%02d ", ha->fw_major_version, @@ -580,7 +580,7 @@ * * Note: **************************************************************************/ -int +static int qla2xxx_eh_abort(struct scsi_cmnd *cmd) { scsi_qla_host_t *ha = to_qla_host(cmd->device->host); @@ -714,7 +714,7 @@ * SUCCESS/FAILURE (defined as macro in scsi.h). * **************************************************************************/ -int +static int qla2xxx_eh_device_reset(struct scsi_cmnd *cmd) { scsi_qla_host_t *ha = to_qla_host(cmd->device->host); @@ -845,7 +845,7 @@ * SUCCESS/FAILURE (defined as macro in scsi.h). * **************************************************************************/ -int +static int qla2xxx_eh_bus_reset(struct scsi_cmnd *cmd) { scsi_qla_host_t *ha = to_qla_host(cmd->device->host); @@ -906,7 +906,7 @@ * * Note: **************************************************************************/ -int +static int qla2xxx_eh_host_reset(struct scsi_cmnd *cmd) { scsi_qla_host_t *ha = to_qla_host(cmd->device->host); --- linux-2.6.15-rc1-mm2-full/drivers/scsi/qla2xxx/qla_sup.c.old 2005-11-20 19:58:00.000000000 +0100 +++ linux-2.6.15-rc1-mm2-full/drivers/scsi/qla2xxx/qla_sup.c 2005-11-20 20:00:10.000000000 +0100 @@ -428,7 +428,7 @@ return FARX_ACCESS_NVRAM_DATA | naddr; } -uint32_t +static uint32_t qla24xx_read_flash_dword(scsi_qla_host_t *ha, uint32_t addr) { int rval; @@ -469,7 +469,7 @@ return dwptr; } -int +static int qla24xx_write_flash_dword(scsi_qla_host_t *ha, uint32_t addr, uint32_t data) { int rval; @@ -491,6 +491,8 @@ return rval; } +#if 0 + void qla24xx_get_flash_manufacturer(scsi_qla_host_t *ha, uint8_t *man_id, uint8_t *flash_id) @@ -581,6 +583,8 @@ return ret; } +#endif /* 0 */ + uint8_t * qla2x00_read_nvram_data(scsi_qla_host_t *ha, uint8_t *buf, uint32_t naddr, uint32_t bytes) - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html