diff options
author | Alan <alan@lxorguk.ukuu.org.uk> | 2007-01-25 15:09:05 +0000 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-01-25 18:27:16 -0500 |
commit | 61dd08c6c8d2b4ede530e43c01fa72f789ef65b1 (patch) | |
tree | ed50d46e0b8f089cf97aba54aa3e38ed3c1af57b | |
parent | b2a8bbe67d73631c71492fd60b757fc50a87f182 (diff) | |
download | op-kernel-dev-61dd08c6c8d2b4ede530e43c01fa72f789ef65b1.zip op-kernel-dev-61dd08c6c8d2b4ede530e43c01fa72f789ef65b1.tar.gz |
libata-sff: Don't call bmdma_stop on non DMA capable controllers
Fixes bogus accesses to ports 0-15 with a non DMA capable controller.
This I think should go in for 2.6.20
Signed-off-by: Alan Cox <alan@redhat.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
-rw-r--r-- | drivers/ata/libata-sff.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/ata/libata-sff.c b/drivers/ata/libata-sff.c index 114fa81d..942aeba 100644 --- a/drivers/ata/libata-sff.c +++ b/drivers/ata/libata-sff.c @@ -827,7 +827,8 @@ void ata_bmdma_error_handler(struct ata_port *ap) */ void ata_bmdma_post_internal_cmd(struct ata_queued_cmd *qc) { - ata_bmdma_stop(qc); + if (qc->ap->ioaddr.bmdma_addr) + ata_bmdma_stop(qc); } #ifdef CONFIG_PCI |