Maulik V Vaghela has uploaded this change for review.

View Change

soc/intel/cannonlake: Enable all C states in FSP

FSP will not enable all C-states by default when we use FSP to do MP
initialization. We need to set UPD from coreboot to enable C-states in
FSP.

Change-Id: I845c61fd14f2f5de21288067eeb7c371710da249
Signed-off-by: Maulik V Vaghela <maulik.v.vaghela@intel.com>
---
M src/soc/intel/cannonlake/fsp_params.c
1 file changed, 3 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/63/32463/1
diff --git a/src/soc/intel/cannonlake/fsp_params.c b/src/soc/intel/cannonlake/fsp_params.c
index 2b83275..7b28058 100644
--- a/src/soc/intel/cannonlake/fsp_params.c
+++ b/src/soc/intel/cannonlake/fsp_params.c
@@ -352,6 +352,9 @@

/* Unlock all GPIO pads */
tconfig->PchUnlockGpioPads = config->PchUnlockGpioPads;
+
+ /* Enable all Cx state in FSP */
+ tconfig->Cx = 1;
}

/* Mainboard GPIO Configuration */

To view, visit change 32463. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I845c61fd14f2f5de21288067eeb7c371710da249
Gerrit-Change-Number: 32463
Gerrit-PatchSet: 1
Gerrit-Owner: Maulik V Vaghela <maulik.v.vaghela@intel.com>
Gerrit-MessageType: newchange