diff options
author | Jeff Garzik <jgarzik@pobox.com> | 2005-10-29 13:58:21 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2005-10-29 13:58:21 -0400 |
commit | ac19bff25b6834d858274406a686f2227dd8489d (patch) | |
tree | d77670070abc662b6b14b5a9f9d6c08a56a476a9 /drivers/scsi/sata_qstor.c | |
parent | 9dfb7808fb05643b0d06df7411b94d9546696bf1 (diff) |
[libata] ensure ->tf_read() hook reads Status and Error registers
We want ->tf_read() to get a complete snapshot of all taskfile
registers, without requiring the callers to manually call
ata_chk_status() and ata_chk_err() themselves.
This also fixes a minor bug in sata_vsc where the lower bits of the
feature register were incorrectly placed in the HOB (high order bits)
portion of struct ata_taskfile.
Diffstat (limited to 'drivers/scsi/sata_qstor.c')
-rw-r--r-- | drivers/scsi/sata_qstor.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/sata_qstor.c b/drivers/scsi/sata_qstor.c index 250dafa6bc36..1aaf3304d397 100644 --- a/drivers/scsi/sata_qstor.c +++ b/drivers/scsi/sata_qstor.c | |||
@@ -433,7 +433,7 @@ static inline unsigned int qs_intr_mmio(struct ata_host_set *host_set) | |||
433 | if (qc && (!(qc->tf.ctl & ATA_NIEN))) { | 433 | if (qc && (!(qc->tf.ctl & ATA_NIEN))) { |
434 | 434 | ||
435 | /* check main status, clearing INTRQ */ | 435 | /* check main status, clearing INTRQ */ |
436 | u8 status = ata_chk_status(ap); | 436 | u8 status = ata_check_status(ap); |
437 | if ((status & ATA_BUSY)) | 437 | if ((status & ATA_BUSY)) |
438 | continue; | 438 | continue; |
439 | DPRINTK("ata%u: protocol %d (dev_stat 0x%X)\n", | 439 | DPRINTK("ata%u: protocol %d (dev_stat 0x%X)\n", |