]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "pata_sis: Implement MWDMA for the UDMA 133 capable chips"
authorJeff Garzik <jeff@garzik.org>
Mon, 7 Dec 2009 16:41:25 +0000 (11:41 -0500)
committerJeff Garzik <jgarzik@redhat.com>
Mon, 7 Dec 2009 16:41:25 +0000 (11:41 -0500)
commit7d6ae4028bc6c106f2c03d3d389f9a539f8cdf74
treef71bd703c40d3b5dec2e9db074e2ea43c0d8f4a5
parent71f85cc8ad9b8a264e64ead7eee218e90116bf82
Revert "pata_sis: Implement MWDMA for the UDMA 133 capable chips"

This reverts commit acb8e25e38869d620482548a45a38bacc718a065.

Sergei Shtylyov notes "You call min() on uncomparables [in
mwdma_clip_to_pio()], i.e. mwdma_to_pio[] contains XFER_PIO_* and
adev->pio_mode - XFER_PIO_0 yields you a mode number.  Thus the second
argument will always "win" as a minimal one"

Bartlomiej Zolnierkiewicz adds "There are more issues with the patch related
to mwdma_clip_to_pio().  The function can return values between 0 and
4 which obviously won't work well for the new code below for values
>2 (i.e. resulting in out-of-bounds array access for the common-case
of dev->pio_mode == XFER_PIO_4)."

Bartlomiej Zolnierkiewicz also notes the patch is incomplete, failing to
update MWDMA mode masks.

Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
drivers/ata/pata_sis.c