diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-02-21 02:29:40 -0500 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2012-02-29 17:47:13 -0500 |
commit | 8a6552299e2c3b01bb49ea52be2788f7c6bd8bf2 (patch) | |
tree | 2c736f4ae61380630c6bc8e3eee24d8e11763195 /drivers/scsi | |
parent | 13de2c541993394ec4db4949302312a0ce0af7b1 (diff) |
[SCSI] qla2xxx: handle default case in qla2x00_request_firmware()
This silences a static checker warning. Also we're always adding new
types of firmware, so it might fix a bug in real life some day.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/qla2xxx/qla_os.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c index 094af74989d6..a2f999273a5f 100644 --- a/drivers/scsi/qla2xxx/qla_os.c +++ b/drivers/scsi/qla2xxx/qla_os.c | |||
@@ -4058,7 +4058,6 @@ qla2x00_request_firmware(scsi_qla_host_t *vha) | |||
4058 | struct qla_hw_data *ha = vha->hw; | 4058 | struct qla_hw_data *ha = vha->hw; |
4059 | struct fw_blob *blob; | 4059 | struct fw_blob *blob; |
4060 | 4060 | ||
4061 | blob = NULL; | ||
4062 | if (IS_QLA2100(ha)) { | 4061 | if (IS_QLA2100(ha)) { |
4063 | blob = &qla_fw_blobs[FW_ISP21XX]; | 4062 | blob = &qla_fw_blobs[FW_ISP21XX]; |
4064 | } else if (IS_QLA2200(ha)) { | 4063 | } else if (IS_QLA2200(ha)) { |
@@ -4079,6 +4078,8 @@ qla2x00_request_firmware(scsi_qla_host_t *vha) | |||
4079 | blob = &qla_fw_blobs[FW_ISP2031]; | 4078 | blob = &qla_fw_blobs[FW_ISP2031]; |
4080 | } else if (IS_QLA8031(ha)) { | 4079 | } else if (IS_QLA8031(ha)) { |
4081 | blob = &qla_fw_blobs[FW_ISP8031]; | 4080 | blob = &qla_fw_blobs[FW_ISP8031]; |
4081 | } else { | ||
4082 | return NULL; | ||
4082 | } | 4083 | } |
4083 | 4084 | ||
4084 | mutex_lock(&qla_fw_lock); | 4085 | mutex_lock(&qla_fw_lock); |