From: Quinn Tran <quinn.t...@cavium.com>

The qla2x00_reset_active only cover the window of
turning the chip off, add check to cover Chip on.

Signed-off-by: Quinn Tran <quinn.t...@cavium.com>
Signed-off-by: Himanshu Madhani <himanshu.madh...@cavium.com>
---
 drivers/scsi/qla2xxx/qla_dbg.c  | 3 +++
 drivers/scsi/qla2xxx/qla_tmpl.c | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/scsi/qla2xxx/qla_dbg.c b/drivers/scsi/qla2xxx/qla_dbg.c
index 5fd44c50bbac..c7533fa7f46e 100644
--- a/drivers/scsi/qla2xxx/qla_dbg.c
+++ b/drivers/scsi/qla2xxx/qla_dbg.c
@@ -1130,6 +1130,7 @@ qla24xx_fw_dump(scsi_qla_host_t *vha, int hardware_locked)
                    ha->fw_dump);
                goto qla24xx_fw_dump_failed;
        }
+       QLA_FW_STOPPED(ha);
        fw = &ha->fw_dump->isp.isp24;
        qla2xxx_prep_dump(ha, ha->fw_dump);
 
@@ -1384,6 +1385,7 @@ qla25xx_fw_dump(scsi_qla_host_t *vha, int hardware_locked)
                    ha->fw_dump);
                goto qla25xx_fw_dump_failed;
        }
+       QLA_FW_STOPPED(ha);
        fw = &ha->fw_dump->isp.isp25;
        qla2xxx_prep_dump(ha, ha->fw_dump);
        ha->fw_dump->version = htonl(2);
@@ -2036,6 +2038,7 @@ qla83xx_fw_dump(scsi_qla_host_t *vha, int hardware_locked)
                    "request...\n", ha->fw_dump);
                goto qla83xx_fw_dump_failed;
        }
+       QLA_FW_STOPPED(ha);
        fw = &ha->fw_dump->isp.isp83;
        qla2xxx_prep_dump(ha, ha->fw_dump);
 
diff --git a/drivers/scsi/qla2xxx/qla_tmpl.c b/drivers/scsi/qla2xxx/qla_tmpl.c
index 731ca0d8520a..b170eb54aab5 100644
--- a/drivers/scsi/qla2xxx/qla_tmpl.c
+++ b/drivers/scsi/qla2xxx/qla_tmpl.c
@@ -1028,8 +1028,10 @@ qla27xx_fwdump(scsi_qla_host_t *vha, int hardware_locked)
                ql_log(ql_log_warn, vha, 0xd300,
                    "Firmware has been previously dumped (%p),"
                    " -- ignoring request\n", vha->hw->fw_dump);
-       else
+       else {
+               QLA_FW_STOPPED(vha->hw);
                qla27xx_execute_fwdt_template(vha);
+       }
 
 #ifndef __CHECKER__
        if (!hardware_locked)
-- 
2.12.0

Reply via email to