struct msm_dsi_pll;
 #ifdef CONFIG_DRM_MSM_DSI_PLL
 void msm_dsi_pll_destroy(struct msm_dsi_pll *pll);
-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);
 #else
 static inline void msm_dsi_pll_destroy(struct msm_dsi_pll *pll)
 {
 }
-static inline int msm_dsi_pll_get_clk_provider(struct msm_dsi_pll *pll,
-       struct clk **byte_clk_provider, struct clk **pixel_clk_provider)
-{
-       return -ENODEV;
-}
 static inline void msm_dsi_pll_save_state(struct msm_dsi_pll *pll)
 {
 }
 int msm_dsi_host_register(struct mipi_dsi_host *host, bool check_defer);
 void msm_dsi_host_unregister(struct mipi_dsi_host *host);
 int msm_dsi_host_set_src_pll(struct mipi_dsi_host *host,
-                       struct msm_dsi_pll *src_pll);
+                       struct msm_dsi_phy *src_phy);
 void msm_dsi_host_reset_phy(struct mipi_dsi_host *host);
 void msm_dsi_host_get_phy_clk_req(struct mipi_dsi_host *host,
        struct msm_dsi_phy_clk_request *clk_req,
 struct msm_dsi_pll *msm_dsi_phy_get_pll(struct msm_dsi_phy *phy);
 void msm_dsi_phy_set_usecase(struct msm_dsi_phy *phy,
                             enum msm_dsi_phy_usecase uc);
+int msm_dsi_phy_get_clk_provider(struct msm_dsi_phy *phy,
+       struct clk **byte_clk_provider, struct clk **pixel_clk_provider);
 
 #endif /* __DSI_CONNECTOR_H__ */
 
 
 }
 
 int msm_dsi_host_set_src_pll(struct mipi_dsi_host *host,
-       struct msm_dsi_pll *src_pll)
+       struct msm_dsi_phy *src_phy)
 {
        struct msm_dsi_host *msm_host = to_msm_dsi_host(host);
        struct clk *byte_clk_provider, *pixel_clk_provider;
        int ret;
 
-       ret = msm_dsi_pll_get_clk_provider(src_pll,
+       ret = msm_dsi_phy_get_clk_provider(src_phy,
                                &byte_clk_provider, &pixel_clk_provider);
        if (ret) {
                pr_info("%s: can't get provider from pll, don't set parent\n",
 
        struct msm_dsi *other_dsi = dsi_mgr_get_other_dsi(id);
        struct msm_dsi *clk_master_dsi = dsi_mgr_get_dsi(DSI_CLOCK_MASTER);
        struct msm_dsi *clk_slave_dsi = dsi_mgr_get_dsi(DSI_CLOCK_SLAVE);
-       struct msm_dsi_pll *src_pll;
        int ret;
 
        if (!IS_DUAL_DSI()) {
                        return ret;
 
                msm_dsi_phy_set_usecase(msm_dsi->phy, MSM_DSI_PHY_STANDALONE);
-               src_pll = msm_dsi_phy_get_pll(msm_dsi->phy);
-               if (IS_ERR(src_pll))
-                       return PTR_ERR(src_pll);
-               ret = msm_dsi_host_set_src_pll(msm_dsi->host, src_pll);
+               ret = msm_dsi_host_set_src_pll(msm_dsi->host, msm_dsi->phy);
        } else if (!other_dsi) {
                ret = 0;
        } else {
                                        MSM_DSI_PHY_MASTER);
                msm_dsi_phy_set_usecase(clk_slave_dsi->phy,
                                        MSM_DSI_PHY_SLAVE);
-               src_pll = msm_dsi_phy_get_pll(clk_master_dsi->phy);
-               if (IS_ERR(src_pll))
-                       return PTR_ERR(src_pll);
-               ret = msm_dsi_host_set_src_pll(msm_dsi->host, src_pll);
+               ret = msm_dsi_host_set_src_pll(msm_dsi->host, clk_master_dsi->phy);
                if (ret)
                        return ret;
-               ret = msm_dsi_host_set_src_pll(other_dsi->host, src_pll);
+               ret = msm_dsi_host_set_src_pll(other_dsi->host, clk_master_dsi->phy);
        }
 
        return ret;
 
  * Copyright (c) 2015, The Linux Foundation. All rights reserved.
  */
 
+#include <linux/clk-provider.h>
 #include <linux/platform_device.h>
 
 #include "dsi_phy.h"
        if (!match)
                return -ENODEV;
 
+       phy->provided_clocks = devm_kzalloc(dev,
+                       struct_size(phy->provided_clocks, hws, NUM_PROVIDED_CLKS),
+                       GFP_KERNEL);
+       if (!phy->provided_clocks)
+               return -ENOMEM;
+
+       phy->provided_clocks->num = NUM_PROVIDED_CLKS;
+
        phy->cfg = match->data;
        phy->pdev = pdev;
 
                }
        }
 
+       ret = of_clk_add_hw_provider(dev->of_node, of_clk_hw_onecell_get,
+                                    phy->provided_clocks);
+       if (ret) {
+               DRM_DEV_ERROR(dev, "%s: failed to register clk provider: %d\n", __func__, ret);
+               goto fail;
+       }
+
        dsi_phy_disable_resource(phy);
 
        platform_set_drvdata(pdev, phy);
        return 0;
 
 fail:
+       if (phy->pll) {
+               of_clk_del_provider(dev->of_node);
+               msm_dsi_pll_destroy(phy->pll);
+               phy->pll = NULL;
+       }
+
        return ret;
 }
 
        struct msm_dsi_phy *phy = platform_get_drvdata(pdev);
 
        if (phy && phy->pll) {
+               of_clk_del_provider(pdev->dev.of_node);
                msm_dsi_pll_destroy(phy->pll);
                phy->pll = NULL;
        }
        if (phy)
                phy->usecase = uc;
 }
+
+int msm_dsi_phy_get_clk_provider(struct msm_dsi_phy *phy,
+       struct clk **byte_clk_provider, struct clk **pixel_clk_provider)
+{
+       if (byte_clk_provider)
+               *byte_clk_provider = phy->provided_clocks->hws[DSI_BYTE_PLL_CLK]->clk;
+       if (pixel_clk_provider)
+               *pixel_clk_provider = phy->provided_clocks->hws[DSI_PIXEL_PLL_CLK]->clk;
+
+       return -EINVAL;
+}
 
 struct msm_dsi_pll_ops {
        int (*enable_seq)(struct msm_dsi_pll *pll);
        void (*disable_seq)(struct msm_dsi_pll *pll);
-       int (*get_provider)(struct msm_dsi_pll *pll,
-                       struct clk **byte_clk_provider,
-                       struct clk **pixel_clk_provider);
        void (*destroy)(struct msm_dsi_pll *pll);
        void (*save_state)(struct msm_dsi_pll *pll);
        int (*restore_state)(struct msm_dsi_pll *pll);
        u8 hs_halfbyte_en_ckln;
 };
 
+#define DSI_BYTE_PLL_CLK               0
+#define DSI_PIXEL_PLL_CLK              1
+#define NUM_PROVIDED_CLKS              2
+
 struct msm_dsi_phy {
        struct platform_device *pdev;
        void __iomem *base;
        bool regulator_ldo_mode;
 
        struct msm_dsi_pll *pll;
+
+       struct clk_hw_onecell_data *provided_clocks;
 };
 
 /*
 
  *                                                  dsi0_pll_post_out_div_clk
  */
 
-#define DSI_BYTE_PLL_CLK               0
-#define DSI_PIXEL_PLL_CLK              1
-#define NUM_PROVIDED_CLKS              2
-
 #define VCO_REF_CLK_RATE               19200000
 
 struct dsi_pll_regs {
        struct clk_hw *pclk_mux_hw;
        struct clk_hw *out_dsiclk_hw;
 
-       /* clock-provider: */
-       struct clk_hw_onecell_data *hw_data;
-
        struct pll_10nm_cached_state cached_state;
 
        enum msm_dsi_phy_usecase uc;
        return 0;
 }
 
-static int dsi_pll_10nm_get_provider(struct msm_dsi_pll *pll,
-                                    struct clk **byte_clk_provider,
-                                    struct clk **pixel_clk_provider)
-{
-       struct dsi_pll_10nm *pll_10nm = to_pll_10nm(pll);
-       struct clk_hw_onecell_data *hw_data = pll_10nm->hw_data;
-
-       DBG("DSI PLL%d", pll_10nm->id);
-
-       if (byte_clk_provider)
-               *byte_clk_provider = hw_data->hws[DSI_BYTE_PLL_CLK]->clk;
-       if (pixel_clk_provider)
-               *pixel_clk_provider = hw_data->hws[DSI_PIXEL_PLL_CLK]->clk;
-
-       return 0;
-}
-
 static void dsi_pll_10nm_destroy(struct msm_dsi_pll *pll)
 {
        struct dsi_pll_10nm *pll_10nm = to_pll_10nm(pll);
-       struct device *dev = &pll_10nm->pdev->dev;
 
        DBG("DSI PLL%d", pll_10nm->id);
-       of_clk_del_provider(dev->of_node);
 
        clk_hw_unregister_divider(pll_10nm->out_dsiclk_hw);
        clk_hw_unregister_mux(pll_10nm->pclk_mux_hw);
  * state to follow the master PLL's divider/mux state. Therefore, we don't
  * require special clock ops that also configure the slave PLL registers
  */
-static int pll_10nm_register(struct dsi_pll_10nm *pll_10nm)
+static int pll_10nm_register(struct dsi_pll_10nm *pll_10nm, struct clk_hw **provided_clocks)
 {
        char clk_name[32], parent[32], vco_name[32];
        char parent2[32], parent3[32], parent4[32];
                .ops = &clk_ops_dsi_pll_10nm_vco,
        };
        struct device *dev = &pll_10nm->pdev->dev;
-       struct clk_hw_onecell_data *hw_data;
        struct clk_hw *hw;
        int ret;
 
        DBG("DSI%d", pll_10nm->id);
 
-       hw_data = devm_kzalloc(dev, sizeof(*hw_data) +
-                              NUM_PROVIDED_CLKS * sizeof(struct clk_hw *),
-                              GFP_KERNEL);
-       if (!hw_data)
-               return -ENOMEM;
-
        snprintf(vco_name, 32, "dsi%dvco_clk", pll_10nm->id);
        pll_10nm->base.clk_hw.init = &vco_init;
 
        }
 
        pll_10nm->byte_clk_hw = hw;
-       hw_data->hws[DSI_BYTE_PLL_CLK] = hw;
+       provided_clocks[DSI_BYTE_PLL_CLK] = hw;
 
        snprintf(clk_name, 32, "dsi%d_pll_by_2_bit_clk", pll_10nm->id);
        snprintf(parent, 32, "dsi%d_pll_bit_clk", pll_10nm->id);
        }
 
        pll_10nm->out_dsiclk_hw = hw;
-       hw_data->hws[DSI_PIXEL_PLL_CLK] = hw;
-
-       hw_data->num = NUM_PROVIDED_CLKS;
-       pll_10nm->hw_data = hw_data;
-
-       ret = of_clk_add_hw_provider(dev->of_node, of_clk_hw_onecell_get,
-                                    pll_10nm->hw_data);
-       if (ret) {
-               DRM_DEV_ERROR(dev, "failed to register clk provider: %d\n", ret);
-               goto err_dsiclk_hw;
-       }
+       provided_clocks[DSI_PIXEL_PLL_CLK] = hw;
 
        return 0;
 
-err_dsiclk_hw:
-       clk_hw_unregister_divider(pll_10nm->out_dsiclk_hw);
 err_pclk_mux_hw:
        clk_hw_unregister_mux(pll_10nm->pclk_mux_hw);
 err_post_out_div_clk_hw:
 
        pll_10nm->vco_delay = 1;
 
-       ret = pll_10nm_register(pll_10nm);
+       ret = pll_10nm_register(pll_10nm, phy->provided_clocks->hws);
        if (ret) {
                DRM_DEV_ERROR(&pdev->dev, "failed to register PLL: %d\n", ret);
                return ret;
                .pll_init = dsi_pll_10nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_10nm_get_provider,
                .destroy = dsi_pll_10nm_destroy,
                .save_state = dsi_pll_10nm_save_state,
                .restore_state = dsi_pll_10nm_restore_state,
                .pll_init = dsi_pll_10nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_10nm_get_provider,
                .destroy = dsi_pll_10nm_destroy,
                .save_state = dsi_pll_10nm_save_state,
                .restore_state = dsi_pll_10nm_restore_state,
 
 #define POLL_MAX_READS                 15
 #define POLL_TIMEOUT_US                        1000
 
-#define NUM_PROVIDED_CLKS              2
-
 #define VCO_REF_CLK_RATE               19200000
 #define VCO_MIN_RATE                   1300000000UL
 #define VCO_MAX_RATE                   2600000000UL
 
-#define DSI_BYTE_PLL_CLK               0
-#define DSI_PIXEL_PLL_CLK              1
-
 #define DSI_PLL_DEFAULT_VCO_POSTDIV    1
 
 struct dsi_pll_input {
        struct clk_hw *hws[NUM_DSI_CLOCKS_MAX];
        u32 num_hws;
 
-       /* clock-provider: */
-       struct clk_hw_onecell_data *hw_data;
-
        struct pll_14nm_cached_state cached_state;
 
        enum msm_dsi_phy_usecase uc;
        return 0;
 }
 
-static int dsi_pll_14nm_get_provider(struct msm_dsi_pll *pll,
-                                    struct clk **byte_clk_provider,
-                                    struct clk **pixel_clk_provider)
-{
-       struct dsi_pll_14nm *pll_14nm = to_pll_14nm(pll);
-       struct clk_hw_onecell_data *hw_data = pll_14nm->hw_data;
-
-       if (byte_clk_provider)
-               *byte_clk_provider = hw_data->hws[DSI_BYTE_PLL_CLK]->clk;
-       if (pixel_clk_provider)
-               *pixel_clk_provider = hw_data->hws[DSI_PIXEL_PLL_CLK]->clk;
-
-       return 0;
-}
-
 static void dsi_pll_14nm_destroy(struct msm_dsi_pll *pll)
 {
        struct dsi_pll_14nm *pll_14nm = to_pll_14nm(pll);
-       struct platform_device *pdev = pll_14nm->pdev;
        int num_hws = pll_14nm->num_hws;
 
-       of_clk_del_provider(pdev->dev.of_node);
-
        while (num_hws--)
                clk_hw_unregister(pll_14nm->hws[num_hws]);
 }
        return &pll_postdiv->hw;
 }
 
-static int pll_14nm_register(struct dsi_pll_14nm *pll_14nm)
+static int pll_14nm_register(struct dsi_pll_14nm *pll_14nm, struct clk_hw **provided_clocks)
 {
        char clk_name[32], parent[32], vco_name[32];
        struct clk_init_data vco_init = {
        };
        struct device *dev = &pll_14nm->pdev->dev;
        struct clk_hw **hws = pll_14nm->hws;
-       struct clk_hw_onecell_data *hw_data;
        struct clk_hw *hw;
        int num = 0;
        int ret;
 
        DBG("DSI%d", pll_14nm->id);
 
-       hw_data = devm_kzalloc(dev, sizeof(*hw_data) +
-                              NUM_PROVIDED_CLKS * sizeof(struct clk_hw *),
-                              GFP_KERNEL);
-       if (!hw_data)
-               return -ENOMEM;
-
        snprintf(vco_name, 32, "dsi%dvco_clk", pll_14nm->id);
        pll_14nm->base.clk_hw.init = &vco_init;
 
                return PTR_ERR(hw);
 
        hws[num++] = hw;
-       hw_data->hws[DSI_BYTE_PLL_CLK] = hw;
+       provided_clocks[DSI_BYTE_PLL_CLK] = hw;
 
        snprintf(clk_name, 32, "dsi%dn1_postdivby2_clk", pll_14nm->id);
        snprintf(parent, 32, "dsi%dn1_postdiv_clk", pll_14nm->id);
                return PTR_ERR(hw);
 
        hws[num++] = hw;
-       hw_data->hws[DSI_PIXEL_PLL_CLK] = hw;
+       provided_clocks[DSI_PIXEL_PLL_CLK]      = hw;
 
        pll_14nm->num_hws = num;
 
-       hw_data->num = NUM_PROVIDED_CLKS;
-       pll_14nm->hw_data = hw_data;
-
-       ret = of_clk_add_hw_provider(dev->of_node, of_clk_hw_onecell_get,
-                                    pll_14nm->hw_data);
-       if (ret) {
-               DRM_DEV_ERROR(dev, "failed to register clk provider: %d\n", ret);
-               return ret;
-       }
-
        return 0;
 }
 
 
        pll_14nm->vco_delay = 1;
 
-       ret = pll_14nm_register(pll_14nm);
+       ret = pll_14nm_register(pll_14nm, phy->provided_clocks->hws);
        if (ret) {
                DRM_DEV_ERROR(&pdev->dev, "failed to register PLL: %d\n", ret);
                return ret;
                .pll_init = dsi_pll_14nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_14nm_get_provider,
                .destroy = dsi_pll_14nm_destroy,
                .save_state = dsi_pll_14nm_save_state,
                .restore_state = dsi_pll_14nm_restore_state,
                .pll_init = dsi_pll_14nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_14nm_get_provider,
                .destroy = dsi_pll_14nm_destroy,
                .save_state = dsi_pll_14nm_save_state,
                .restore_state = dsi_pll_14nm_restore_state,
 
 #define POLL_MAX_READS                 10
 #define POLL_TIMEOUT_US                50
 
-#define NUM_PROVIDED_CLKS              2
-
 #define VCO_REF_CLK_RATE               19200000
 #define VCO_MIN_RATE                   350000000
 #define VCO_MAX_RATE                   750000000
 
-#define DSI_BYTE_PLL_CLK               0
-#define DSI_PIXEL_PLL_CLK              1
-
 /* v2.0.0 28nm LP implementation */
 #define DSI_PHY_28NM_QUIRK_PHY_LP      BIT(0)
 
        struct clk *clks[NUM_DSI_CLOCKS_MAX];
        u32 num_clks;
 
-       /* clock-provider: */
-       struct clk *provided_clks[NUM_PROVIDED_CLKS];
-       struct clk_onecell_data clk_data;
-
        struct pll_28nm_cached_state cached_state;
 };
 
        return 0;
 }
 
-static int dsi_pll_28nm_get_provider(struct msm_dsi_pll *pll,
-                               struct clk **byte_clk_provider,
-                               struct clk **pixel_clk_provider)
-{
-       struct dsi_pll_28nm *pll_28nm = to_pll_28nm(pll);
-
-       if (byte_clk_provider)
-               *byte_clk_provider = pll_28nm->provided_clks[DSI_BYTE_PLL_CLK];
-       if (pixel_clk_provider)
-               *pixel_clk_provider =
-                               pll_28nm->provided_clks[DSI_PIXEL_PLL_CLK];
-
-       return 0;
-}
-
 static void dsi_pll_28nm_destroy(struct msm_dsi_pll *pll)
 {
        struct dsi_pll_28nm *pll_28nm = to_pll_28nm(pll);
-       int i;
-
-       msm_dsi_pll_helper_unregister_clks(pll_28nm->pdev,
-                                       pll_28nm->clks, pll_28nm->num_clks);
 
-       for (i = 0; i < NUM_PROVIDED_CLKS; i++)
-               pll_28nm->provided_clks[i] = NULL;
+       msm_dsi_pll_helper_unregister_clks(pll_28nm->clks, pll_28nm->num_clks);
 
        pll_28nm->num_clks = 0;
-       pll_28nm->clk_data.clks = NULL;
-       pll_28nm->clk_data.clk_num = 0;
 }
 
-static int pll_28nm_register(struct dsi_pll_28nm *pll_28nm)
+static int pll_28nm_register(struct dsi_pll_28nm *pll_28nm, struct clk_hw **provided_clocks)
 {
        char clk_name[32], parent1[32], parent2[32], vco_name[32];
        struct clk_init_data vco_init = {
        };
        struct device *dev = &pll_28nm->pdev->dev;
        struct clk **clks = pll_28nm->clks;
-       struct clk **provided_clks = pll_28nm->provided_clks;
        int num = 0;
-       int ret;
 
        DBG("%d", pll_28nm->id);
 
 
        snprintf(clk_name, 32, "dsi%dpll", pll_28nm->id);
        snprintf(parent1, 32, "dsi%dvco_clk", pll_28nm->id);
-       clks[num++] = provided_clks[DSI_PIXEL_PLL_CLK] =
-                       clk_register_divider(dev, clk_name,
+       clks[num++] = clk_register_divider(dev, clk_name,
                                parent1, 0, pll_28nm->mmio +
                                REG_DSI_28nm_PHY_PLL_POSTDIV3_CFG,
                                0, 8, 0, NULL);
+       provided_clocks[DSI_PIXEL_PLL_CLK] = __clk_get_hw(clks[num - 1]);
 
        snprintf(clk_name, 32, "dsi%dbyte_mux", pll_28nm->id);
        snprintf(parent1, 32, "dsi%dvco_clk", pll_28nm->id);
 
        snprintf(clk_name, 32, "dsi%dpllbyte", pll_28nm->id);
        snprintf(parent1, 32, "dsi%dbyte_mux", pll_28nm->id);
-       clks[num++] = provided_clks[DSI_BYTE_PLL_CLK] =
-                       clk_register_fixed_factor(dev, clk_name,
+       clks[num++] = clk_register_fixed_factor(dev, clk_name,
                                parent1, CLK_SET_RATE_PARENT, 1, 4);
+       provided_clocks[DSI_BYTE_PLL_CLK] = __clk_get_hw(clks[num - 1]);
 
        pll_28nm->num_clks = num;
 
-       pll_28nm->clk_data.clk_num = NUM_PROVIDED_CLKS;
-       pll_28nm->clk_data.clks = provided_clks;
-
-       ret = of_clk_add_provider(dev->of_node,
-                       of_clk_src_onecell_get, &pll_28nm->clk_data);
-       if (ret) {
-               DRM_DEV_ERROR(dev, "failed to register clk provider: %d\n", ret);
-               return ret;
-       }
-
        return 0;
 }
 
        }
 
        pll = &pll_28nm->base;
+       pll->cfg = phy->cfg;
        if (phy->cfg->quirks & DSI_PHY_28NM_QUIRK_PHY_LP)
                pll_28nm->vco_delay = 1000;
        else
                pll_28nm->vco_delay = 1;
 
-       pll->cfg = phy->cfg;
-
-       ret = pll_28nm_register(pll_28nm);
+       ret = pll_28nm_register(pll_28nm, phy->provided_clocks->hws);
        if (ret) {
                DRM_DEV_ERROR(&pdev->dev, "failed to register PLL: %d\n", ret);
                return ret;
                .pll_init = dsi_pll_28nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_28nm_get_provider,
                .destroy = dsi_pll_28nm_destroy,
                .save_state = dsi_pll_28nm_save_state,
                .restore_state = dsi_pll_28nm_restore_state,
                .pll_init = dsi_pll_28nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_28nm_get_provider,
                .destroy = dsi_pll_28nm_destroy,
                .save_state = dsi_pll_28nm_save_state,
                .restore_state = dsi_pll_28nm_restore_state,
                .pll_init = dsi_pll_28nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_28nm_get_provider,
                .destroy = dsi_pll_28nm_destroy,
                .save_state = dsi_pll_28nm_save_state,
                .restore_state = dsi_pll_28nm_restore_state,
 
 #define POLL_MAX_READS         8000
 #define POLL_TIMEOUT_US                1
 
-#define NUM_PROVIDED_CLKS      2
-
 #define VCO_REF_CLK_RATE       27000000
 #define VCO_MIN_RATE           600000000
 #define VCO_MAX_RATE           1200000000
 
-#define DSI_BYTE_PLL_CLK       0
-#define DSI_PIXEL_PLL_CLK      1
-
 #define VCO_PREF_DIV_RATIO     27
 
 struct pll_28nm_cached_state {
        struct clk *clks[NUM_DSI_CLOCKS_MAX];
        u32 num_clks;
 
-       /* clock-provider: */
-       struct clk *provided_clks[NUM_PROVIDED_CLKS];
-       struct clk_onecell_data clk_data;
-
        struct pll_28nm_cached_state cached_state;
 };
 
        return 0;
 }
 
-static int dsi_pll_28nm_get_provider(struct msm_dsi_pll *pll,
-                               struct clk **byte_clk_provider,
-                               struct clk **pixel_clk_provider)
-{
-       struct dsi_pll_28nm *pll_28nm = to_pll_28nm(pll);
-
-       if (byte_clk_provider)
-               *byte_clk_provider = pll_28nm->provided_clks[DSI_BYTE_PLL_CLK];
-       if (pixel_clk_provider)
-               *pixel_clk_provider =
-                               pll_28nm->provided_clks[DSI_PIXEL_PLL_CLK];
-
-       return 0;
-}
-
 static void dsi_pll_28nm_destroy(struct msm_dsi_pll *pll)
 {
        struct dsi_pll_28nm *pll_28nm = to_pll_28nm(pll);
 
-       msm_dsi_pll_helper_unregister_clks(pll_28nm->pdev,
-                                       pll_28nm->clks, pll_28nm->num_clks);
+       msm_dsi_pll_helper_unregister_clks(pll_28nm->clks, pll_28nm->num_clks);
 }
 
-static int pll_28nm_register(struct dsi_pll_28nm *pll_28nm)
+static int pll_28nm_register(struct dsi_pll_28nm *pll_28nm, struct clk_hw **provided_clocks)
 {
        char *clk_name, *parent_name, *vco_name;
        struct clk_init_data vco_init = {
        };
        struct device *dev = &pll_28nm->pdev->dev;
        struct clk **clks = pll_28nm->clks;
-       struct clk **provided_clks = pll_28nm->provided_clks;
        struct clk_bytediv *bytediv;
        struct clk_init_data bytediv_init = { };
-       int ret, num = 0;
+       int num = 0;
 
        DBG("%d", pll_28nm->id);
 
        if (!vco_name)
                return -ENOMEM;
 
-       parent_name = devm_kzalloc(dev, 32, GFP_KERNEL);
-       if (!parent_name)
-               return -ENOMEM;
-
        clk_name = devm_kzalloc(dev, 32, GFP_KERNEL);
        if (!clk_name)
                return -ENOMEM;
        bytediv_init.num_parents = 1;
 
        /* DIV2 */
-       clks[num++] = provided_clks[DSI_BYTE_PLL_CLK] =
-                       clk_register(dev, &bytediv->hw);
+       clks[num++] = clk_register(dev, &bytediv->hw);
+       provided_clocks[DSI_BYTE_PLL_CLK] = __clk_get_hw(clks[num - 1]);
 
        snprintf(clk_name, 32, "dsi%dpll", pll_28nm->id);
        /* DIV3 */
-       clks[num++] = provided_clks[DSI_PIXEL_PLL_CLK] =
-                       clk_register_divider(dev, clk_name,
+       clks[num++] = clk_register_divider(dev, clk_name,
                                parent_name, 0, pll_28nm->mmio +
                                REG_DSI_28nm_8960_PHY_PLL_CTRL_10,
                                0, 8, 0, NULL);
+       provided_clocks[DSI_PIXEL_PLL_CLK] = __clk_get_hw(clks[num - 1]);
 
        pll_28nm->num_clks = num;
 
-       pll_28nm->clk_data.clk_num = NUM_PROVIDED_CLKS;
-       pll_28nm->clk_data.clks = provided_clks;
-
-       ret = of_clk_add_provider(dev->of_node,
-                       of_clk_src_onecell_get, &pll_28nm->clk_data);
-       if (ret) {
-               DRM_DEV_ERROR(dev, "failed to register clk provider: %d\n", ret);
-               return ret;
-       }
-
        return 0;
 }
 
 
        pll->cfg = phy->cfg;
 
-       ret = pll_28nm_register(pll_28nm);
+       ret = pll_28nm_register(pll_28nm, phy->provided_clocks->hws);
        if (ret) {
                DRM_DEV_ERROR(&pdev->dev, "failed to register PLL: %d\n", ret);
                return ret;
                .pll_init = dsi_pll_28nm_8960_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_28nm_get_provider,
                .destroy = dsi_pll_28nm_destroy,
                .save_state = dsi_pll_28nm_save_state,
                .restore_state = dsi_pll_28nm_restore_state,
 
  *                                                  dsi0_pll_post_out_div_clk
  */
 
-#define DSI_BYTE_PLL_CLK               0
-#define DSI_PIXEL_PLL_CLK              1
-#define NUM_PROVIDED_CLKS              2
-
 #define VCO_REF_CLK_RATE               19200000
 
 struct dsi_pll_regs {
        struct clk_hw *pclk_mux_hw;
        struct clk_hw *out_dsiclk_hw;
 
-       /* clock-provider: */
-       struct clk_hw_onecell_data *hw_data;
-
        struct pll_7nm_cached_state cached_state;
 
        enum msm_dsi_phy_usecase uc;
        return 0;
 }
 
-static int dsi_pll_7nm_get_provider(struct msm_dsi_pll *pll,
-                                    struct clk **byte_clk_provider,
-                                    struct clk **pixel_clk_provider)
-{
-       struct dsi_pll_7nm *pll_7nm = to_pll_7nm(pll);
-       struct clk_hw_onecell_data *hw_data = pll_7nm->hw_data;
-
-       DBG("DSI PLL%d", pll_7nm->id);
-
-       if (byte_clk_provider)
-               *byte_clk_provider = hw_data->hws[DSI_BYTE_PLL_CLK]->clk;
-       if (pixel_clk_provider)
-               *pixel_clk_provider = hw_data->hws[DSI_PIXEL_PLL_CLK]->clk;
-
-       return 0;
-}
-
 static void dsi_pll_7nm_destroy(struct msm_dsi_pll *pll)
 {
        struct dsi_pll_7nm *pll_7nm = to_pll_7nm(pll);
-       struct device *dev = &pll_7nm->pdev->dev;
 
        DBG("DSI PLL%d", pll_7nm->id);
-       of_clk_del_provider(dev->of_node);
 
        clk_hw_unregister_divider(pll_7nm->out_dsiclk_hw);
        clk_hw_unregister_mux(pll_7nm->pclk_mux_hw);
  * state to follow the master PLL's divider/mux state. Therefore, we don't
  * require special clock ops that also configure the slave PLL registers
  */
-static int pll_7nm_register(struct dsi_pll_7nm *pll_7nm)
+static int pll_7nm_register(struct dsi_pll_7nm *pll_7nm, struct clk_hw **provided_clocks)
 {
        char clk_name[32], parent[32], vco_name[32];
        char parent2[32], parent3[32], parent4[32];
                .ops = &clk_ops_dsi_pll_7nm_vco,
        };
        struct device *dev = &pll_7nm->pdev->dev;
-       struct clk_hw_onecell_data *hw_data;
        struct clk_hw *hw;
        int ret;
 
        DBG("DSI%d", pll_7nm->id);
 
-       hw_data = devm_kzalloc(dev, sizeof(*hw_data) +
-                              NUM_PROVIDED_CLKS * sizeof(struct clk_hw *),
-                              GFP_KERNEL);
-       if (!hw_data)
-               return -ENOMEM;
-
        snprintf(vco_name, 32, "dsi%dvco_clk", pll_7nm->id);
        pll_7nm->base.clk_hw.init = &vco_init;
 
        }
 
        pll_7nm->byte_clk_hw = hw;
-       hw_data->hws[DSI_BYTE_PLL_CLK] = hw;
+       provided_clocks[DSI_BYTE_PLL_CLK] = hw;
 
        snprintf(clk_name, 32, "dsi%d_pll_by_2_bit_clk", pll_7nm->id);
        snprintf(parent, 32, "dsi%d_pll_bit_clk", pll_7nm->id);
        }
 
        pll_7nm->out_dsiclk_hw = hw;
-       hw_data->hws[DSI_PIXEL_PLL_CLK] = hw;
-
-       hw_data->num = NUM_PROVIDED_CLKS;
-       pll_7nm->hw_data = hw_data;
-
-       ret = of_clk_add_hw_provider(dev->of_node, of_clk_hw_onecell_get,
-                                    pll_7nm->hw_data);
-       if (ret) {
-               DRM_DEV_ERROR(dev, "failed to register clk provider: %d\n", ret);
-               goto err_dsiclk_hw;
-       }
+       provided_clocks[DSI_PIXEL_PLL_CLK] = hw;
 
        return 0;
 
-err_dsiclk_hw:
-       clk_hw_unregister_divider(pll_7nm->out_dsiclk_hw);
 err_pclk_mux_hw:
        clk_hw_unregister_mux(pll_7nm->pclk_mux_hw);
 err_post_out_div_clk_hw:
 
        pll_7nm->vco_delay = 1;
 
-       ret = pll_7nm_register(pll_7nm);
+       ret = pll_7nm_register(pll_7nm, phy->provided_clocks->hws);
        if (ret) {
                DRM_DEV_ERROR(&pdev->dev, "failed to register PLL: %d\n", ret);
                return ret;
                .pll_init = dsi_pll_7nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_7nm_get_provider,
                .destroy = dsi_pll_7nm_destroy,
                .save_state = dsi_pll_7nm_save_state,
                .restore_state = dsi_pll_7nm_restore_state,
                .pll_init = dsi_pll_7nm_init,
        },
        .pll_ops = {
-               .get_provider = dsi_pll_7nm_get_provider,
                .destroy = dsi_pll_7nm_destroy,
                .save_state = dsi_pll_7nm_save_state,
                .restore_state = dsi_pll_7nm_restore_state,
 
        pll->pll_on = false;
 }
 
-void msm_dsi_pll_helper_unregister_clks(struct platform_device *pdev,
-                                       struct clk **clks, u32 num_clks)
+void msm_dsi_pll_helper_unregister_clks(struct clk **clks, u32 num_clks)
 {
-       of_clk_del_provider(pdev->dev.of_node);
-
        if (!num_clks || !clks)
                return;
 
 /*
  * DSI PLL API
  */
-int msm_dsi_pll_get_clk_provider(struct msm_dsi_pll *pll,
-       struct clk **byte_clk_provider, struct clk **pixel_clk_provider)
-{
-       if (pll->cfg->pll_ops.get_provider)
-               return pll->cfg->pll_ops.get_provider(pll,
-                                       byte_clk_provider,
-                                       pixel_clk_provider);
-
-       return -EINVAL;
-}
-
 void msm_dsi_pll_destroy(struct msm_dsi_pll *pll)
 {
        if (pll->cfg->pll_ops.destroy)
 
 int msm_dsi_pll_helper_clk_prepare(struct clk_hw *hw);
 void msm_dsi_pll_helper_clk_unprepare(struct clk_hw *hw);
 /* misc */
-void msm_dsi_pll_helper_unregister_clks(struct platform_device *pdev,
-                                       struct clk **clks, u32 num_clks);
+void msm_dsi_pll_helper_unregister_clks(struct clk **clks, u32 num_clks);
 
 #endif /* __DSI_PLL_H__ */