On Mon, 16 Nov 2009 16:21:26 -0500 Jeff Garzik <jgarzik@xxxxxxxxx> wrote: > On 11/16/2009 09:43 AM, Arjan van de Ven wrote: > > is there a hardware way to ask for the link status via a link level > > thing? I thought the sata_scr_read() was by definition a host thing > > It is by definition a link-level thing, which is why sata_scr_read() > takes struct ata_link * as its first argument. > > Think about it... a port multiplier has downstream links, each with > their own link status, and link-related statistics. but also think about it... we can neither call this function with one of those as argument (since the function is called from irq context, and not the EH context) nor represent the result. The former would make it an unsafe API, while right now it's a safe API. -- Arjan van de Ven Intel Open Source Technology Centre For development, discussion and tips for power savings, visit http://www.lesswatts.org -- 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