[coreboot-gerrit] [S] Change in coreboot[main]: soc/intel/cannonlake: Report correct latencies for C states

[Felix Singer (Code Review)]

Attention is currently required from: Nico Huber.

Hello Nico Huber,

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

https://review.coreboot.org/c/coreboot/+/80167?usp=email

to review the following change.

Change subject: soc/intel/cannonlake: Report correct latencies for C states
......................................................................

soc/intel/cannonlake: Report correct latencies for C states

Change-Id: Ie01a53d6f06bc02a53d95e390e16e9963f4c65ee
Signed-off-by: Nico Huber 
---
M src/soc/intel/cannonlake/acpi.c
1 file changed, 9 insertions(+), 9 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/80167/1

diff --git a/src/soc/intel/cannonlake/acpi.c b/src/soc/intel/cannonlake/acpi.c
index 1545d19..1bd81c4 100644
--- a/src/soc/intel/cannonlake/acpi.c
+++ b/src/soc/intel/cannonlake/acpi.c
@@ -53,47 +53,47 @@
    	.resource = MWAIT_RES(0, 1),
    },
    [C_STATE_C6_SHORT_LAT] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(1),
    	.power = C6_POWER,
    	.resource = MWAIT_RES(2, 0),
    },
    [C_STATE_C6_LONG_LAT] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(2),
    	.power = C6_POWER,
    	.resource = MWAIT_RES(2, 1),
    },
    [C_STATE_C7_SHORT_LAT] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(1),
    	.power = C7_POWER,
    	.resource = MWAIT_RES(3, 0),
    },
    [C_STATE_C7_LONG_LAT] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(2),
    	.power = C7_POWER,
    	.resource = MWAIT_RES(3, 1),
    },
    [C_STATE_C7S_SHORT_LAT] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(1),
    	.power = C7_POWER,
    	.resource = MWAIT_RES(3, 2),
    },
    [C_STATE_C7S_LONG_LAT] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(2),
    	.power = C7_POWER,
    	.resource = MWAIT_RES(3, 3),
    },
    [C_STATE_C8] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(3),
    	.power = C8_POWER,
    	.resource = MWAIT_RES(4, 0),
    },
    [C_STATE_C9] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(4),
    	.power = C9_POWER,
    	.resource = MWAIT_RES(5, 0),
    },
    [C_STATE_C10] = {
-		.latency = C_STATE_LATENCY_FROM_LAT_REG(0),
+		.latency = C_STATE_LATENCY_FROM_LAT_REG(5),
    	.power = C10_POWER,
    	.resource = MWAIT_RES(6, 0),
    },