All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] PCI: designware: move remaining rc setup code to dw_pcie_setup_rc()
@ 2016-03-16 11:40 ` Jisheng Zhang
  0 siblings, 0 replies; 62+ messages in thread
From: Jisheng Zhang @ 2016-03-16 11:40 UTC (permalink / raw)
  To: jingoohan1, pratyush.anand, bhelgaas
  Cc: linux-pci, linux-kernel, linux-arm-kernel, Jisheng Zhang

dw_pcie_setup_rc(), as its name indicates, setups the RC. But current
dw_pcie_host_init() also contains some necessary rc setup code.

Another reason: the host may lost power during suspend to ram, the RC
need to be re-setup after resume. The rc can't be correctly resumed
without the rc setup code in dw_pcie_host_init().

So this patch moves the code to dw_pcie_setup_rc() to address the above
two issues. After this patch, each pcie designware driver users could
call dw_pcie_setup_rc() to re-setup rc when resume back.

Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
Since v1:
 - fix gcc warning found by lkp, thanks

 drivers/pci/host/pcie-designware.c | 39 +++++++++++++++++++-------------------
 1 file changed, 19 insertions(+), 20 deletions(-)

diff --git a/drivers/pci/host/pcie-designware.c b/drivers/pci/host/pcie-designware.c
index a4cccd3..261e4a11 100644
--- a/drivers/pci/host/pcie-designware.c
+++ b/drivers/pci/host/pcie-designware.c
@@ -434,7 +434,6 @@ int dw_pcie_host_init(struct pcie_port *pp)
 	struct platform_device *pdev = to_platform_device(pp->dev);
 	struct pci_bus *bus, *child;
 	struct resource *cfg_res;
-	u32 val;
 	int i, ret;
 	LIST_HEAD(res);
 	struct resource_entry *win;
@@ -544,25 +543,6 @@ int dw_pcie_host_init(struct pcie_port *pp)
 	if (pp->ops->host_init)
 		pp->ops->host_init(pp);
 
-	/*
-	 * If the platform provides ->rd_other_conf, it means the platform
-	 * uses its own address translation component rather than ATU, so
-	 * we should not program the ATU here.
-	 */
-	if (!pp->ops->rd_other_conf)
-		dw_pcie_prog_outbound_atu(pp, PCIE_ATU_REGION_INDEX1,
-					  PCIE_ATU_TYPE_MEM, pp->mem_base,
-					  pp->mem_bus_addr, pp->mem_size);
-
-	dw_pcie_wr_own_conf(pp, PCI_BASE_ADDRESS_0, 4, 0);
-
-	/* program correct class for RC */
-	dw_pcie_wr_own_conf(pp, PCI_CLASS_DEVICE, 2, PCI_CLASS_BRIDGE_PCI);
-
-	dw_pcie_rd_own_conf(pp, PCIE_LINK_WIDTH_SPEED_CONTROL, 4, &val);
-	val |= PORT_LOGIC_SPEED_CHANGE;
-	dw_pcie_wr_own_conf(pp, PCIE_LINK_WIDTH_SPEED_CONTROL, 4, val);
-
 	pp->root_bus_nr = pp->busn->start;
 	if (IS_ENABLED(CONFIG_PCI_MSI)) {
 		bus = pci_scan_root_bus_msi(pp->dev, pp->root_bus_nr,
@@ -800,6 +780,25 @@ void dw_pcie_setup_rc(struct pcie_port *pp)
 	val |= PCI_COMMAND_IO | PCI_COMMAND_MEMORY |
 		PCI_COMMAND_MASTER | PCI_COMMAND_SERR;
 	dw_pcie_writel_rc(pp, val, PCI_COMMAND);
+
+	/*
+	 * If the platform provides ->rd_other_conf, it means the platform
+	 * uses its own address translation component rather than ATU, so
+	 * we should not program the ATU here.
+	 */
+	if (!pp->ops->rd_other_conf)
+		dw_pcie_prog_outbound_atu(pp, PCIE_ATU_REGION_INDEX1,
+					  PCIE_ATU_TYPE_MEM, pp->mem_base,
+					  pp->mem_bus_addr, pp->mem_size);
+
+	dw_pcie_wr_own_conf(pp, PCI_BASE_ADDRESS_0, 4, 0);
+
+	/* program correct class for RC */
+	dw_pcie_wr_own_conf(pp, PCI_CLASS_DEVICE, 2, PCI_CLASS_BRIDGE_PCI);
+
+	dw_pcie_rd_own_conf(pp, PCIE_LINK_WIDTH_SPEED_CONTROL, 4, &val);
+	val |= PORT_LOGIC_SPEED_CHANGE;
+	dw_pcie_wr_own_conf(pp, PCIE_LINK_WIDTH_SPEED_CONTROL, 4, val);
 }
 
 MODULE_AUTHOR("Jingoo Han <jg1.han@samsung.com>");
-- 
2.7.0

^ permalink raw reply related	[flat|nested] 62+ messages in thread

end of thread, other threads:[~2016-04-21 15:54 UTC | newest]

Thread overview: 62+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-16 11:40 [PATCH v2] PCI: designware: move remaining rc setup code to dw_pcie_setup_rc() Jisheng Zhang
2016-03-16 11:40 ` Jisheng Zhang
2016-03-17  4:28 ` Pratyush Anand
2016-03-17  4:28   ` Pratyush Anand
2016-03-17  4:28   ` Pratyush Anand
2016-04-05 23:12 ` Bjorn Helgaas
2016-04-05 23:12   ` Bjorn Helgaas
2016-04-06 14:50 ` Gabriele Paoloni
2016-04-06 14:50   ` Gabriele Paoloni
2016-04-06 14:50   ` Gabriele Paoloni
2016-04-07  2:37   ` Jisheng Zhang
2016-04-07  2:37     ` Jisheng Zhang
2016-04-07  2:37     ` Jisheng Zhang
2016-04-07  8:20     ` Gabriele Paoloni
2016-04-07  8:20       ` Gabriele Paoloni
2016-04-07  8:20       ` Gabriele Paoloni
2016-04-07  8:34       ` Jisheng Zhang
2016-04-07  8:34         ` Jisheng Zhang
2016-04-07  8:34         ` Jisheng Zhang
2016-04-07 10:06         ` Gabriele Paoloni
2016-04-07 10:06           ` Gabriele Paoloni
2016-04-07 10:06           ` Gabriele Paoloni
2016-04-07 11:42           ` Jisheng Zhang
2016-04-07 11:42             ` Jisheng Zhang
2016-04-07 11:42             ` Jisheng Zhang
2016-04-07 14:05           ` Bjorn Helgaas
2016-04-07 14:05             ` Bjorn Helgaas
2016-04-07 14:05             ` Bjorn Helgaas
2016-04-08 13:33             ` Gabriele Paoloni
2016-04-08 13:33               ` Gabriele Paoloni
2016-04-08 13:33               ` Gabriele Paoloni
2016-04-08 16:01               ` Bjorn Helgaas
2016-04-08 16:01                 ` Bjorn Helgaas
2016-04-08 16:01                 ` Bjorn Helgaas
2016-04-12  9:43                 ` Gabriele Paoloni
2016-04-12  9:43                   ` Gabriele Paoloni
2016-04-12  9:43                   ` Gabriele Paoloni
2016-04-13  5:51                   ` Jingoo Han
2016-04-13  5:51                     ` Jingoo Han
2016-04-13  7:57                     ` Gabriele Paoloni
2016-04-13  7:57                       ` Gabriele Paoloni
2016-04-13  7:57                       ` Gabriele Paoloni
2016-04-14 11:52                       ` Jingoo Han
2016-04-14 11:52                         ` Jingoo Han
2016-04-14 13:08                         ` Pratyush Anand
2016-04-14 13:08                           ` Pratyush Anand
2016-04-14 13:08                           ` Pratyush Anand
2016-04-14 13:13                           ` Gabriele Paoloni
2016-04-14 13:13                             ` Gabriele Paoloni
2016-04-14 13:13                             ` Gabriele Paoloni
2016-04-21 15:48                   ` Bjorn Helgaas
2016-04-21 15:48                     ` Bjorn Helgaas
2016-04-21 15:48                     ` Bjorn Helgaas
2016-04-21 15:53                     ` Gabriele Paoloni
2016-04-21 15:53                       ` Gabriele Paoloni
2016-04-21 15:53                       ` Gabriele Paoloni
2016-04-07  6:59   ` Pratyush Anand
2016-04-07  6:59     ` Pratyush Anand
2016-04-07  6:59     ` Pratyush Anand
2016-04-07  8:14     ` Gabriele Paoloni
2016-04-07  8:14       ` Gabriele Paoloni
2016-04-07  8:14       ` Gabriele Paoloni

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.