Stefan Reinauer ([email protected]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1297
-gerrit commit a9a1fc5987493575c759af510647836138052a26 Author: Duncan Laurie <[email protected]> Date: Wed Jun 20 14:38:53 2012 -0700 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 <[email protected]> --- src/cpu/intel/model_206ax/acpi.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/cpu/intel/model_206ax/acpi.c b/src/cpu/intel/model_206ax/acpi.c index dea23e7..5d3f3a0 100644 --- a/src/cpu/intel/model_206ax/acpi.c +++ b/src/cpu/intel/model_206ax/acpi.c @@ -79,7 +79,7 @@ static int generate_cstate_entries(acpi_cstate_t *cstates, length += acpigen_write_CST_package_entry(&cstates[c2]); } if (c3 > 0) { - cstates[c2].ctype = 2; + cstates[c3].ctype = 3; length += acpigen_write_CST_package_entry(&cstates[c3]); } -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
