On Mer, 2006-02-22 at 14:00 -0800, Randy Dunlap wrote: > + if (ap->legacy_mode) { > err = pata_get_dev_handle(dev, &dev_handle, &pcidevfn); > if (err < 0) { > if (ata_msg_probe(ap)) ap->legacy mode tells you if one of a subset of PCI controllers is in native or legacy compatibility mode. It tells you nothing about whether the controller is SATA or PATA. In fact it may even be both at once as there may be a SATA bridge on one channel in some configurations. The field is also meaningless for PCI controllers that are not class IDE. The cable type field will tell you in some situations if we are PATA or SATA but we actually don't always know at the moment. We probably have the info for 99.9% of cases if needed it just isnt ap->legacy_mode, its cable type plus the PATA/SATA bridge knobbling check. Alan - : send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html