diff options
author | Tejun Heo <htejun@gmail.com> | 2006-12-03 21:34:13 +0900 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2006-12-03 07:58:10 -0500 |
commit | 800b399669ad495ad4361d134df87401ae36f44f (patch) | |
tree | 19e29f655836cd4808c668d736a83df552b7d8dd /drivers/ata/libata-eh.c | |
parent | 3ac551a6a63dcbc707348772a27bd7090b081524 (diff) | |
download | op-kernel-dev-800b399669ad495ad4361d134df87401ae36f44f.zip op-kernel-dev-800b399669ad495ad4361d134df87401ae36f44f.tar.gz |
[PATCH] libata: always use polling IDENTIFY
libata switched to IRQ-driven IDENTIFY when IRQ-driven PIO was
introduced. This has caused a lot of problems including device
misdetection and phantom device.
ATA_FLAG_DETECT_POLLING was added recently to selectively use polling
IDENTIFY on problemetic drivers but many controllers and devices are
affected by this problem and trying to adding ATA_FLAG_DETECT_POLLING
for each such case is diffcult and not very rewarding.
This patch makes libata always use polling IDENTIFY. This is
consistent with libata's original behavior and drivers/ide's behavior.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/ata/libata-eh.c')
-rw-r--r-- | drivers/ata/libata-eh.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/ata/libata-eh.c b/drivers/ata/libata-eh.c index 2aad7b7..76a85df 100644 --- a/drivers/ata/libata-eh.c +++ b/drivers/ata/libata-eh.c @@ -1692,9 +1692,6 @@ static int ata_eh_revalidate_and_attach(struct ata_port *ap, ata_class_enabled(ehc->classes[dev->devno])) { dev->class = ehc->classes[dev->devno]; - if (ap->flags & ATA_FLAG_DETECT_POLLING) - readid_flags |= ATA_READID_DETECT; - rc = ata_dev_read_id(dev, &dev->class, readid_flags, dev->id); if (rc == 0) { |