On Fri, 08 Feb 2008 10:45:21 -0500 Chuck Ebbert <cebbert@xxxxxxxxxx> wrote: > On 02/08/2008 10:21 AM, Alan Cox wrote: > > + ata_link_for_each_dev(dev, link) { > > + if (!ata_is_40wire(dev)) > > + return 1; > > + } > > return 0; ??? Err yes. That'll teach me to clean up the patch and not re-test all the cases. I'll post a replacement diff - To unsubscribe from this list: 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