[coreboot-gerrit] Change in coreboot[master]: soc/intel/jasperlake: Enable Intel FIVR RFI settings

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/47286 )

Change subject: soc/intel/jasperlake: Enable Intel FIVR RFI settings
......................................................................

soc/intel/jasperlake: Enable Intel FIVR RFI settings

We already have RFI UPD settings to mitigate RFI noise issues in
platform. These UPDs were not getting filled via devicetree but
needed to be filled from fsp_params.c

Exporting these UPDs to chip.h will allow OEM/ODMs to fill it
directly from devicetree and also allow us to control it based
on boards instead of keeping it common across SoCs.

BUG=b:171683785
BRANCH=None
TEST=Compilation works and we're able to fill UPD from devicetree.Value
gets reflected in FSP UPDs.

Change-Id: I495cd2294368e6b3035c48b9556a83418d5632de
Signed-off-by: Maulik V Vaghela 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/47286
Reviewed-by: EricR Lai 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/jasperlake/chip.h
M src/soc/intel/jasperlake/fsp_params.c
2 files changed, 22 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  EricR Lai: Looks good to me, approved

diff --git a/src/soc/intel/jasperlake/chip.h b/src/soc/intel/jasperlake/chip.h
index f157f92..0ed4205 100644
--- a/src/soc/intel/jasperlake/chip.h
+++ b/src/soc/intel/jasperlake/chip.h
@@ -341,6 +341,24 @@
     *  - PM_CFG.SLP_LAN_MIN_ASST_WDTH
     */
    uint8_t PchPmPwrCycDur;
+
+	/*
+	 * FIVR RFI Frequency
+	 * PCODE MMIO Mailbox: Set the desired RFI frequency, in increments of 100KHz.
+	 * 0: Auto.
+	 * Range varies based on XTAL clock:
+	 *    0-1918 (Up to 191.8HMz) for 24MHz clock;
+	 *    0-1535 (Up to 153.5MHz) for 19MHz clock.
+	 */
+	uint16_t FivrRfiFrequency;
+
+	/*
+	 * FIVR RFI Spread Spectrum
+	 * Set the Spread Spectrum Range. <b>0: 0%</b>;
+	 * FIVR RFI Spread Spectrum, in 0.1% increments.
+	 * Range: 0.0% to 10.0% (0-100)
+	 */
+	uint8_t FivrSpreadSpectrum;
 };

typedef struct soc_intel_jasperlake_config config_t;
diff --git a/src/soc/intel/jasperlake/fsp_params.c b/src/soc/intel/jasperlake/fsp_params.c
index db27234..92c35c6 100644
--- a/src/soc/intel/jasperlake/fsp_params.c
+++ b/src/soc/intel/jasperlake/fsp_params.c
@@ -216,6 +216,10 @@
    /* Provide correct UART number for FSP debug logs */
    params->SerialIoDebugUartNumber = CONFIG_UART_FOR_CONSOLE;

+	/* Configure FIVR RFI related settings */
+	params->FivrRfiFrequency = config->FivrRfiFrequency;
+	params->FivrSpreadSpectrum = config->FivrSpreadSpectrum;
+
    /* Apply minimum assertion width settings if non-zero */
    if (config->PchPmSlpS3MinAssert)
    	params->PchPmSlpS3MinAssert = config->PchPmSlpS3MinAssert;