Hello, Jeff.
Jeff Garzik wrote:
Patchset looks OK to me, with a few notes:
1) might want to separate out this change and forward it to
#upstream-fixes:
- ahci_tf_read(ap, &tf);
- if (tf.command & (ATA_BUSY | ATA_DRQ)) {
+ if (ahci_check_status(ap) & (ATA_BUSY | ATA_DRQ)) {
Okay, will do that.
2) In general, it's a good idea to clear SError after twiddling with the
phy. Not just on vt8521.
It's done in ->post_reset() after a successful soft/hard reset, so the
issue occurs only when the controller needs follow-up SRST after
hardreset and the softreset doesn't work unless SError isn't cleared.
And, vt8251 is the only such controller at the moment (heh heh
surprise!), so I think we can leave it there for the time being.
3) Any positive test results yet?
Aalderd confirmed the change. He has reported a spurious IRQ problem
but it's emitted when loading vmware modules and doesn't seem to be
related to libata at all. My vt8251 also does hotplug happily with the
change, so we have two positive test results.
Thanks.
--
tejun
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
