diff options
author | Tejun Heo <htejun@gmail.com> | 2008-04-07 09:47:20 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-04-17 15:44:23 -0400 |
commit | 22183bf569c8600ff414ac25f23134044e0ef453 (patch) | |
tree | 1dea79cdbe439637477e1c4214b600b6823cb866 /drivers/scsi/ipr.c | |
parent | 305d2a1ab137d11d573319c315748a87060fe82d (diff) |
libata: add qc_fill_rtf port operation
On command completion, ata_qc_complete() directly called ops->tf_read
to fill qc->result_tf. This patch adds ops->qc_fill_rtf to replace
hardcoded ops->tf_read usage.
ata_sff_qc_fill_rtf() which uses ops->tf_read to fill result_tf is
implemented and set in ata_base_port_ops and other ops tables which
don't inherit from ata_base_port_ops, so this patch doesn't introduce
any behavior change.
ops->qc_fill_rtf() is similar to ops->sff_tf_read() but can only be
called when a command finishes. As some non-SFF controllers don't
have TF registers defined unless they're associated with in-flight
commands, this limited operation makes life easier for those drivers
and help lifting SFF assumptions from libata core layer.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Diffstat (limited to 'drivers/scsi/ipr.c')
-rw-r--r-- | drivers/scsi/ipr.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c index 5bddae19d1fb..2ecd32991522 100644 --- a/drivers/scsi/ipr.c +++ b/drivers/scsi/ipr.c | |||
@@ -5280,6 +5280,7 @@ static struct ata_port_operations ipr_sata_ops = { | |||
5280 | .sff_tf_read = ipr_tf_read, | 5280 | .sff_tf_read = ipr_tf_read, |
5281 | .qc_prep = ata_noop_qc_prep, | 5281 | .qc_prep = ata_noop_qc_prep, |
5282 | .qc_issue = ipr_qc_issue, | 5282 | .qc_issue = ipr_qc_issue, |
5283 | .qc_fill_rtf = ata_sff_qc_fill_rtf, | ||
5283 | .port_start = ata_sas_port_start, | 5284 | .port_start = ata_sas_port_start, |
5284 | .port_stop = ata_sas_port_stop | 5285 | .port_stop = ata_sas_port_stop |
5285 | }; | 5286 | }; |