diff options
author | Andrew Vasquez <andrew.vasquez@qlogic.com> | 2007-11-12 13:30:57 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-01-11 19:22:45 -0500 |
commit | 06e23b7470ca7974b0ca8150c5415b55b5ea2a99 (patch) | |
tree | d5d9c06d09bf609312cb938ff5e4e9c787cc57bc /drivers/scsi/qla2xxx | |
parent | 860784c8a2b077157b6a51fb8749524d0363cc49 (diff) |
[SCSI] qla2xxx: Don't explicitly read mbx registers while processing a system-error.
Callers of qla2x00_async_event() already populate the mb[] array
upon invocation, doing so via the appropriate mailbox register
accessors. The stale codes removed are leftover-bits kept during
the FWI2 transition. Though relatively benign, the extra-reads
are not valid for FWI2 boards (ISP24xx and above) and peek into
the incorrect regions of registers.
Signed-off-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/scsi/qla2xxx')
-rw-r--r-- | drivers/scsi/qla2xxx/qla_isr.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c index 1104bd2eed40..343af399c95c 100644 --- a/drivers/scsi/qla2xxx/qla_isr.c +++ b/drivers/scsi/qla2xxx/qla_isr.c | |||
@@ -347,10 +347,6 @@ qla2x00_async_event(scsi_qla_host_t *ha, uint16_t *mb) | |||
347 | break; | 347 | break; |
348 | 348 | ||
349 | case MBA_SYSTEM_ERR: /* System Error */ | 349 | case MBA_SYSTEM_ERR: /* System Error */ |
350 | mb[1] = RD_MAILBOX_REG(ha, reg, 1); | ||
351 | mb[2] = RD_MAILBOX_REG(ha, reg, 2); | ||
352 | mb[3] = RD_MAILBOX_REG(ha, reg, 3); | ||
353 | |||
354 | qla_printk(KERN_INFO, ha, | 350 | qla_printk(KERN_INFO, ha, |
355 | "ISP System Error - mbx1=%xh mbx2=%xh mbx3=%xh.\n", | 351 | "ISP System Error - mbx1=%xh mbx2=%xh mbx3=%xh.\n", |
356 | mb[1], mb[2], mb[3]); | 352 | mb[1], mb[2], mb[3]); |