[coreboot-gerrit] [S] Change in coreboot[master]: soc/intel/meteorlake: Hook UPD CnviWifiCore

[Kapil Porwal (Code Review)]

Kapil Porwal has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74306 )

Change subject: soc/intel/meteorlake: Hook UPD CnviWifiCore
......................................................................

soc/intel/meteorlake: Hook UPD CnviWifiCore

Hook the newly created/exposed CnviWifiCore UPD using chip driver.
Enable this option by default to maintain the existing behavior.

BUG=b:270985197
TEST=Verified by enabling/disabling the UPD on google/rex

Signed-off-by: Kapil Porwal 
Change-Id: I5b4662c2a064f7c9074797c8a2541dcf1dd686fe
---
M src/soc/intel/meteorlake/chip.h
M src/soc/intel/meteorlake/chipset.cb
M src/soc/intel/meteorlake/fsp_params.c
3 files changed, 24 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/74306/1

diff --git a/src/soc/intel/meteorlake/chip.h b/src/soc/intel/meteorlake/chip.h
index 9d9a600..2ded40e 100644
--- a/src/soc/intel/meteorlake/chip.h
+++ b/src/soc/intel/meteorlake/chip.h
@@ -242,6 +242,9 @@
     */
    uint8_t serial_io_gspi_cs_state[CONFIG_SOC_INTEL_COMMON_BLOCK_GSPI_MAX];

+	/* CNVi WiFi Core Enable/Disable */
+	bool cnvi_wifi_core;
+
    /* CNVi BT Core Enable/Disable */
    bool cnvi_bt_core;

diff --git a/src/soc/intel/meteorlake/chipset.cb b/src/soc/intel/meteorlake/chipset.cb
index 0962a7c..7752b63 100644
--- a/src/soc/intel/meteorlake/chipset.cb
+++ b/src/soc/intel/meteorlake/chipset.cb
@@ -18,6 +18,10 @@
    # putting it under register "common_soc_config" in overridetree.cb file.
    register "common_soc_config.pch_thermal_trip" = "130"

+
+	# Enable CNVi WiFi
+	register "cnvi_wifi_core" = "true"
+
    device domain 0 on
    	device pci 00.0 alias system_agent on end
    	device pci 01.0 alias pcie_rp12 off end
diff --git a/src/soc/intel/meteorlake/fsp_params.c b/src/soc/intel/meteorlake/fsp_params.c
index d345f1b..6be7887 100644
--- a/src/soc/intel/meteorlake/fsp_params.c
+++ b/src/soc/intel/meteorlake/fsp_params.c
@@ -562,6 +562,7 @@
 {
    /* CNVi */
    s_cfg->CnviMode = is_devfn_enabled(PCI_DEVFN_CNVI_WIFI);
+	s_cfg->CnviWifiCore = config->cnvi_wifi_core;
    s_cfg->CnviBtCore = config->cnvi_bt_core;
    s_cfg->CnviBtAudioOffload = config->cnvi_bt_audio_offload;
    /* Assert if CNVi BT is enabled without CNVi being enabled. */