When ath10k_ahb_init() fails, it does not unregister ath10k_pci_driver, which will cause a resource leak issue, call pci_unregister_driver() in the error path to fix this issue. Fixes: 0b523ced9a3c ("ath10k: add basic skeleton to support ahb") Signed-off-by: Xiu Jianfeng <xiujianfeng@xxxxxxxxxx> --- drivers/net/wireless/ath/ath10k/pci.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c index e56c6a6b1379..22f8f8b20762 100644 --- a/drivers/net/wireless/ath/ath10k/pci.c +++ b/drivers/net/wireless/ath/ath10k/pci.c @@ -3800,8 +3800,10 @@ static int __init ath10k_pci_init(void) ret); ret = ath10k_ahb_init(); - if (ret) + if (ret) { printk(KERN_ERR "ahb init failed: %d\n", ret); + pci_unregister_driver(&ath10k_pci_driver); + } return ret; } -- 2.17.1