Properly identify ACPI C3 states in _CST table.

Dump and disassemble ACPI tables and look in _CST.

In the last entry the state was getting set to 0:

Package (0x04)
{
  ResourceTemplate ()
  {
    Register (FFixedHW,
              0x01,               // Bit Width
              0x02,               // Bit Offset
              0x0000000000000030, // Address
              0x01,               // Access Size
              )
  },
  0x00000000,                     // State
  0x0000005A,                     // Latency
  0x000000C8                      // Power
}

Now it is properly identifed as state 3:

Package (0x04)
{
  ResourceTemplate ()
  {
    Register (FFixedHW,
              0x01,               // Bit Width
              0x02,               // Bit Offset
              0x0000000000000030, // Address
              0x01,               // Access Size
              )
  },
  0x00000003,                     // State
  0x0000005A,                     // Latency
  0x000000C8                      // Power
}

Change-Id: Ie0a68606c5a43ac5fb5ba7bb9a3fef933ad67b64
Signed-off-by: Duncan Laurie <dlaurie@google.com>
Reviewed-on: http://review.coreboot.org/1297
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
Tested-by: build bot (Jenkins)
This commit is contained in:
Duncan Laurie 2012-06-20 14:38:53 -07:00 committed by Patrick Georgi
parent 52e61183cc
commit b38e0c3509
1 changed files with 1 additions and 1 deletions

View File

@ -79,7 +79,7 @@ static int generate_cstate_entries(acpi_cstate_t *cstates,
length += acpigen_write_CST_package_entry(&cstates[c2]); length += acpigen_write_CST_package_entry(&cstates[c2]);
} }
if (c3 > 0) { if (c3 > 0) {
cstates[c2].ctype = 2; cstates[c3].ctype = 3;
length += acpigen_write_CST_package_entry(&cstates[c3]); length += acpigen_write_CST_package_entry(&cstates[c3]);
} }