[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Configure TcssDma0En and TcssDma1En

[Furquan Shaikh (Code Review)]

Furquan Shaikh has submitted this change. ( https://review.coreboot.org/c/coreboot/+/41812 )

Change subject: soc/intel/tigerlake: Configure TcssDma0En and TcssDma1En
......................................................................

soc/intel/tigerlake: Configure TcssDma0En and TcssDma1En

Determine the TcssDma0 and TcssDma1 enabling based on TBT DMA
controllers setting.

BUG=:b:146624360
TEST=Booted on Volteer and verified TcssDma0 and TcssDma1 enabling.
lspci shows TcssDma0(0d.2) and TcssDma1(0d.3).

Signed-off-by: John Zhao 
Change-Id: I61ac4131481374e9a2a34d1a30f822046c3897fb
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41812
Reviewed-by: Wonkyu Kim 
Reviewed-by: Duncan Laurie 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/tigerlake/chip.h
M src/soc/intel/tigerlake/romstage/fsp_params.c
2 files changed, 11 insertions(+), 6 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Duncan Laurie: Looks good to me, approved
  Wonkyu Kim: Looks good to me, approved

diff --git a/src/soc/intel/tigerlake/chip.h b/src/soc/intel/tigerlake/chip.h
index 5892829..ed09aaa 100644
--- a/src/soc/intel/tigerlake/chip.h
+++ b/src/soc/intel/tigerlake/chip.h
@@ -216,10 +216,6 @@
    uint8_t TcssXhciEn;
    uint8_t TcssXdciEn;

-	/* TCSS DMA */
-	uint8_t TcssDma0En;
-	uint8_t TcssDma1En;
-
    /*
     * IOM Port Config
     * If a port orientation needs to be controlled by the SOC this setting must be
diff --git a/src/soc/intel/tigerlake/romstage/fsp_params.c b/src/soc/intel/tigerlake/romstage/fsp_params.c
index ede5059..f7956c8 100644
--- a/src/soc/intel/tigerlake/romstage/fsp_params.c
+++ b/src/soc/intel/tigerlake/romstage/fsp_params.c
@@ -116,8 +116,17 @@
    m_cfg->TcssXdciEn = config->TcssXdciEn;

/* TCSS DMA */
-	m_cfg->TcssDma0En = config->TcssDma0En;
-	m_cfg->TcssDma1En = config->TcssDma1En;
+	dev = pcidev_path_on_root(SA_DEVFN_TCSS_DMA0);
+	if (dev)
+		m_cfg->TcssDma0En = dev->enabled;
+	else
+		m_cfg->TcssDma0En = 0;
+
+	dev = pcidev_path_on_root(SA_DEVFN_TCSS_DMA1);
+	if (dev)
+		m_cfg->TcssDma1En = dev->enabled;
+	else
+		m_cfg->TcssDma1En = 0;

/* USB4/TBT */
    dev = pcidev_path_on_root(SA_DEVFN_TBT0);