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

[Karthik Ramasubramanian (Code Review)]

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

(

3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
 )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 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84939
Reviewed-by: Jayvik Desai 
Reviewed-by: Subrata Banik 
Tested-by: build bot (Jenkins) 
---
M src/soc/intel/alderlake/fsp_params.c
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Jayvik Desai: Looks good to me, but someone else must approve
  build bot (Jenkins): Verified
  Subrata Banik: Looks good to me, approved

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;
    }