On Thursday, May 15, 2014 01:13:02 PM Rafael J. Wysocki wrote: > On Wednesday, May 14, 2014 10:53:16 AM Alan Stern wrote: > > On Tue, 13 May 2014, Rafael J. Wysocki wrote: [cut] > > > > if (dev->power.direct_complete) { > > if (pm_runtime_status_suspended(dev)) { > > pm_runtime_disable(dev); > > if (dev->power.disable_depth == 1 > > && pm_runtime_status_suspended(dev)) > > goto Complete; > > pm_runtime_enable(dev); > > } > > dev->power.direct_complete = false; > > } > > That is a good idea, thanks! New patches follow. [1/3] is the core change with the above added. > > Also, now that we have finally settled on the appropriate API, there > > needs to ba a patch updating the PM documentation. > > Absolutely. I thought about updating the documentation in the same patch > (at least the comments in pm.h), but I guess a separate patch for files > under Documentation/ may be better. [2/3] is the corresponding documentation update (I hope I haven't overlooked anything important). [3/3] is a resend of the ACPI PM patch on top of the core change. Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html