diff options
author | Andrew Vasquez <andrew.vasquez@qlogic.com> | 2008-04-24 18:21:24 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-04-27 13:19:58 -0400 |
commit | c1ec1f1bf9cb1ba80e79a74d48bcfb5da246d6f6 (patch) | |
tree | 5e3087cbef3fb4cb1cfa8dbfe6a47f178315685b /drivers | |
parent | 550bf57dfb2200721baa43cfd9a8c75c2c166870 (diff) |
[SCSI] qla2xxx: Correct ISP84XX verify-chip response handling.
Earlier code could trigger an infinite-retry if 1st invocation
returned a non-CS_COMPLETE status.
Signed-off-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/scsi/qla2xxx/qla_mbx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/qla2xxx/qla_mbx.c b/drivers/scsi/qla2xxx/qla_mbx.c index 7d0a8a4c7719..a9cb8291f58e 100644 --- a/drivers/scsi/qla2xxx/qla_mbx.c +++ b/drivers/scsi/qla2xxx/qla_mbx.c | |||
@@ -2982,8 +2982,8 @@ qla84xx_verify_chip(struct scsi_qla_host *ha, uint16_t *status) | |||
2982 | /* We update the firmware with only one data sequence. */ | 2982 | /* We update the firmware with only one data sequence. */ |
2983 | options |= VCO_END_OF_DATA; | 2983 | options |= VCO_END_OF_DATA; |
2984 | 2984 | ||
2985 | retry = 0; | ||
2986 | do { | 2985 | do { |
2986 | retry = 0; | ||
2987 | memset(mn, 0, sizeof(*mn)); | 2987 | memset(mn, 0, sizeof(*mn)); |
2988 | mn->p.req.entry_type = VERIFY_CHIP_IOCB_TYPE; | 2988 | mn->p.req.entry_type = VERIFY_CHIP_IOCB_TYPE; |
2989 | mn->p.req.entry_count = 1; | 2989 | mn->p.req.entry_count = 1; |