Commit af48f822 authored by Richard Zhu's avatar Richard Zhu Committed by Bjorn Helgaas
Browse files

PCI: imx6: Reduce resume time by only starting link if it was up before suspend 

i.MX PCIe doesn't support hotplug. During resume, only start PCIe link
training when the link was up before system suspend to avoid the long
latency in the link training period.

Link: https://lore.kernel.org/r/1657783869-19194-14-git-send-email-hongxing.zhu@nxp.com


Signed-off-by: default avatarRichard Zhu <hongxing.zhu@nxp.com>
Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
parent 508919d0

drivers/pci/controller/dwc/pci-imx6.c

+6 −1
Original line number Diff line number Diff line
@@ -67,6 +67,7 @@ struct imx6_pcie { 
	struct dw_pcie		*pci;

	int			reset_gpio;

	bool			gpio_active_high;

	bool			link_is_up;

	struct clk		*pcie_bus;

	struct clk		*pcie_phy;

	struct clk		*pcie_inbound_axi;
@@ -881,11 +882,13 @@ static int imx6_pcie_start_link(struct dw_pcie *pci) 
		dev_info(dev, "Link: Gen2 disabled\n");

	}



	imx6_pcie->link_is_up = true;

	tmp = dw_pcie_readw_dbi(pci, offset + PCI_EXP_LNKSTA);

	dev_info(dev, "Link up, Gen%i\n", tmp & PCI_EXP_LNKSTA_CLS);

	return 0;



err_reset_phy:

	imx6_pcie->link_is_up = false;

	dev_dbg(dev, "PHY DEBUG_R0=0x%08x DEBUG_R1=0x%08x\n",

		dw_pcie_readl_dbi(pci, PCIE_PORT_DEBUG0),

		dw_pcie_readl_dbi(pci, PCIE_PORT_DEBUG1));
@@ -1031,7 +1034,9 @@ static int imx6_pcie_resume_noirq(struct device *dev) 
		return ret;

	dw_pcie_setup_rc(pp);



	if (imx6_pcie->link_is_up)

		imx6_pcie_start_link(imx6_pcie->pci);



	return 0;

}