[coreboot-gerrit] Patch set updated for coreboot: getac/p470: Add C-State values

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Tue Aug 11 10:18:47 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11142

-gerrit

commit 16751368ba390c731b6f029f8cfdc9d8a1209625
Author: Patrick Georgi <patrick at 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 at georgi-clan.de>
    Found-by: fwts 15.08
---
 src/mainboard/getac/p470/cstates.c | 22 +++++++++++++++++++++-
 1 file changed, 21 insertions(+), 1 deletion(-)

diff --git a/src/mainboard/getac/p470/cstates.c b/src/mainboard/getac/p470/cstates.c
index 2d543ff..7a2a3e5 100644
--- a/src/mainboard/getac/p470/cstates.c
+++ b/src/mainboard/getac/p470/cstates.c
@@ -1,7 +1,27 @@
 #include <device/device.h>
 #include <arch/x86/include/arch/acpigen.h>
+#include <southbridge/intel/i82801gx/i82801gx.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 }, DEFAULT_PMBASE + LV2, 0 }
+	},
+	{
+		/* acpi C3 / cpu C2 */
+		2, 0x11,  250,
+		{ ACPI_ADDRESS_SPACE_IO, 8, 0, { 0 }, DEFAULT_PMBASE + LV3, 0 }
+	},
+};
 
 int get_cst_entries(acpi_cstate_t **entries)
 {
-	return 0;
+	*entries = cst_entries;
+	return ARRAY_SIZE(cst_entries);
 }



More information about the coreboot-gerrit mailing list