* [PATCH net v2] net: fealnx: fix missing pci_disable_device()
@ 2022-10-25 13:07 Yang Yingliang
2022-10-25 18:29 ` Jakub Kicinski
0 siblings, 1 reply; 2+ messages in thread
From: Yang Yingliang @ 2022-10-25 13:07 UTC (permalink / raw)
To: netdev; +Cc: davem, kuba, leon
Add missing pci_disable_device() in error path of probe() and remove() path.
Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Suggested-by: Leon Romanovsky <leon@kernel.org>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
v1 -> v2:
Don't use pcim_enable_device(), call pci_disable_device() directly.
---
drivers/net/ethernet/fealnx.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/fealnx.c b/drivers/net/ethernet/fealnx.c
index ed18450fd2cc..d72256391d49 100644
--- a/drivers/net/ethernet/fealnx.c
+++ b/drivers/net/ethernet/fealnx.c
@@ -502,7 +502,8 @@ static int fealnx_init_one(struct pci_dev *pdev,
if (len < MIN_REGION_SIZE) {
dev_err(&pdev->dev,
"region size %ld too small, aborting\n", len);
- return -ENODEV;
+ err = -ENODEV;
+ goto err_out_disable;
}
i = pci_request_regions(pdev, boardname);
@@ -671,6 +672,8 @@ static int fealnx_init_one(struct pci_dev *pdev,
pci_iounmap(pdev, ioaddr);
err_out_res:
pci_release_regions(pdev);
+err_out_disable:
+ pci_disable_device(pdev);
return err;
}
@@ -690,6 +693,7 @@ static void fealnx_remove_one(struct pci_dev *pdev)
pci_iounmap(pdev, np->mem);
free_netdev(dev);
pci_release_regions(pdev);
+ pci_disable_device(pdev);
} else
printk(KERN_ERR "fealnx: remove for unknown device\n");
}
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net v2] net: fealnx: fix missing pci_disable_device()
2022-10-25 13:07 [PATCH net v2] net: fealnx: fix missing pci_disable_device() Yang Yingliang
@ 2022-10-25 18:29 ` Jakub Kicinski
0 siblings, 0 replies; 2+ messages in thread
From: Jakub Kicinski @ 2022-10-25 18:29 UTC (permalink / raw)
To: Yang Yingliang; +Cc: netdev, davem, leon
On Tue, 25 Oct 2022 21:07:51 +0800 Yang Yingliang wrote:
> Add missing pci_disable_device() in error path of probe() and remove() path.
>
> Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
> Suggested-by: Leon Romanovsky <leon@kernel.org>
> Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
This is pointless, I'm deleting this driver.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-10-25 18:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-25 13:07 [PATCH net v2] net: fealnx: fix missing pci_disable_device() Yang Yingliang
2022-10-25 18:29 ` Jakub Kicinski
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.