[coreboot] [PATCH] ASUS P2B ACPI sleep preparations

Tobias Diedrich ranma+coreboot at tdiedrich.de
Mon Nov 29 21:47:02 CET 2010


Uwe Hermann wrote:
> > Index: coreboot-svn-p2b/src/arch/i386/boot/acpi.c
> > ===================================================================
> > --- coreboot-svn-p2b.orig/src/arch/i386/boot/acpi.c	2010-11-27 11:48:28.000000000 +0100
> > +++ coreboot-svn-p2b/src/arch/i386/boot/acpi.c	2010-11-27 11:48:41.000000000 +0100
> > @@ -481,7 +481,7 @@
> >  
> >  static int acpi_is_wakeup(void)
> >  {
> > -	return (acpi_slp_type == 3);
> > +	return (acpi_slp_type == 3 || acpi_slp_type == 2);
> 
> Can this have negative effects for other ACPI-enabled chipsets/boards?

No, I grepped the tree and the other chipsets supporting this only
ever use 3 (S3).  But since S2 is virtually identically to S3 from a
wakeup perspective I thought adding the acpi_slp_type == 2 check
would be the right thing to do.
The other option would be to set acpi_slp_type to 3 even for S2
wakeup, but then there is not much sense in making it '3' either and
it could be changed to be a 'is_wakeup_from_sleep' boolean type.

-- 
Tobias						PGP: http://8ef7ddba.uguu.de




More information about the coreboot mailing list