Re: [PATCH] pata_sis: fix MWDMA for <= UDMA66 chipsets and UDMA forUDMA33 chipsets

From: Jeff Garzik
Date: Wed Aug 01 2007 - 10:02:56 EST


Bartlomiej Zolnierkiewicz wrote:
* Fix MWDMA timings setup in sis_old_set_dmamode() and sis_66_set_dmamode().

The old timings were overclocked (even worse behavior than sis5513 IDE driver
which depends on BIOS to program correct timings), the new timings are taken
from the datasheet (they match timings from ATA spec).

* Fix UDMA timings setup in sis_old_set_dmamode().

Misplaced pci_write_config_word() call resulted in UDMA timings never
being set.

* Fix comments for sis_133_early_set_dmamode() and sis_133_set_dmamode():
- only the former function handles early SiS 961 bridges
- both functions lack MWDMA timings setup

* Fix typos in sis_100_set_piomode() and sis_133_set_piomode() comments.

* Bump driver version.

Cc: Alan Cox <alan@xxxxxxxxxxxxxxxxxxx>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx>
---
Sergei: Yes, I'm working on sis5513 IDE driver. :)

drivers/ata/pata_sis.c | 20 +++++++++-----------
1 file changed, 9 insertions(+), 11 deletions(-)

applied to #upstream-fixes


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/