usb: phytium_v2: fix uninitialized phytium_usb in phytium_pci_probe

Log:
drivers/usb/phytium_usb_v2/pci.c:33:6: error: variable 'phytium_usb' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized]
    33 |         if (pci_is_enabled(pdev)) {
       |             ^~~~~~~~~~~~~~~~~~~~
drivers/usb/phytium_usb_v2/pci.c:45:2: note: uninitialized use occurs here
   45 |         phytium_usb->otg_res.start = pci_resource_start(pdev, 0);
      |         ^~~~~~~~~~~
drivers/usb/phytium_usb_v2/pci.c:33:2: note: remove the 'if' if its condition is always true
   33 |         if (pci_is_enabled(pdev)) {
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/usb/phytium_usb_v2/pci.c:24:33: note: initialize the variable 'phytium_usb' to silence this warning
   24 |         struct phytium_usb *phytium_usb;
      |                                        ^
      |                                         = NULL
1 error generated.
similar to the following logic:
	pci_set_master(pdev);
	if (pci_is_enabled(func)) {
		cdnsp = pci_get_drvdata(func);
	} else {
		cdnsp = kzalloc(sizeof(*cdnsp), GFP_KERNEL);
		if (!cdnsp) {
			ret = -ENOMEM;
			goto disable_pci;
		}
	}

Signed-off-by: Wentao Guan <guanwentao@uniontech.com>

Author: opsiff

drivers/usb/phytium_usb_v2/pci.c

@@ -21,7 +21,7 @@ static int phytium_pci_probe(struct pci_dev *pdev,
 {
 	int ret;
 
-	struct phytium_usb *phytium_usb;
+	struct phytium_usb *phytium_usb = NULL;
 
 	ret = pcim_enable_device(pdev);
 	if (ret) {
@@ -32,12 +32,13 @@ static int phytium_pci_probe(struct pci_dev *pdev,
 	pci_set_master(pdev);
 	if (pci_is_enabled(pdev)) {
 		phytium_usb = pci_get_drvdata(pdev);
+	}
+	
+	if (!phytium_usb) {
+		phytium_usb = kzalloc(sizeof(*phytium_usb), GFP_KERNEL);
 		if (!phytium_usb) {
-			phytium_usb = kzalloc(sizeof(*phytium_usb), GFP_KERNEL);
-			if (!phytium_usb) {
-				ret = -ENOMEM;
-				goto disabled_pci;
-			}
+			ret = -ENOMEM;
+			goto disabled_pci;
 		}
 	}