Shelley Chen has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79650?usp=email )
Change subject: soc/intel/xeon/spr: Enforce POR frequency setting ......................................................................
soc/intel/xeon/spr: Enforce POR frequency setting
For RMT build, add kconfig option to enforce Plan Of Record restriction on DDR5 frequency & voltage settings.
Change-Id: Ibfcaaf47fec3bd5d8a858309918b3af2f8d976e9 Signed-off-by: Naresh Solanki naresh.solanki@9elements.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/79650 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Lean Sheng Tan sheng.tan@9elements.com --- M src/soc/intel/xeon_sp/spr/Kconfig M src/soc/intel/xeon_sp/spr/romstage.c 2 files changed, 10 insertions(+), 1 deletion(-)
Approvals: Lean Sheng Tan: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/soc/intel/xeon_sp/spr/Kconfig b/src/soc/intel/xeon_sp/spr/Kconfig index b1c4c78..3960724 100644 --- a/src/soc/intel/xeon_sp/spr/Kconfig +++ b/src/soc/intel/xeon_sp/spr/Kconfig @@ -187,4 +187,12 @@ help Enable Rank Margining Tool. This option is intended for debugging and validation and should normally be disabled. + +config RMT_MEM_POR_FREQ + bool "Enforce Plan Of Record restrictions for DDR5 frequency and voltage" + default n + depends on ENABLE_RMT + help + When RMT is enabled. Select this option to enforce Intel Plan Of Record(POR) + restriction on DDR5 frequency & voltage settings. endif diff --git a/src/soc/intel/xeon_sp/spr/romstage.c b/src/soc/intel/xeon_sp/spr/romstage.c index 2b377a4..4cce21f 100644 --- a/src/soc/intel/xeon_sp/spr/romstage.c +++ b/src/soc/intel/xeon_sp/spr/romstage.c @@ -279,7 +279,8 @@ mupd->FspmConfig.serialDebugMsgLvl = 0x3; mupd->FspmConfig.AllowedSocketsInParallel = 0x1; mupd->FspmConfig.EnforcePopulationPor = 0x1; - mupd->FspmConfig.EnforceDdrMemoryFreqPor = 0x0; + if (CONFIG(RMT_MEM_POR_FREQ)) + mupd->FspmConfig.EnforceDdrMemoryFreqPor = 0x0; }
/* SPR-FSP has no UPD to disable HDA, so do it manually here... */