Change in coreboot[master]: soc/intel/alderlake: Use `is_devfn_enabled()` for Crashlog UPDs

[Patrick Georgi (Code Review)]

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

Change subject: soc/intel/alderlake: Use `is_devfn_enabled()` for Crashlog UPDs
......................................................................

soc/intel/alderlake: Use `is_devfn_enabled()` for Crashlog UPDs

Enable FSP Crashlog UPDs if SA_DEVFN_TMT is enabled and
SOC_INTEL_CRASHLOG is selected by the SoC user.

Change-Id: I0244e2a3f9c000a5c6ecdade1419aa47f51b1e80
Signed-off-by: Subrata Banik 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56298
Reviewed-by: Angel Pons 
Reviewed-by: EricR Lai 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/alderlake/romstage/fsp_params.c
1 file changed, 2 insertions(+), 4 deletions(-)

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

diff --git a/src/soc/intel/alderlake/romstage/fsp_params.c b/src/soc/intel/alderlake/romstage/fsp_params.c
index f9b3d60..b32831e 100644
--- a/src/soc/intel/alderlake/romstage/fsp_params.c
+++ b/src/soc/intel/alderlake/romstage/fsp_params.c
@@ -305,10 +305,8 @@
    m_cfg->PlatformDebugConsent = CONFIG_SOC_INTEL_ALDERLAKE_DEBUG_CONSENT;

/* CrashLog config */
-	if (CONFIG(SOC_INTEL_CRASHLOG)) {
-		m_cfg->CpuCrashLogDevice = 1;
-		m_cfg->CpuCrashLogEnable = 1;
-	}
+	m_cfg->CpuCrashLogDevice = CONFIG(SOC_INTEL_CRASHLOG) && is_devfn_enabled(SA_DEVFN_TMT);
+	m_cfg->CpuCrashLogEnable = m_cfg->CpuCrashLogDevice;
 }

static void soc_memory_init_params(FSP_M_CONFIG *m_cfg,