Re: [PATCH 3/8] Add S3 state to DSDT. Handle resume event in the BIOS.

[Avi Kivity <avi@redhat.com>]

Gleb Natapov wrote:
> Signed-off-by: Gleb Natapov <gleb@redhat.com>
> ---
>
>  bios/acpi-dsdt.dsl |   33 ++++++++++++++++++-----
>  bios/rombios.c     |   35 +++++++++++++++++++++++++
>  bios/rombios32.c   |   74 ++++++++++++++++++++++++++++++++++++++++++++++++----
>  3 files changed, 130 insertions(+), 12 deletions(-)
>
> diff --git a/bios/acpi-dsdt.dsl b/bios/acpi-dsdt.dsl
> index 577b3fe..02f53a1 100755
> --- a/bios/acpi-dsdt.dsl
> +++ b/bios/acpi-dsdt.dsl
> @@ -65,6 +65,32 @@ DefinitionBlock (
>  	gen_processor(14, E)
>      }
>  
> +    /*
> +     * S3 (suspend-to-ram), S4 (suspend-to-disc) and S5 (power-off) type codes:
> +     * must match piix4 emulation.
> +     */
> +    Name (\_S3, Package (0x04)
> +    {
> +        0x01,  /* PM1a_CNT.SLP_TYP */
> +        0x01,  /* PM1b_CNT.SLP_TYP */
> +        Zero,  /* reserved */
> +        Zero   /* reserved */
> +    })
> +    Name (\_S4, Package (0x04)
> +    {
> +        Zero,  /* PM1a_CNT.SLP_TYP */
> +        Zero,  /* PM1b_CNT.SLP_TYP */
> +        Zero,  /* reserved */
> +        Zero   /* reserved */
> +    })
> +    Name (\_S5, Package (0x04)
> +    {
> +        Zero,  /* PM1a_CNT.SLP_TYP */
> +        Zero,  /* PM1b_CNT.SLP_TYP */
> +        Zero,  /* reserved */
> +        Zero   /* reserved */
> +    })
> +
>      Scope (\)
>      {
>          /* Debug Output */
> @@ -626,13 +652,6 @@ DefinitionBlock (
>          }
>      }
>  
> -    /* S5 = power off state */
> -    Name (_S5, Package (4) {
> -        0x00, // PM1a_CNT.SLP_TYP
> -        0x00, // PM2a_CNT.SLP_TYP
> -        0x00, // reserved
> -        0x00, // reserved
> -    })
>      Scope (\_GPE)
>      {
>   
Any reason for the code movement?


-- 
error compiling committee.c: too many arguments to function