From a4298bc3f82f476ce82e3e948f6573f5e38dfb25 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Thu, 11 May 2023 10:44:01 +0200 Subject: mb/prodrive/atlas: Shorten FSP-M UPD statements Replace `memupd->FspmConfig.` with `mcfg->` for the sake of brevity. Change-Id: If2e7cccca955b0c1e07c1ecf100d29a923107856 Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/75136 Reviewed-by: Lean Sheng Tan Reviewed-by: Maximilian Brune Reviewed-by: Arthur Heymans Tested-by: build bot (Jenkins) Reviewed-by: Benjamin Doron --- src/mainboard/prodrive/atlas/romstage_fsp_params.c | 49 +++++++++++----------- 1 file changed, 25 insertions(+), 24 deletions(-) diff --git a/src/mainboard/prodrive/atlas/romstage_fsp_params.c b/src/mainboard/prodrive/atlas/romstage_fsp_params.c index d460a0558226..7e357eca4613 100644 --- a/src/mainboard/prodrive/atlas/romstage_fsp_params.c +++ b/src/mainboard/prodrive/atlas/romstage_fsp_params.c @@ -31,6 +31,7 @@ static const struct mb_cfg ddr5_mem_config = { void mainboard_memory_init_params(FSPM_UPD *memupd) { + FSP_M_CONFIG *mcfg = &memupd->FspmConfig; const struct mb_cfg *mem_config = &ddr5_mem_config; const bool half_populated = false; @@ -51,17 +52,17 @@ void mainboard_memory_init_params(FSPM_UPD *memupd) /* Apply profile-specific settings */ switch (get_emi_eeprom_vpd()->profile) { case ATLAS_PROF_REALTIME_PERFORMANCE: - memupd->FspmConfig.HyperThreading = 0; - memupd->FspmConfig.DisPgCloseIdleTimeout = 1; - memupd->FspmConfig.PowerDownMode = 0; - memupd->FspmConfig.DisableStarv2medPrioOnNewReq = 1; + mcfg->HyperThreading = 0; + mcfg->DisPgCloseIdleTimeout = 1; + mcfg->PowerDownMode = 0; + mcfg->DisableStarv2medPrioOnNewReq = 1; break; } /* Enable Audio */ - memupd->FspmConfig.PchHdaAudioLinkHdaEnable = 1; - memupd->FspmConfig.PchHdaSdiEnable[0] = 1; - memupd->FspmConfig.PchHdaSdiEnable[1] = 1; + mcfg->PchHdaAudioLinkHdaEnable = 1; + mcfg->PchHdaSdiEnable[0] = 1; + mcfg->PchHdaSdiEnable[1] = 1; // CPU rootports do not have a ClockReq connected on Atlas. If this is not done, // the following will happens: @@ -69,16 +70,16 @@ void mainboard_memory_init_params(FSPM_UPD *memupd) // - coreboot enables ASPM on CPU root port on pci enemuration // - machine exception is thrown, when trying to access pci configuration space after // enabling ASPM src/device/pciexp_device.c:pciexp_tune_dev(). - memupd->FspmConfig.CpuPcieRpClockReqMsgEnable[0] = 0; - memupd->FspmConfig.CpuPcieRpClockReqMsgEnable[1] = 0; - memupd->FspmConfig.CpuPcieRpClockReqMsgEnable[2] = 0; + mcfg->CpuPcieRpClockReqMsgEnable[0] = 0; + mcfg->CpuPcieRpClockReqMsgEnable[1] = 0; + mcfg->CpuPcieRpClockReqMsgEnable[2] = 0; // shared clock - memupd->FspmConfig.PcieClkSrcUsage[0] = 0x80; - memupd->FspmConfig.PcieClkSrcClkReq[0] = 0xFF; + mcfg->PcieClkSrcUsage[0] = 0x80; + mcfg->PcieClkSrcClkReq[0] = 0xFF; // i225 - memupd->FspmConfig.PcieClkSrcUsage[1] = 9; // RP 10 - memupd->FspmConfig.PcieClkSrcClkReq[1] = 1; + mcfg->PcieClkSrcUsage[1] = 9; // RP 10 + mcfg->PcieClkSrcClkReq[1] = 1; // FIX Apparently Rootports don't like the idea of not having a clksrc and clkreq // attached to it. For example if we set PcieClkSrcClkReq[1] above to 0xFF (unused) @@ -88,14 +89,14 @@ void mainboard_memory_init_params(FSPM_UPD *memupd) // Therefore we put each rootport (that does not have a clkreq, clksrc) // to a not connected (not routed out) clksrc and clkreq. That seems to be a current FSP Bug. // workaround and will be removed as soon as FSP is fixed. - memupd->FspmConfig.PcieClkSrcUsage[2] = 4; // Rootport 5 - memupd->FspmConfig.PcieClkSrcClkReq[2] = 0; - memupd->FspmConfig.PcieClkSrcUsage[3] = 5; // Rootport 6 - memupd->FspmConfig.PcieClkSrcClkReq[3] = 0; - memupd->FspmConfig.PcieClkSrcUsage[4] = 8; // Rootport 9 - memupd->FspmConfig.PcieClkSrcClkReq[4] = 0; - memupd->FspmConfig.PcieClkSrcUsage[5] = 6; // Rootport 7 - memupd->FspmConfig.PcieClkSrcClkReq[5] = 0; - memupd->FspmConfig.PcieClkSrcUsage[6] = 7; // Rootport 8 - memupd->FspmConfig.PcieClkSrcClkReq[6] = 0; + mcfg->PcieClkSrcUsage[2] = 4; // Rootport 5 + mcfg->PcieClkSrcClkReq[2] = 0; + mcfg->PcieClkSrcUsage[3] = 5; // Rootport 6 + mcfg->PcieClkSrcClkReq[3] = 0; + mcfg->PcieClkSrcUsage[4] = 8; // Rootport 9 + mcfg->PcieClkSrcClkReq[4] = 0; + mcfg->PcieClkSrcUsage[5] = 6; // Rootport 7 + mcfg->PcieClkSrcClkReq[5] = 0; + mcfg->PcieClkSrcUsage[6] = 7; // Rootport 8 + mcfg->PcieClkSrcClkReq[6] = 0; } -- cgit v1.2.3