[coreboot-gerrit] Change in coreboot[master]: sb/intel/lynxpoint: Drop P_LVLx support in FADT

[Angel Pons (Code Review)]

Attention is currently required from: Nico Huber, Arthur Heymans, Patrick Rudolph.
Hello Nico Huber, Arthur Heymans, Patrick Rudolph,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/coreboot/+/55219

to review the following change.

Change subject: sb/intel/lynxpoint: Drop P_LVLx support in FADT
......................................................................

sb/intel/lynxpoint: Drop P_LVLx support in FADT

IO MWAIT redirection is not enabled, and C-states are reported using the
_CST ACPI object, which overrides the P_LVLx values.

Change-Id: I1b623d19a85045797921b4909e01d5ba521de3ad
Signed-off-by: Angel Pons 
---
M src/southbridge/intel/lynxpoint/fadt.c
1 file changed, 3 insertions(+), 3 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/19/55219/1

diff --git a/src/southbridge/intel/lynxpoint/fadt.c b/src/southbridge/intel/lynxpoint/fadt.c
index dee0601..abf56ca 100644
--- a/src/southbridge/intel/lynxpoint/fadt.c
+++ b/src/southbridge/intel/lynxpoint/fadt.c
@@ -44,8 +44,9 @@
    else
    	fadt->gpe0_blk_len = 2 * 8;

-	fadt->p_lvl2_lat = 1;
-	fadt->p_lvl3_lat = 87;
+	/* P_LVLx not used */
+	fadt->p_lvl2_lat = 101;
+	fadt->p_lvl3_lat = 1001;
    fadt->duty_offset = 0;
    fadt->duty_width = 0;
    fadt->day_alrm = 0xd;
@@ -55,7 +56,6 @@

fadt->flags |= ACPI_FADT_WBINVD |
    	       ACPI_FADT_C1_SUPPORTED |
-		       ACPI_FADT_C2_MP_SUPPORTED |
    	       ACPI_FADT_SLEEP_BUTTON |
    	       ACPI_FADT_SEALED_CASE |
    	       ACPI_FADT_S4_RTC_WAKE |