Hello, On Tue, Jan 14, 2014 at 04:31:40PM -0800, Todd E Brandt wrote: .... > Signed-off-by: Todd Brandt <todd.e.brandt@xxxxxxxxx> > Signed-off-by: Arjan van de Ven <arjan@xxxxxxxxxxxxxxx> > > drivers/ata/libata-core.c | 32 +++++++++++++++++--------------- > 1 file changed, 17 insertions(+), 15 deletions(-) > > Signed-off-by: Todd Brandt <todd.e.brandt@xxxxxxxxx> > Signed-off-by: Arjan van de Ven <arjan@xxxxxxxxxxxxxxx> Hmmm? Duplicate SOBs and there's supposed to be "---" separator between patch description and the rest. > @@ -5326,7 +5326,7 @@ bool ata_link_offline(struct ata_link *link) > #ifdef CONFIG_PM > static int ata_port_request_pm(struct ata_port *ap, pm_message_t mesg, > unsigned int action, unsigned int ehi_flags, > - int *async) > + bool async, int *async_result) Do we still need @async_result? Only resume is async and resume doesn't care about the result. Suspend is always synchronous and thus can just return the result, no? Thanks. -- tejun -- 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