diff options
author | Honghui Zhang <honghui.zhang@mediatek.com> | 2018-10-15 04:08:54 -0400 |
---|---|---|
committer | Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> | 2018-10-17 10:35:34 -0400 |
commit | 88c0e230bb4a79680ee5f05ddba879d436f98f13 (patch) | |
tree | 7fc7118c5f42eedcbb8f1f5bfc3533ca6ea140eb | |
parent | a7f172ab6a8e755e60311f27512034b0441ef421 (diff) |
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: Honghui Zhang <honghui.zhang@mediatek.com>
[lorenzo.pieralisi@arm.com: commit log changes]
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Acked-by: Ryder Lee <ryder.lee@mediatek.com>
-rw-r--r-- | drivers/pci/controller/pcie-mediatek.c | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/drivers/pci/controller/pcie-mediatek.c b/drivers/pci/controller/pcie-mediatek.c index 8d1364c31774..547e4c76edbb 100644 --- a/drivers/pci/controller/pcie-mediatek.c +++ b/drivers/pci/controller/pcie-mediatek.c | |||
@@ -225,10 +225,8 @@ static void mtk_pcie_subsys_powerdown(struct mtk_pcie *pcie) | |||
225 | 225 | ||
226 | clk_disable_unprepare(pcie->free_ck); | 226 | clk_disable_unprepare(pcie->free_ck); |
227 | 227 | ||
228 | if (dev->pm_domain) { | 228 | pm_runtime_put_sync(dev); |
229 | pm_runtime_put_sync(dev); | 229 | pm_runtime_disable(dev); |
230 | pm_runtime_disable(dev); | ||
231 | } | ||
232 | } | 230 | } |
233 | 231 | ||
234 | static void mtk_pcie_port_free(struct mtk_pcie_port *port) | 232 | 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) | |||
998 | pcie->free_ck = NULL; | 996 | pcie->free_ck = NULL; |
999 | } | 997 | } |
1000 | 998 | ||
1001 | if (dev->pm_domain) { | 999 | pm_runtime_enable(dev); |
1002 | pm_runtime_enable(dev); | 1000 | pm_runtime_get_sync(dev); |
1003 | pm_runtime_get_sync(dev); | ||
1004 | } | ||
1005 | 1001 | ||
1006 | /* enable top level clock */ | 1002 | /* enable top level clock */ |
1007 | err = clk_prepare_enable(pcie->free_ck); | 1003 | err = clk_prepare_enable(pcie->free_ck); |
@@ -1013,10 +1009,8 @@ static int mtk_pcie_subsys_powerup(struct mtk_pcie *pcie) | |||
1013 | return 0; | 1009 | return 0; |
1014 | 1010 | ||
1015 | err_free_ck: | 1011 | err_free_ck: |
1016 | if (dev->pm_domain) { | 1012 | pm_runtime_put_sync(dev); |
1017 | pm_runtime_put_sync(dev); | 1013 | pm_runtime_disable(dev); |
1018 | pm_runtime_disable(dev); | ||
1019 | } | ||
1020 | 1014 | ||
1021 | return err; | 1015 | return err; |
1022 | } | 1016 | } |