]> git.karo-electronics.de Git - karo-tx-linux.git/commit
ide: relax DMA info validity checking
authorBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Mon, 22 Jun 2009 07:38:26 +0000 (07:38 +0000)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sun, 16 Aug 2009 21:18:48 +0000 (14:18 -0700)
commita27b9c59f2a252c4fd8723a929cc46067a4ad58e
treecf5f5c56c1d8019da4048d0cbf59ad070e0eee8b
parent957157bfe829d5900ae43b46674bae4d97207712
ide: relax DMA info validity checking

commit 346c17a6cf60375323adfaa4b8a9d841049f890e upstream.

There are some broken devices that report multiple DMA xfer modes
enabled at once (ATA spec doesn't allow it) but otherwise work fine
with DMA so just delete ide_id_dma_bug().

[ As discovered by detective work by Frans and Bart, due to how
  handling of the ID block was handled before commit c419993
  ("ide-iops: only clear DMA words on setting DMA mode") this
  check was always seeing zeros in the fields or other similar
  garbage.  Therefore this check wasn't actually checking anything.
  Now that the tests actually check the real bits, all we see are
  devices that trigger the check yet work perfectly fine, therefore
  killing this useless check is the best thing to do. -DaveM ]

Reported-by: Frans Pop <elendil@planet.nl>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/ide/ide-dma.c
drivers/ide/ide-iops.c
include/linux/ide.h