Commit 1cc9a559 authored by Rob Herring's avatar Rob Herring Committed by Lorenzo Pieralisi
Browse files

PCI: dwc/intel-gw: Remove some unneeded function wrappers 

Remove some of the pointless levels of functions that just wrap or group
a series of other functions.

Link: https://lore.kernel.org/r/20201105211159.1814485-5-robh@kernel.org


Tested-by: default avatarMarek Szyprowski <m.szyprowski@samsung.com>
Signed-off-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarLorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>
parent a0fd361d

drivers/pci/controller/dwc/pcie-intel-gw.c

+16 −31
Original line number Diff line number Diff line
@@ -152,19 +152,6 @@ static void intel_pcie_init_n_fts(struct dw_pcie *pci) 
	pci->n_fts[0] = PORT_AFR_N_FTS_GEN12_DFT;

}



static void intel_pcie_rc_setup(struct intel_pcie_port *lpp)

{

	struct dw_pcie *pci = &lpp->pci;



	pci->atu_base = pci->dbi_base + 0xC0000;



	intel_pcie_ltssm_disable(lpp);

	intel_pcie_link_setup(lpp);

	intel_pcie_init_n_fts(pci);

	dw_pcie_setup_rc(&pci->pp);

	dw_pcie_upconfig_setup(pci);

}



static int intel_pcie_ep_rst_init(struct intel_pcie_port *lpp)

{

	struct device *dev = lpp->pci.dev;
@@ -216,14 +203,6 @@ static void intel_pcie_device_rst_deassert(struct intel_pcie_port *lpp) 
	gpiod_set_value_cansleep(lpp->reset_gpio, 0);

}



static int intel_pcie_app_logic_setup(struct intel_pcie_port *lpp)

{

	intel_pcie_device_rst_deassert(lpp);

	intel_pcie_ltssm_enable(lpp);



	return dw_pcie_wait_for_link(&lpp->pci);

}



static void intel_pcie_core_irq_disable(struct intel_pcie_port *lpp)

{

	pcie_app_wr(lpp, PCIE_APP_IRNEN, 0);
@@ -273,11 +252,6 @@ static int intel_pcie_get_resources(struct platform_device *pdev) 
	return 0;

}



static void intel_pcie_deinit_phy(struct intel_pcie_port *lpp)

{

	phy_exit(lpp->phy);

}



static int intel_pcie_wait_l2(struct intel_pcie_port *lpp)

{

	u32 value;
@@ -314,6 +288,7 @@ static void intel_pcie_turn_off(struct intel_pcie_port *lpp) 
static int intel_pcie_host_setup(struct intel_pcie_port *lpp)

{

	int ret;

	struct dw_pcie *pci = &lpp->pci;



	intel_pcie_core_rst_assert(lpp);

	intel_pcie_device_rst_assert(lpp);
@@ -330,8 +305,18 @@ static int intel_pcie_host_setup(struct intel_pcie_port *lpp) 
		goto clk_err;

	}



	intel_pcie_rc_setup(lpp);

	ret = intel_pcie_app_logic_setup(lpp);

	pci->atu_base = pci->dbi_base + 0xC0000;



	intel_pcie_ltssm_disable(lpp);

	intel_pcie_link_setup(lpp);

	intel_pcie_init_n_fts(pci);

	dw_pcie_setup_rc(&pci->pp);

	dw_pcie_upconfig_setup(pci);



	intel_pcie_device_rst_deassert(lpp);

	intel_pcie_ltssm_enable(lpp);



	ret = dw_pcie_wait_for_link(pci);

	if (ret)

		goto app_init_err;
@@ -345,7 +330,7 @@ static int intel_pcie_host_setup(struct intel_pcie_port *lpp) 
	clk_disable_unprepare(lpp->core_clk);

clk_err:

	intel_pcie_core_rst_assert(lpp);

	intel_pcie_deinit_phy(lpp);

	phy_exit(lpp->phy);



	return ret;

}
@@ -356,7 +341,7 @@ static void __intel_pcie_remove(struct intel_pcie_port *lpp) 
	intel_pcie_turn_off(lpp);

	clk_disable_unprepare(lpp->core_clk);

	intel_pcie_core_rst_assert(lpp);

	intel_pcie_deinit_phy(lpp);

	phy_exit(lpp->phy);

}



static int intel_pcie_remove(struct platform_device *pdev)
@@ -380,7 +365,7 @@ static int __maybe_unused intel_pcie_suspend_noirq(struct device *dev) 
	if (ret)

		return ret;



	intel_pcie_deinit_phy(lpp);

	phy_exit(lpp->phy);

	clk_disable_unprepare(lpp->core_clk);

	return ret;

}