diff options
author | Saurav Kashyap <saurav.kashyap@qlogic.com> | 2013-11-07 02:54:56 -0500 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2013-12-19 10:39:03 -0500 |
commit | c41afc9a754ee085f45a95d9dd58d90ab3bb25dd (patch) | |
tree | 7272e6f8dd88b9eff21daff0b9d6cad735229627 /drivers/scsi | |
parent | ef80d1e18b014af08741cf688e3fdda1fb71363f (diff) |
[SCSI] qla2xxx: Fix warning reported by smatch.
Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
Signed-off-by: Saurav Kashyap <saurav.kashyap@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/qla2xxx/qla_gbl.h | 2 | ||||
-rw-r--r-- | drivers/scsi/qla2xxx/qla_nx.c | 2 | ||||
-rw-r--r-- | drivers/scsi/qla2xxx/qla_nx2.c | 8 | ||||
-rw-r--r-- | drivers/scsi/qla2xxx/qla_os.c | 4 |
4 files changed, 8 insertions, 8 deletions
diff --git a/drivers/scsi/qla2xxx/qla_gbl.h b/drivers/scsi/qla2xxx/qla_gbl.h index 00427e0ff5b2..1f426628a0a5 100644 --- a/drivers/scsi/qla2xxx/qla_gbl.h +++ b/drivers/scsi/qla2xxx/qla_gbl.h | |||
@@ -732,7 +732,7 @@ extern inline void qla8044_set_qsnt_ready(struct scsi_qla_host *vha); | |||
732 | extern inline void qla8044_need_reset_handler(struct scsi_qla_host *vha); | 732 | extern inline void qla8044_need_reset_handler(struct scsi_qla_host *vha); |
733 | extern int qla8044_device_state_handler(struct scsi_qla_host *vha); | 733 | extern int qla8044_device_state_handler(struct scsi_qla_host *vha); |
734 | extern void qla8044_clear_qsnt_ready(struct scsi_qla_host *vha); | 734 | extern void qla8044_clear_qsnt_ready(struct scsi_qla_host *vha); |
735 | extern void qla8044_clear_drv_active(struct scsi_qla_host *vha); | 735 | extern void qla8044_clear_drv_active(struct qla_hw_data *); |
736 | void qla8044_get_minidump(struct scsi_qla_host *vha); | 736 | void qla8044_get_minidump(struct scsi_qla_host *vha); |
737 | int qla8044_collect_md_data(struct scsi_qla_host *vha); | 737 | int qla8044_collect_md_data(struct scsi_qla_host *vha); |
738 | extern int qla8044_md_get_template(scsi_qla_host_t *); | 738 | extern int qla8044_md_get_template(scsi_qla_host_t *); |
diff --git a/drivers/scsi/qla2xxx/qla_nx.c b/drivers/scsi/qla2xxx/qla_nx.c index 1468c59d8960..1e6ba4a369e2 100644 --- a/drivers/scsi/qla2xxx/qla_nx.c +++ b/drivers/scsi/qla2xxx/qla_nx.c | |||
@@ -3017,7 +3017,7 @@ qla8xxx_dev_failed_handler(scsi_qla_host_t *vha) | |||
3017 | qla82xx_clear_drv_active(ha); | 3017 | qla82xx_clear_drv_active(ha); |
3018 | qla82xx_idc_unlock(ha); | 3018 | qla82xx_idc_unlock(ha); |
3019 | } else if (IS_QLA8044(ha)) { | 3019 | } else if (IS_QLA8044(ha)) { |
3020 | qla8044_clear_drv_active(vha); | 3020 | qla8044_clear_drv_active(ha); |
3021 | qla8044_idc_unlock(ha); | 3021 | qla8044_idc_unlock(ha); |
3022 | } | 3022 | } |
3023 | 3023 | ||
diff --git a/drivers/scsi/qla2xxx/qla_nx2.c b/drivers/scsi/qla2xxx/qla_nx2.c index cc3522fbcaa2..b45f4d78ee36 100644 --- a/drivers/scsi/qla2xxx/qla_nx2.c +++ b/drivers/scsi/qla2xxx/qla_nx2.c | |||
@@ -1257,10 +1257,10 @@ exit_start_fw: | |||
1257 | } | 1257 | } |
1258 | 1258 | ||
1259 | void | 1259 | void |
1260 | qla8044_clear_drv_active(struct scsi_qla_host *vha) | 1260 | qla8044_clear_drv_active(struct qla_hw_data *ha) |
1261 | { | 1261 | { |
1262 | uint32_t drv_active; | 1262 | uint32_t drv_active; |
1263 | struct qla_hw_data *ha = vha->hw; | 1263 | struct scsi_qla_host *vha = pci_get_drvdata(ha->pdev); |
1264 | 1264 | ||
1265 | drv_active = qla8044_rd_direct(vha, QLA8044_CRB_DRV_ACTIVE_INDEX); | 1265 | drv_active = qla8044_rd_direct(vha, QLA8044_CRB_DRV_ACTIVE_INDEX); |
1266 | drv_active &= ~(1 << (ha->portnum)); | 1266 | drv_active &= ~(1 << (ha->portnum)); |
@@ -1324,7 +1324,7 @@ qla8044_device_bootstrap(struct scsi_qla_host *vha) | |||
1324 | if (rval != QLA_SUCCESS) { | 1324 | if (rval != QLA_SUCCESS) { |
1325 | ql_log(ql_log_info, vha, 0xb0b3, | 1325 | ql_log(ql_log_info, vha, 0xb0b3, |
1326 | "%s: HW State: FAILED\n", __func__); | 1326 | "%s: HW State: FAILED\n", __func__); |
1327 | qla8044_clear_drv_active(vha); | 1327 | qla8044_clear_drv_active(ha); |
1328 | qla8044_wr_direct(vha, QLA8044_CRB_DEV_STATE_INDEX, | 1328 | qla8044_wr_direct(vha, QLA8044_CRB_DEV_STATE_INDEX, |
1329 | QLA8XXX_DEV_FAILED); | 1329 | QLA8XXX_DEV_FAILED); |
1330 | return rval; | 1330 | return rval; |
@@ -1737,7 +1737,7 @@ qla8044_update_idc_reg(struct scsi_qla_host *vha) | |||
1737 | 1737 | ||
1738 | rval = qla8044_set_idc_ver(vha); | 1738 | rval = qla8044_set_idc_ver(vha); |
1739 | if (rval == QLA_FUNCTION_FAILED) | 1739 | if (rval == QLA_FUNCTION_FAILED) |
1740 | qla8044_clear_drv_active(vha); | 1740 | qla8044_clear_drv_active(ha); |
1741 | qla8044_idc_unlock(ha); | 1741 | qla8044_idc_unlock(ha); |
1742 | 1742 | ||
1743 | exit_update_idc_reg: | 1743 | exit_update_idc_reg: |
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c index 20d58be7b8d5..89a53002b585 100644 --- a/drivers/scsi/qla2xxx/qla_os.c +++ b/drivers/scsi/qla2xxx/qla_os.c | |||
@@ -2881,7 +2881,7 @@ probe_hw_failed: | |||
2881 | } | 2881 | } |
2882 | if (IS_QLA8044(ha)) { | 2882 | if (IS_QLA8044(ha)) { |
2883 | qla8044_idc_lock(ha); | 2883 | qla8044_idc_lock(ha); |
2884 | qla8044_clear_drv_active(base_vha); | 2884 | qla8044_clear_drv_active(ha); |
2885 | qla8044_idc_unlock(ha); | 2885 | qla8044_idc_unlock(ha); |
2886 | } | 2886 | } |
2887 | iospace_config_failed: | 2887 | iospace_config_failed: |
@@ -3045,7 +3045,7 @@ qla2x00_clear_drv_active(scsi_qla_host_t *vha) | |||
3045 | 3045 | ||
3046 | if (IS_QLA8044(ha)) { | 3046 | if (IS_QLA8044(ha)) { |
3047 | qla8044_idc_lock(ha); | 3047 | qla8044_idc_lock(ha); |
3048 | qla8044_clear_drv_active(vha); | 3048 | qla8044_clear_drv_active(ha); |
3049 | qla8044_idc_unlock(ha); | 3049 | qla8044_idc_unlock(ha); |
3050 | } else if (IS_QLA82XX(ha)) { | 3050 | } else if (IS_QLA82XX(ha)) { |
3051 | qla82xx_idc_lock(ha); | 3051 | qla82xx_idc_lock(ha); |