[patch] Re: suspend2 checks

[David Zeuthen]

On Fri, 2005-05-13 at 14:39 +0100, Richard Hughes wrote:
> On Fri, 2005-05-13 at 14:03 +0100, Richard Hughes wrote:
> > On Fri, 2005-05-13 at 10:01 +0100, Hughes, Richard (UK Rochester) wrote:
> > > Is this something that belongs in HAL? I know that hal-spec.xml.in
> > > would need patching too, but included the patch as received for now.
> > 
> > After an email from Stefan Seyfried, (who reminded me that suspend2 was
> > not just acpi), I've changed the original patch to the one attached.
> > 
> > Comments most welcome.
> > 

Thanks for the patch. As discussed, how about a patch that sets 
 
 bool power_management.sleep.can_quick_sleep (in ACPI this is S1)
 bool power_management.sleep.can_deep_sleep  (in ACPI this is S3)
 bool power_management.sleep.can_hibernate   (in ACPI this is S4)

Here's how I would choose the values

 quick_sleep: true only if /sys/power/state contains 'standby'
 deep_sleep: true only if /sys/power/state contains 'mem'
 hibernate: use the algorithm in your patch (extra points for code to
            detect swsup also)

Thanks,
David


_______________________________________________
hal mailing list
hal at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/hal