Re: [PATCH 00/86] PATA fixes

Jeff Garzik
2009-12-03T21:16:18Z

Alan,

Can you sanity check me on these?

> Bartlomiej Zolnierkiewicz (28):
> ide-pci-generic: build fix

ACK but skipped -- already applied myself

> ata_piix: fix MWDMA handling on PIIX3

applied

> pata_efar: fix wrong PIO timings being programmed

applied

> pata_efar: fix wrong MWDMA timings being programmed

applied

> pata_efar: MWDMA0 is unsupported

skipped, pending discussion (just sent email)

> pata_cmd640: document known issues

rejected, for reasons already explained

> pata_cs5520: remove dead VDMA support

applied

> pata_cypress: document known issues

rejected, for reasons already explained

> pata_hpt3x2n: fix overclocked MWDMA0 timing

skipped, pending discussion (just sent email)

> pata_hpt3x3: Power Management fix

applied, on a hope and a prayer (did not see this posted to mailing
list?). It looks correct to me.

> pata_it8213: fix UDMA handling

applied

> pata_it8213: fix wrong PIO timings being programmed

applied

> pata_it8213: fix PIO2 underclocking

applied

> pata_it8213: fix wrong MWDMA timings being programmed

applied

> pata_it8213: fix it8213_pre_reset() documentation

applied

> pata_it8213: MWDMA0 is unsupported

skipped, pending discussion (just sent email)

> pata_legacy: fix QDI6580DP support

applied

> pata_legacy: fix access to control register for QDI6580

applied

> pata_legacy: add pointers to QDI65x0 documentation

applied. I did not think the file path issue raised was important
enough to warrant patch rejection.

> pata_marvell: fix marvell_pre_reset() documentation

applied

> pata_ns87415: Power Management fix

applied

> pata_oldpiix: MWDMA0 is unsupported

skipped -- Alan, Sergey, comments?

> pata_pdc202xx_old: document known issues

rejected, for reasons already explained

> pata_radisys: fix UDMA handling

applied

> pata_rdc: MWDMA0 is unsupported

skipped -- Alan, Sergey, comments?

> pata_rz1000: Power Management fix

applied

> pata_sis: Power Management fix

applied

> pata_via: clear UDMA transfer mode bit for PIO and MWDMA

applied -- even though Alan's comment was correct. It is standard
kernel practice to place cosmetic changes into their own patches,
because it is standard kernel practice to break up logically distinct
changes.