diff options
| author | Marek Szyprowski <[email protected]> | 2022-06-29 00:04:09 +0200 |
|---|---|---|
| committer | Bjorn Helgaas <[email protected]> | 2022-08-01 15:25:16 -0500 |
| commit | 22f3571cbc84b9150238a43d271ffbf7fad3d81d (patch) | |
| tree | e08ba244102b764c331e47ad90c2f55a11077288 | |
| parent | f2812227bb07e2eaee74253f11cea1576945df31 (diff) | |
PCI: exynos: Correct generic PHY usage
The proper initialization for generic PHYs is to call first phy_init(),
then phy_power_on().
While touching this, remove the phy_reset() call. It is just a left-over
from the obsoleted Exynos5440 support and the current exynos-pcie PHY
driver doesn't even support this function. It is also rarely used by other
drivers.
Link: https://lore.kernel.org/r/[email protected]
Reported-by: Bjorn Helgaas <[email protected]>
Signed-off-by: Marek Szyprowski <[email protected]>
Signed-off-by: Bjorn Helgaas <[email protected]>
Reviewed-by: Chanho Park <[email protected]>
Acked-by: Krzysztof Kozlowski <[email protected]>
| -rw-r--r-- | drivers/pci/controller/dwc/pci-exynos.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pci/controller/dwc/pci-exynos.c b/drivers/pci/controller/dwc/pci-exynos.c index 2ede426e3295..3786946cb09e 100644 --- a/drivers/pci/controller/dwc/pci-exynos.c +++ b/drivers/pci/controller/dwc/pci-exynos.c @@ -258,9 +258,8 @@ static int exynos_pcie_host_init(struct dw_pcie_rp *pp) exynos_pcie_assert_core_reset(ep); - phy_reset(ep->phy); - phy_power_on(ep->phy); phy_init(ep->phy); + phy_power_on(ep->phy); exynos_pcie_deassert_core_reset(ep); exynos_pcie_enable_irq_pulse(ep); |