Patrick Georgi (pgeorgi@google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11142
-gerrit
commit 25bb0e7b6d56939713ce3fef0ee080ec8465984d Author: Patrick Georgi patrick@georgi-clan.de Date: Sun Aug 9 09:22:30 2015 +0200
getac/p470: Add C-State values
Derived from what the vendor BIOS is doing.
Change-Id: Ie2cba7b86b6bb3f1dcc4a5e1c189aa45d0aab109 Signed-off-by: Patrick Georgi patrick@georgi-clan.de Found-by: fwts 15.08 --- src/mainboard/getac/p470/cstates.c | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-)
diff --git a/src/mainboard/getac/p470/cstates.c b/src/mainboard/getac/p470/cstates.c index 2d543ff..6dec35e 100644 --- a/src/mainboard/getac/p470/cstates.c +++ b/src/mainboard/getac/p470/cstates.c @@ -1,7 +1,26 @@ #include <device/device.h> #include <arch/x86/include/arch/acpigen.h>
+static acpi_cstate_t cst_entries[] = { + { + /* acpi C1 / cpu C1 */ + 1, 0x01, 1000, + { ACPI_ADDRESS_SPACE_FIXED, 1, 2, { 1 }, 0, 0 } + }, + { + /* acpi C2 / cpu C2 */ + 2, 0x01, 500, + { ACPI_ADDRESS_SPACE_IO, 8, 0, { 0 }, 0x1014, 0 } + }, + { + /* acpi C3 / cpu C2 */ + 2, 0x39, 100, + { ACPI_ADDRESS_SPACE_IO, 8, 0, { 0 }, 0x1016, 0 } + }, +}; + int get_cst_entries(acpi_cstate_t **entries) { - return 0; + *entries = cst_entries; + return ARRAY_SIZE(cst_entries); }