diff options
| author | Richard Zhu <[email protected]> | 2024-12-10 08:15:56 +0000 |
|---|---|---|
| committer | Bjorn Helgaas <[email protected]> | 2025-01-18 17:35:26 +0000 |
| commit | 86a016e278b78cc2281edd4ffaddbc011c87a593 (patch) | |
| tree | 5fbfd00680949823c4b03b1ea52de1c078d43c89 /drivers/pci/controller/dwc/pcie-designware-host.c | |
| parent | PCI: dw-rockchip: Don't wait for link since we can detect Link Up (diff) | |
| download | kernel-86a016e278b78cc2281edd4ffaddbc011c87a593.tar.gz kernel-86a016e278b78cc2281edd4ffaddbc011c87a593.zip | |
PCI: dwc: Always stop link in the dw_pcie_suspend_noirq
On the i.MX8QM, PCIe link can't be re-established again in
dw_pcie_resume_noirq(), if the LTSSM_EN bit is not cleared
properly in dw_pcie_suspend_noirq().
So, add dw_pcie_stop_link() to dw_pcie_suspend_noirq() to fix
this issue and to align the suspend/resume functions since there
is dw_pcie_start_link() in dw_pcie_resume_noirq() already.
Fixes: 4774faf854f5 ("PCI: dwc: Implement generic suspend/resume functionality")
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Richard Zhu <[email protected]>
[kwilczynski: commit log]
Signed-off-by: Krzysztof Wilczyński <[email protected]>
Signed-off-by: Bjorn Helgaas <[email protected]>
Reviewed-by: Manivannan Sadhasivam <[email protected]>
Diffstat (limited to 'drivers/pci/controller/dwc/pcie-designware-host.c')
| -rw-r--r-- | drivers/pci/controller/dwc/pcie-designware-host.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/pci/controller/dwc/pcie-designware-host.c b/drivers/pci/controller/dwc/pcie-designware-host.c index bcbbe02f8f86..d6d5fff1121c 100644 --- a/drivers/pci/controller/dwc/pcie-designware-host.c +++ b/drivers/pci/controller/dwc/pcie-designware-host.c @@ -952,6 +952,7 @@ int dw_pcie_suspend_noirq(struct dw_pcie *pci) return ret; } + dw_pcie_stop_link(pci); if (pci->pp.ops->deinit) pci->pp.ops->deinit(&pci->pp); |