Change in coreboot[master]: soc/intel/cannonlake: Disable ACPI PM timer to reduce S0ix power usage

[Subrata Banik (Code Review)]

Subrata Banik has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/34610 )

Change subject: soc/intel/cannonlake: Disable ACPI PM timer to reduce S0ix power usage
......................................................................

soc/intel/cannonlake: Disable ACPI PM timer to reduce S0ix power usage

This patch overrides EnableTcoTimer FSP UPD default value based on
PmTimerDisabled coreboot devcietree config.

BRANCH=none
BUG=b:138152075

Change-Id: Ic051297a960a6810012c4ad11ad4cfbfe7637082
Signed-off-by: Subrata Banik 
---
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/10/34610/1

diff --git a/src/soc/intel/cannonlake/fsp_params.c b/src/soc/intel/cannonlake/fsp_params.c
index 6fb3060..aa8ebc3 100644
--- a/src/soc/intel/cannonlake/fsp_params.c
+++ b/src/soc/intel/cannonlake/fsp_params.c
@@ -366,6 +366,9 @@
    params->PchPwrOptEnable = config->dmipwroptimize;
    params->SataPwrOptEnable = config->satapwroptimize;

+	/* Disable PCH ACPI timer */
+	params->EnableTcoTimer = !config->PmTimerDisabled;
+
    /* Apply minimum assertion width settings if non-zero */
    if (config->PchPmSlpS3MinAssert)
    	params->PchPmSlpS3MinAssert = config->PchPmSlpS3MinAssert;

[Subrata Banik (Code Review)]

Subrata Banik has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/34610 )

Change subject: soc/intel/cannonlake: Disable ACPI PM timer to reduce S0ix power usage
......................................................................

Abandoned