soc/intel/meteorlake: Switch to SOC_INTEL_COMMON_DEBUG_CONSENT
This patch replaces the SoC-specific config option `SOC_INTEL_METEORLAKE_DEBUG_CONSENT` with the generic `SOC_INTEL_COMMON_DEBUG_CONSENT`. Additionally, updates the FSP configuration to use the new generic config option. TEST=Able to build and boot google/rex0 without any functional impact while debugging. Change-Id: I657d20a38e15eee333a4e45c0c600736148173d4 Signed-off-by: Subrata Banik <subratabanik@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/83961 Reviewed-by: Dinesh Gehlot <digehlot@google.com> Reviewed-by: Felix Held <felix-coreboot@felixheld.de> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
committed by
Felix Held
parent
e6f6d2b76c
commit
2142053986
@@ -357,21 +357,13 @@ config FSP_FD_PATH
|
||||
depends on FSP_USE_REPO
|
||||
default "3rdparty/fsp/MeteorLakeFspBinPkg/Fsp.fd"
|
||||
|
||||
config SOC_INTEL_METEORLAKE_DEBUG_CONSENT
|
||||
int "Debug Consent for MTL"
|
||||
# USB DBC is more common for developers so make this default to 6 if
|
||||
# SOC_INTEL_DEBUG_CONSENT=y
|
||||
# Override platform debug consent value:
|
||||
# 0:Disabled, 2:Enabled Trace Active, 4:Enabled Trace Ready,
|
||||
# 6:Enable Trace Power-Off, 7:Manual
|
||||
config SOC_INTEL_COMMON_DEBUG_CONSENT
|
||||
int
|
||||
default 6 if SOC_INTEL_DEBUG_CONSENT
|
||||
default 2 if SOC_INTEL_COMMON_BLOCK_TRACEHUB
|
||||
default 0
|
||||
help
|
||||
This is to control debug interface on SOC.
|
||||
Setting non-zero value will allow to use DBC or DCI to debug SOC.
|
||||
PlatformDebugConsent in FspmUpd.h has the details.
|
||||
|
||||
Desired platform debug type are
|
||||
0:Disabled, 2:Enabled Trace Active, 4:Enabled Trace Ready,
|
||||
6:Enable Trace Power-Off, 7:Manual
|
||||
|
||||
config DATA_BUS_WIDTH
|
||||
int
|
||||
|
@@ -364,7 +364,7 @@ static void fill_fspm_trace_params(FSP_M_CONFIG *m_cfg,
|
||||
const struct soc_intel_meteorlake_config *config)
|
||||
{
|
||||
/* Set debug probe type */
|
||||
m_cfg->PlatformDebugOption = CONFIG_SOC_INTEL_METEORLAKE_DEBUG_CONSENT;
|
||||
m_cfg->PlatformDebugOption = CONFIG_SOC_INTEL_COMMON_DEBUG_CONSENT;
|
||||
|
||||
/* CrashLog config */
|
||||
if (CONFIG(SOC_INTEL_CRASHLOG)) {
|
||||
|
Reference in New Issue
Block a user