net: enetc: clean up before returning in probe()

We recently added this error  path.  We need to call enetc_pci_remove()
before returning.  It cleans up the resources from enetc_pci_probe().

Fixes: 99100d0d9922 ("net: enetc: add preliminary support for i.MX95 ENETC PF")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Wei Fang <wei.fang@nxp.com>
Link: https://patch.msgid.link/93888efa-c838-4682-a7e5-e6bf318e844e@stanley.mountain
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/freescale/enetc/enetc_vf.c b/drivers/net/ethernet/freescale/enetc/enetc_vf.c
index d18c11e406fca85721c38d160114798d255b9bd5..a5f8ce576b6e80ed0f396ae7fb200464d90506c2 100644 (file)
--- a/drivers/net/ethernet/freescale/enetc/enetc_vf.c
+++ b/drivers/net/ethernet/freescale/enetc/enetc_vf.c
@@ -174,9 +174,11 @@ static int enetc_vf_probe(struct pci_dev *pdev,
        si = pci_get_drvdata(pdev);
        si->revision = ENETC_REV_1_0;
        err = enetc_get_driver_data(si);
-       if (err)
-               return dev_err_probe(&pdev->dev, err,
-                                    "Could not get VF driver data\n");
+       if (err) {
+               dev_err_probe(&pdev->dev, err,
+                             "Could not get VF driver data\n");
+               goto err_alloc_netdev;
+       }
 
        enetc_get_si_caps(si);