[XS] Change in coreboot[main]: soc/intel/alderlake: Fix uninitialized usb_cfg pointer

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Dinesh Gehlot, Jayvik Desai, Kapil Porwal, Nick Vaccaro, Subrata Banik.

Karthik Ramasubramanian has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/84939?usp=email )

Change subject: soc/intel/alderlake: Fix uninitialized usb_cfg pointer
......................................................................

soc/intel/alderlake: Fix uninitialized usb_cfg pointer

This patch addresses uninitialized usb_cfg pointer warning which is also
an error - src/soc/intel/alderlake/fsp_params.c:936:48: error: 'usb_cfg'
may be used uninitialized in this function [-Werror=maybe-uninitialized]

BUG=None
TEST=./util/abuild/abuild

Change-Id: I764fed561dfe2a571f3404fe505997edd7aa5ff7
Signed-off-by: Karthikeyan Ramasubramanian 
---
M src/soc/intel/alderlake/fsp_params.c
1 file changed, 2 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/84939/1

diff --git a/src/soc/intel/alderlake/fsp_params.c b/src/soc/intel/alderlake/fsp_params.c
index 839d161..d4cbf0f 100644
--- a/src/soc/intel/alderlake/fsp_params.c
+++ b/src/soc/intel/alderlake/fsp_params.c
@@ -897,7 +897,7 @@
    	const struct soc_intel_alderlake_config *config)
 {
    struct device *port = NULL;
-	struct drivers_usb_acpi_config *usb_cfg;
+	struct drivers_usb_acpi_config *usb_cfg = NULL;
    bool usb_audio_offload = false;

/* Search through the devicetree for matching USB devices */
@@ -931,7 +931,7 @@
    if (s_cfg->CnviBtAudioOffload && !usb_audio_offload) {
    	printk(BIOS_WARNING, "CNVi BT Audio offload enabled but not in USB driver.\n");
    }
-	if (!s_cfg->CnviBtAudioOffload && usb_audio_offload) {
+	if (!s_cfg->CnviBtAudioOffload && usb_cfg && usb_audio_offload) {
    	printk(BIOS_ERR, "USB BT Audio offload enabled but CNVi BT offload disabled\n");
    	usb_cfg->cnvi_bt_audio_offload = 0;
    }