phy: phy-rockchip-samsung-hdptx: Don't request RST_PHY/RST_ROPLL/RST_LCPLL
authorAndy Yan <andy.yan@rock-chips.com>
Fri, 20 Sep 2024 09:36:28 +0000 (17:36 +0800)
committerVinod Koul <vkoul@kernel.org>
Thu, 17 Oct 2024 15:20:22 +0000 (20:50 +0530)
RST_PHY/RST_ROPLL/RST_LCPLL are used for debug only on rk3588,
and they are not exported on rk3576, no need to request it in
driver.

Signed-off-by: Andy Yan <andy.yan@rock-chips.com>
Link: https://lore.kernel.org/r/20240920093629.7410-1-andyshrk@163.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/rockchip/phy-rockchip-samsung-hdptx.c

index 9f08469..ceab9c7 100644 (file)
@@ -256,13 +256,10 @@ struct ropll_config {
 };
 
 enum rk_hdptx_reset {
-       RST_PHY = 0,
-       RST_APB,
+       RST_APB = 0,
        RST_INIT,
        RST_CMN,
        RST_LANE,
-       RST_ROPLL,
-       RST_LCPLL,
        RST_MAX
 };
 
@@ -665,11 +662,6 @@ static void rk_hdptx_phy_disable(struct rk_hdptx_phy *hdptx)
 {
        u32 val;
 
-       /* reset phy and apb, or phy locked flag may keep 1 */
-       reset_control_assert(hdptx->rsts[RST_PHY].rstc);
-       usleep_range(20, 30);
-       reset_control_deassert(hdptx->rsts[RST_PHY].rstc);
-
        reset_control_assert(hdptx->rsts[RST_APB].rstc);
        usleep_range(20, 30);
        reset_control_deassert(hdptx->rsts[RST_APB].rstc);
@@ -792,10 +784,6 @@ static int rk_hdptx_ropll_tmds_cmn_config(struct rk_hdptx_phy *hdptx,
 
        rk_hdptx_pre_power_up(hdptx);
 
-       reset_control_assert(hdptx->rsts[RST_ROPLL].rstc);
-       usleep_range(20, 30);
-       reset_control_deassert(hdptx->rsts[RST_ROPLL].rstc);
-
        rk_hdptx_multi_reg_write(hdptx, rk_hdtpx_common_cmn_init_seq);
        rk_hdptx_multi_reg_write(hdptx, rk_hdtpx_tmds_cmn_init_seq);
 
@@ -1098,13 +1086,10 @@ static int rk_hdptx_phy_probe(struct platform_device *pdev)
                return dev_err_probe(dev, PTR_ERR(hdptx->regmap),
                                     "Failed to init regmap\n");
 
-       hdptx->rsts[RST_PHY].id = "phy";
        hdptx->rsts[RST_APB].id = "apb";
        hdptx->rsts[RST_INIT].id = "init";
        hdptx->rsts[RST_CMN].id = "cmn";
        hdptx->rsts[RST_LANE].id = "lane";
-       hdptx->rsts[RST_ROPLL].id = "ropll";
-       hdptx->rsts[RST_LCPLL].id = "lcpll";
 
        ret = devm_reset_control_bulk_get_exclusive(dev, RST_MAX, hdptx->rsts);
        if (ret)