Kane Chen has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32297
Change subject: soc/intel/cannonlake: Enable PlatformDebugConsent by Kconfig ......................................................................
soc/intel/cannonlake: Enable PlatformDebugConsent by Kconfig
This change is mainly to control PlatformDebugConsent FSP UPD. If CONFIG_USE_DBC_CANNONLAKE=y, the PlatformDebugConsent be enabled.
BUG=b:130203864 TEST=boot ok and PlatformDebugConsent can be controlled by Kconfig
Change-Id: Ib845b5e42bc78fb352a0c97c6301f2aeca522f29 Signed-off-by: Kane Chen kane.chen@intel.com --- M src/soc/intel/cannonlake/Kconfig M src/soc/intel/cannonlake/romstage/fsp_params.c 2 files changed, 8 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/97/32297/1
diff --git a/src/soc/intel/cannonlake/Kconfig b/src/soc/intel/cannonlake/Kconfig index 9796546..bcd3f30 100644 --- a/src/soc/intel/cannonlake/Kconfig +++ b/src/soc/intel/cannonlake/Kconfig @@ -289,4 +289,7 @@ depends on FSP_USE_REPO default "3rdparty/fsp/CoffeeLakeFspBinPkg/Fsp.fd" if SOC_INTEL_COFFEELAKE || SOC_INTEL_WHISKEYLAKE
+config USE_DBC_CANNONLAKE + bool "Support USB DBC for Cannonlake" + endif diff --git a/src/soc/intel/cannonlake/romstage/fsp_params.c b/src/soc/intel/cannonlake/romstage/fsp_params.c index ffdcee4..53040ff 100644 --- a/src/soc/intel/cannonlake/romstage/fsp_params.c +++ b/src/soc/intel/cannonlake/romstage/fsp_params.c @@ -111,8 +111,12 @@ m_cfg->SmbusEnable = 0; else m_cfg->SmbusEnable = smbus->enabled; + /* Set debug probe type */ - m_cfg->PlatformDebugConsent = config->DebugConsent; + if (CONFIG(USE_DBC_CANNONLAKE)) + m_cfg->PlatformDebugConsent = DebugConsent_USB3_DBC; + else + m_cfg->PlatformDebugConsent = config->DebugConsent;
mainboard_memory_init_params(mupd); }