mirror of
https://github.com/torvalds/linux.git
synced 2026-05-28 00:53:34 +02:00
PCI: dwc: Perform cleanup in the error path of dw_pcie_resume_noirq()
If the dw_pcie_resume_noirq() API fails, it just returns the errno without
doing cleanup in the error path, leading to resource leak.
So perform cleanup in the error path.
Fixes: 4774faf854 ("PCI: dwc: Implement generic suspend/resume functionality")
Reported-by: Senchuan Zhang <zhangsenchuan@eswincomputing.com>
Closes: https://lore.kernel.org/linux-pci/78296255.3869.19c8eb694d6.Coremail.zhangsenchuan@eswincomputing.com
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@oss.qualcomm.com>
Signed-off-by: Manivannan Sadhasivam <mani@kernel.org>
Link: https://patch.msgid.link/20260226133951.296743-1-mani@kernel.org
This commit is contained in:
parent
94cbea0f63
commit
edb5ca3262
|
|
@ -1304,15 +1304,24 @@ int dw_pcie_resume_noirq(struct dw_pcie *pci)
|
|||
|
||||
ret = dw_pcie_start_link(pci);
|
||||
if (ret)
|
||||
return ret;
|
||||
goto err_deinit;
|
||||
|
||||
ret = dw_pcie_wait_for_link(pci);
|
||||
if (ret)
|
||||
return ret;
|
||||
if (ret == -ETIMEDOUT)
|
||||
goto err_stop_link;
|
||||
|
||||
if (pci->pp.ops->post_init)
|
||||
pci->pp.ops->post_init(&pci->pp);
|
||||
|
||||
return 0;
|
||||
|
||||
err_stop_link:
|
||||
dw_pcie_stop_link(pci);
|
||||
|
||||
err_deinit:
|
||||
if (pci->pp.ops->deinit)
|
||||
pci->pp.ops->deinit(&pci->pp);
|
||||
|
||||
return ret;
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(dw_pcie_resume_noirq);
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user