On Tue, May 11, 2010 at 02:18:01PM -0400, Phillip Susi wrote: > On 5/11/2010 1:19 PM, Matthew Garrett wrote: > > Just have suspend_devices_and_enter conditionalise the device suspend on > > whether or not it's PM_SUSPEND_STANDBY. > > Ahh, that looks like the place. Is there a reason why it does not > already do that? The exact meaning of PM_STANDBY_SUSPEND isn't defined. The sanest approach might be to add a callback in the platform_suspend_ops structure, pass the sleep state to that and then use that to decide whether or not devices need to sleep. > Also Documentation/power/states.txt says that S1 puts devices that > support it into D1. I would think the code to do that would be here in > suspend_devices_and_enter() but I don't see it. Do you know where that > is? I checked and my ATI Radeon video card supports D1 and D2 so could > potentially save a good bit of power by using these states. Putting a video card into D1 is either going to turn off the output or do nothing, so I don't think that's terribly compatible with what you want here. But to answer your question, the PCI core will decide what state to put devices in based on the sleep type provided. -- Matthew Garrett | mjg59@xxxxxxxxxxxxx -- 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
