diff options
author | Mikael Pettersson <mikpe@it.uu.se> | 2009-09-15 15:08:47 +0200 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2009-09-17 16:46:34 -0400 |
commit | ff7cddf59e3618d29e12b061651ab3f5a53f4a98 (patch) | |
tree | b45b879b6ca820b074ea69ee218f92247932ebd4 /COPYING | |
parent | 0ae6654da437db4ae6333d232e718b570c7a3eac (diff) |
sata_promise: update reset code
sata_promise's reset code has deviated quite a bit from
the Promise reference driver's, and it has been observed
to fail to recover from errors in some cases.
This patch thus updates the reset code to more closely
match the reference driver:
- soft reset (pdc_reset_port):
* wait for ATA engine to not be in packet command mode
(2nd gen only)
* write reset bit in PDC_CTLSTAT before the first read
in the loop
* for 2nd gen SATA follow up with FPDMA reset and clearing
error status registers
- hard reset (pdc_sata_hardreset):
* wait for ATA engine to not be in packet command mode
(2nd gen only)
* reset ATA engine via the PCI control register
* Tejun's change to use non-waiting hardreset + follow-up SRST
I'm not changing the hotplug mask bits since they are taken care
of by sata_promise's ->freeze() and ->thaw() operations. And I'm
not writing the PMP port # because that's always zero (for now).
Tested here on various controllers. In particular, one disk
which used to timeout and fail to recover from certain hdparm
and smartmonctl commands now works nicely.
Signed-off-by: Mikael Pettersson <mikpe@it.uu.se>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions