Commit 88c0e230 authored by Honghui Zhang's avatar Honghui Zhang Committed by Lorenzo Pieralisi
Browse files

PCI: mediatek: Remove the redundant dev->pm_domain check 



There is no need to check whether device have a PM domain attached before
calling the PM runtime methods. Remove it.

Signed-off-by: default avatarHonghui Zhang <honghui.zhang@mediatek.com>
[lorenzo.pieralisi@arm.com: commit log changes]
Signed-off-by: default avatarLorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Acked-by: default avatarRyder Lee <ryder.lee@mediatek.com>
parent a7f172ab

drivers/pci/controller/pcie-mediatek.c

+6 −12
Original line number Original line Diff line number Diff line
@@ -225,11 +225,9 @@ static void mtk_pcie_subsys_powerdown(struct mtk_pcie *pcie) 




	clk_disable_unprepare(pcie->free_ck);

	clk_disable_unprepare(pcie->free_ck);





	if (dev->pm_domain) {

	pm_runtime_put_sync(dev);

	pm_runtime_put_sync(dev);

	pm_runtime_disable(dev);

	pm_runtime_disable(dev);

}

}

}





static void mtk_pcie_port_free(struct mtk_pcie_port *port)

static void mtk_pcie_port_free(struct mtk_pcie_port *port)

{

{
@@ -998,10 +996,8 @@ static int mtk_pcie_subsys_powerup(struct mtk_pcie *pcie) 
		pcie->free_ck = NULL;

		pcie->free_ck = NULL;

	}

	}





	if (dev->pm_domain) {

	pm_runtime_enable(dev);

	pm_runtime_enable(dev);

	pm_runtime_get_sync(dev);

	pm_runtime_get_sync(dev);

	}





	/* enable top level clock */

	/* enable top level clock */

	err = clk_prepare_enable(pcie->free_ck);

	err = clk_prepare_enable(pcie->free_ck);
@@ -1013,10 +1009,8 @@ static int mtk_pcie_subsys_powerup(struct mtk_pcie *pcie) 
	return 0;

	return 0;





err_free_ck:

err_free_ck:

	if (dev->pm_domain) {

	pm_runtime_put_sync(dev);

	pm_runtime_put_sync(dev);

	pm_runtime_disable(dev);

	pm_runtime_disable(dev);

	}





	return err;

	return err;

}

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE