diff options
author | Shawn Lin <shawn.lin@rock-chips.com> | 2016-10-13 12:42:13 +0800 |
---|---|---|
committer | Kishon Vijay Abraham I <kishon@ti.com> | 2016-11-05 13:45:02 +0530 |
commit | 232c260982ab0100cbb29055933e45b928252687 (patch) | |
tree | a63d4435166ba9829ef556dfdb956982e5a4b12c /drivers/phy | |
parent | 766325427fdb2a6816778b39331637761bccf13a (diff) | |
download | talos-obmc-linux-232c260982ab0100cbb29055933e45b928252687.tar.gz talos-obmc-linux-232c260982ab0100cbb29055933e45b928252687.zip |
phy-rockchip-pcie: remove deassert of phy_rst from exit callback
The deassert of phy_rst from exit callback is incorrect as when
doing phy_exit, we expect the phy_rst is on asserted state which was
done by power_off callback, but not deasserted state. Meanwhile when
disabling clk_pciephy_ref, the assert/deassert signal can't actually
take effect on the phy. So let's fix it anyway.
Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
Diffstat (limited to 'drivers/phy')
-rw-r--r-- | drivers/phy/phy-rockchip-pcie.c | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/drivers/phy/phy-rockchip-pcie.c b/drivers/phy/phy-rockchip-pcie.c index a2b4c6b58aea..6904633cad68 100644 --- a/drivers/phy/phy-rockchip-pcie.c +++ b/drivers/phy/phy-rockchip-pcie.c @@ -249,21 +249,10 @@ err_refclk: static int rockchip_pcie_phy_exit(struct phy *phy) { struct rockchip_pcie_phy *rk_phy = phy_get_drvdata(phy); - int err = 0; clk_disable_unprepare(rk_phy->clk_pciephy_ref); - err = reset_control_deassert(rk_phy->phy_rst); - if (err) { - dev_err(&phy->dev, "deassert phy_rst err %d\n", err); - goto err_reset; - } - - return err; - -err_reset: - clk_prepare_enable(rk_phy->clk_pciephy_ref); - return err; + return 0; } static const struct phy_ops ops = { |