[coreboot-gerrit] Change in coreboot[master]: soc/intel/jasperlake: Update acoustic noise related parameters

[Maulik V Vaghela (Code Review)]

Maulik V Vaghela has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/49187 )

Change subject: soc/intel/jasperlake: Update acoustic noise related parameters
......................................................................

soc/intel/jasperlake: Update acoustic noise related parameters

We need to fill Acoustic noise mitigation related UPDs only in
case when acoustic noise mitigation is enabled. This will also
clarify the user that they need to enable Acoustic noise
mitigation while using this config in mainboard.

BUG=None
BRANCH=dedede
TEST=Check that UPD values are getting filled correctly.

Change-Id: I0cf4ccfced13b0d32b3d20713eace63e66945332
Signed-off-by: Maulik V Vaghela 
---
M src/soc/intel/jasperlake/fsp_params.c
1 file changed, 9 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/87/49187/1

diff --git a/src/soc/intel/jasperlake/fsp_params.c b/src/soc/intel/jasperlake/fsp_params.c
index c03e9dd..0ccc461 100644
--- a/src/soc/intel/jasperlake/fsp_params.c
+++ b/src/soc/intel/jasperlake/fsp_params.c
@@ -237,12 +237,16 @@
    			config->PchPmPwrCycDur);

/* Fill Acoustic noise mitigation related configuration */
-	params->FastPkgCRampDisable[0] = config->FastPkgCRampDisable;
-	params->SlowSlewRate[0] = config->SlowSlewRate;
+	
    params->AcousticNoiseMitigation = config->AcousticNoiseMitigation;
-	params->PreWake = config->PreWake;
-	params->RampUp = config->RampUp;
-	params->RampDown = config->RampDown;
+
+	if (params->AcousticNoiseMitigation) {
+		params->FastPkgCRampDisable[0] = config->FastPkgCRampDisable;
+		params->SlowSlewRate[0] = config->SlowSlewRate;
+		params->PreWake = config->PreWake;
+		params->RampUp = config->RampUp;
+		params->RampDown = config->RampDown;
+	}

/* Override/Fill FSP Silicon Param for mainboard */
    mainboard_silicon_init_params(params);