On Iau, 2005-12-15 at 17:06 -0500, Jeff Garzik wrote:

quoted

	struct pci_dev *pdev = to_pci_dev(ap->host_set->dev);
	static struct pci_bits efar_enable_bits[] = {

const

Fixed

quoted

	/* FIXME: Check for PIO2 if the required timing needs IORDY
	   We should push this up into libata and have a requires_iordy() */

Yes, you should :)

The old IDE driver doesn't (but then it gets all this wrong). I've added
a function to look at old EIDE drive bits and see if PIO2 is iordy or
not and used it in the various intel drivers and this now.

There are some corner cases left (Are there drives that don't support
iordy at all but can do > PIO0 cycle time because some controllers
don't)

quoted

 *	efar_init_one - Register PIIX ATA PCI device with kernel services

s/PIIX/EFAR/ ?

Yep

quoted

 *	@pdev: PCI device to register
 *	@ent: Entry in efar_pci_tbl matching with @pdev
 *
 *	Called from kernel PCI layer.  We probe for combined mode (sigh),

You probe for combined mode?

Umm no, the joys of cut and waste.