On Tue, 15 May 2007 16:14:57 +1000 Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> wrote: > That one is the right one. I think the only diffs vs. the previous > one that was missing a quilt ref are I wasn't initializing altstatus > address and I turned some printk's into dev_dbg() as that's really > what they are. I'm going to have to NAK this on further review as it will break SRST handling in some cases. Until ata_bus_softreset has been fixed (see FIXME: entries) we shouldn't push this patch mainstream, or if we have then someone should fix the software reset posting as SRST only has an effect if the timing is honoured. Alan - 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