msm_dsi_pll_set_usecase() function is not used outside of individual DSI
PHY drivers, so drop it in favour of calling the the respective
set_usecase functions directly.

Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>
---
 drivers/gpu/drm/msm/dsi/dsi.h              | 7 -------
 drivers/gpu/drm/msm/dsi/phy/dsi_phy.h      | 2 --
 drivers/gpu/drm/msm/dsi/phy/dsi_phy_10nm.c | 4 +---
 drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c | 4 +---
 drivers/gpu/drm/msm/dsi/phy/dsi_phy_7nm.c  | 4 +---
 drivers/gpu/drm/msm/dsi/phy/dsi_pll.c      | 9 ---------
 6 files changed, 3 insertions(+), 27 deletions(-)

diff --git a/drivers/gpu/drm/msm/dsi/dsi.h b/drivers/gpu/drm/msm/dsi/dsi.h
index 21cf883fb6f1..a130330b97b6 100644
--- a/drivers/gpu/drm/msm/dsi/dsi.h
+++ b/drivers/gpu/drm/msm/dsi/dsi.h
@@ -112,8 +112,6 @@ int msm_dsi_pll_get_clk_provider(struct msm_dsi_pll *pll,
        struct clk **byte_clk_provider, struct clk **pixel_clk_provider);
 void msm_dsi_pll_save_state(struct msm_dsi_pll *pll);
 int msm_dsi_pll_restore_state(struct msm_dsi_pll *pll);
-int msm_dsi_pll_set_usecase(struct msm_dsi_pll *pll,
-                           enum msm_dsi_phy_usecase uc);
 #else
 static inline void msm_dsi_pll_destroy(struct msm_dsi_pll *pll)
 {
@@ -130,11 +128,6 @@ static inline int msm_dsi_pll_restore_state(struct 
msm_dsi_pll *pll)
 {
        return 0;
 }
-static inline int msm_dsi_pll_set_usecase(struct msm_dsi_pll *pll,
-                                         enum msm_dsi_phy_usecase uc)
-{
-       return -ENODEV;
-}
 #endif
 
 /* dsi host */
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h 
b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
index ce2969c63927..b8dd8072d24c 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
@@ -33,8 +33,6 @@ struct msm_dsi_pll_ops {
        void (*destroy)(struct msm_dsi_pll *pll);
        void (*save_state)(struct msm_dsi_pll *pll);
        int (*restore_state)(struct msm_dsi_pll *pll);
-       int (*set_usecase)(struct msm_dsi_pll *pll,
-                          enum msm_dsi_phy_usecase uc);
 };
 
 struct msm_dsi_phy_cfg {
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_10nm.c 
b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_10nm.c
index 799d7f44cf6d..339f7a56e555 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_10nm.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_10nm.c
@@ -1046,7 +1046,7 @@ static int dsi_10nm_phy_enable(struct msm_dsi_phy *phy, 
int src_pll_id,
        /* Select full-rate mode */
        dsi_phy_write(base + REG_DSI_10nm_PHY_CMN_CTRL_2, 0x40);
 
-       ret = msm_dsi_pll_set_usecase(phy->pll, phy->usecase);
+       ret = dsi_pll_10nm_set_usecase(phy->pll, phy->usecase);
        if (ret) {
                DRM_DEV_ERROR(&phy->pdev->dev, "%s: set pll usecase failed, 
%d\n",
                        __func__, ret);
@@ -1122,7 +1122,6 @@ const struct msm_dsi_phy_cfg dsi_phy_10nm_cfgs = {
                .destroy = dsi_pll_10nm_destroy,
                .save_state = dsi_pll_10nm_save_state,
                .restore_state = dsi_pll_10nm_restore_state,
-               .set_usecase = dsi_pll_10nm_set_usecase,
        },
        .min_pll_rate = 1000000000UL,
        .max_pll_rate = 3500000000UL,
@@ -1150,7 +1149,6 @@ const struct msm_dsi_phy_cfg dsi_phy_10nm_8998_cfgs = {
                .destroy = dsi_pll_10nm_destroy,
                .save_state = dsi_pll_10nm_save_state,
                .restore_state = dsi_pll_10nm_restore_state,
-               .set_usecase = dsi_pll_10nm_set_usecase,
        },
        .min_pll_rate = 1000000000UL,
        .max_pll_rate = 3500000000UL,
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c 
b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c
index cd66b01ad17d..a141c37d86ab 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_14nm.c
@@ -1190,7 +1190,7 @@ static int dsi_14nm_phy_enable(struct msm_dsi_phy *phy, 
int src_pll_id,
                                REG_DSI_14nm_PHY_CMN_GLBL_TEST_CTRL,
                                DSI_14nm_PHY_CMN_GLBL_TEST_CTRL_BITCLK_HS_SEL);
 
-       ret = msm_dsi_pll_set_usecase(phy->pll, phy->usecase);
+       ret = dsi_pll_14nm_set_usecase(phy->pll, phy->usecase);
        if (ret) {
                DRM_DEV_ERROR(&phy->pdev->dev, "%s: set pll usecase failed, 
%d\n",
                        __func__, ret);
@@ -1247,7 +1247,6 @@ const struct msm_dsi_phy_cfg dsi_phy_14nm_cfgs = {
                .destroy = dsi_pll_14nm_destroy,
                .save_state = dsi_pll_14nm_save_state,
                .restore_state = dsi_pll_14nm_restore_state,
-               .set_usecase = dsi_pll_14nm_set_usecase,
                .disable_seq = dsi_pll_14nm_disable_seq,
                .enable_seq = dsi_pll_14nm_enable_seq,
        },
@@ -1277,7 +1276,6 @@ const struct msm_dsi_phy_cfg dsi_phy_14nm_660_cfgs = {
                .destroy = dsi_pll_14nm_destroy,
                .save_state = dsi_pll_14nm_save_state,
                .restore_state = dsi_pll_14nm_restore_state,
-               .set_usecase = dsi_pll_14nm_set_usecase,
                .disable_seq = dsi_pll_14nm_disable_seq,
                .enable_seq = dsi_pll_14nm_enable_seq,
        },
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_7nm.c 
b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_7nm.c
index 5e53682c4275..d34ddf50c5ff 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_7nm.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_7nm.c
@@ -1061,7 +1061,7 @@ static int dsi_7nm_phy_enable(struct msm_dsi_phy *phy, 
int src_pll_id,
        /* Select full-rate mode */
        dsi_phy_write(base + REG_DSI_7nm_PHY_CMN_CTRL_2, 0x40);
 
-       ret = msm_dsi_pll_set_usecase(phy->pll, phy->usecase);
+       ret = dsi_pll_7nm_set_usecase(phy->pll, phy->usecase);
        if (ret) {
                DRM_DEV_ERROR(&phy->pdev->dev, "%s: set pll usecase failed, 
%d\n",
                        __func__, ret);
@@ -1155,7 +1155,6 @@ const struct msm_dsi_phy_cfg dsi_phy_7nm_cfgs = {
                .destroy = dsi_pll_7nm_destroy,
                .save_state = dsi_pll_7nm_save_state,
                .restore_state = dsi_pll_7nm_restore_state,
-               .set_usecase = dsi_pll_7nm_set_usecase,
        },
        .min_pll_rate = 1000000000UL,
        .max_pll_rate = 3500000000UL,
@@ -1183,7 +1182,6 @@ const struct msm_dsi_phy_cfg dsi_phy_7nm_8150_cfgs = {
                .destroy = dsi_pll_7nm_destroy,
                .save_state = dsi_pll_7nm_save_state,
                .restore_state = dsi_pll_7nm_restore_state,
-               .set_usecase = dsi_pll_7nm_set_usecase,
        },
        .min_pll_rate = 600000000UL,
        .max_pll_rate = (5000000000ULL < ULONG_MAX) ? 5000000000ULL : ULONG_MAX,
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_pll.c 
b/drivers/gpu/drm/msm/dsi/phy/dsi_pll.c
index e607adffe001..98ee4560581a 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_pll.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_pll.c
@@ -113,12 +113,3 @@ int msm_dsi_pll_restore_state(struct msm_dsi_pll *pll)
 
        return 0;
 }
-
-int msm_dsi_pll_set_usecase(struct msm_dsi_pll *pll,
-                           enum msm_dsi_phy_usecase uc)
-{
-       if (pll->cfg->pll_ops.set_usecase)
-               return pll->cfg->pll_ops.set_usecase(pll, uc);
-
-       return 0;
-}
-- 
2.30.2

_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

Reply via email to