On Fri, May 11, 2018 at 08:19:37PM +0530, Rajesh Yadav wrote:
> Now, since dpu_power_handle manages only bus scaling
> and power enable/disable notifications which are restricted
> to dpu driver, move dpu_power_handle to dpu folder.
> 
> Changes in v2:
>       - resolved conflict in dpu_unbind
>       - dropped (Reviewed-by: Sean Paul) due to above change
> 

Reviewed-by: Sean Paul <seanp...@chromium.org>

> Signed-off-by: Rajesh Yadav <rya...@codeaurora.org>
> ---
>  drivers/gpu/drm/msm/Makefile                     |   2 +-
>  drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c     |   1 -
>  drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c    |   5 +-
>  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c         |   7 +-
>  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h         |   2 +
>  drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c      |   1 -
>  drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c          |  39 +-
>  drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h          |   1 +
>  drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c | 693 
> +++++++++++++++++++++++
>  drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h | 288 ++++++++++
>  drivers/gpu/drm/msm/dpu_power_handle.c           | 693 
> -----------------------
>  drivers/gpu/drm/msm/dpu_power_handle.h           | 288 ----------
>  drivers/gpu/drm/msm/msm_drv.c                    |   9 -
>  drivers/gpu/drm/msm/msm_drv.h                    |   4 -
>  14 files changed, 1013 insertions(+), 1020 deletions(-)
>  create mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c
>  create mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h
>  delete mode 100644 drivers/gpu/drm/msm/dpu_power_handle.c
>  delete mode 100644 drivers/gpu/drm/msm/dpu_power_handle.h
> 
> diff --git a/drivers/gpu/drm/msm/Makefile b/drivers/gpu/drm/msm/Makefile
> index d9826c1..f578d5a 100644
> --- a/drivers/gpu/drm/msm/Makefile
> +++ b/drivers/gpu/drm/msm/Makefile
> @@ -82,10 +82,10 @@ msm-y := \
>       disp/dpu1/dpu_rm.o \
>       disp/dpu1/dpu_vbif.o \
>       disp/dpu1/dpu_mdss.o \
> +     disp/dpu1/dpu_power_handle.o \
>       dpu_dbg.o \
>       dpu_io_util.o \
>       dpu_dbg_evtlog.o \
> -     dpu_power_handle.o \
>       msm_prop.o \
>       msm_atomic.o \
>       msm_debugfs.o \
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c
> index 5c5cc56..33ab2ac 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c
> @@ -18,7 +18,6 @@
>  #include <linux/kthread.h>
>  
>  #include "dpu_core_irq.h"
> -#include "dpu_power_handle.h"
>  
>  /**
>   * dpu_core_irq_callback_handler - dispatch core interrupts
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c
> index 2cf3fca..d3a1ed9 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c
> @@ -257,7 +257,6 @@ static void _dpu_core_perf_crtc_update_bus(struct dpu_kms 
> *kms,
>                                       = dpu_crtc_get_client_type(crtc);
>       struct drm_crtc *tmp_crtc;
>       struct dpu_crtc_state *dpu_cstate;
> -     struct msm_drm_private *priv = kms->dev->dev_private;
>  
>       drm_for_each_crtc(tmp_crtc, crtc->dev) {
>               if (_dpu_core_perf_crtc_is_power_on(tmp_crtc) &&
> @@ -287,7 +286,7 @@ static void _dpu_core_perf_crtc_update_bus(struct dpu_kms 
> *kms,
>  
>       switch (curr_client_type) {
>       case NRT_CLIENT:
> -             dpu_power_data_bus_set_quota(&priv->phandle, kms->core_client,
> +             dpu_power_data_bus_set_quota(&kms->phandle, kms->core_client,
>                               DPU_POWER_HANDLE_DATA_BUS_CLIENT_NRT,
>                               bus_id, bus_ab_quota, bus_ib_quota);
>               DPU_DEBUG("client:%s bus_id=%d ab=%llu ib=%llu\n", "nrt",
> @@ -295,7 +294,7 @@ static void _dpu_core_perf_crtc_update_bus(struct dpu_kms 
> *kms,
>               break;
>  
>       case RT_CLIENT:
> -             dpu_power_data_bus_set_quota(&priv->phandle, kms->core_client,
> +             dpu_power_data_bus_set_quota(&kms->phandle, kms->core_client,
>                               DPU_POWER_HANDLE_DATA_BUS_CLIENT_RT,
>                               bus_id, bus_ab_quota, bus_ib_quota);
>               DPU_DEBUG("client:%s bus_id=%d ab=%llu ib=%llu\n", "rt",
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> index e2d2e32..99c5e75 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
> @@ -598,6 +598,7 @@ static void dpu_crtc_destroy(struct drm_crtc *crtc)
>       _dpu_crtc_destroy_dest_scaler(dpu_crtc);
>  
>       _dpu_crtc_deinit_events(dpu_crtc);
> +     dpu_crtc->phandle = NULL;
>  
>       drm_crtc_cleanup(crtc);
>       mutex_destroy(&dpu_crtc->crtc_lock);
> @@ -2572,7 +2573,7 @@ static void dpu_crtc_disable(struct drm_crtc *crtc)
>       }
>  
>       if (dpu_crtc->power_event)
> -             dpu_power_handle_unregister_event(&priv->phandle,
> +             dpu_power_handle_unregister_event(dpu_crtc->phandle,
>                               dpu_crtc->power_event);
>  
>  
> @@ -2643,7 +2644,7 @@ static void dpu_crtc_enable(struct drm_crtc *crtc,
>       mutex_unlock(&dpu_crtc->crtc_lock);
>  
>       dpu_crtc->power_event = dpu_power_handle_register_event(
> -             &priv->phandle,
> +             dpu_crtc->phandle,
>               DPU_POWER_EVENT_POST_ENABLE | DPU_POWER_EVENT_POST_DISABLE |
>               DPU_POWER_EVENT_PRE_DISABLE,
>               dpu_crtc_handle_power_event, crtc, dpu_crtc->name);
> @@ -3938,6 +3939,8 @@ struct drm_crtc *dpu_crtc_init(struct drm_device *dev, 
> struct drm_plane *plane)
>       dpu_cp_crtc_init(crtc);
>       dpu_cp_crtc_install_properties(crtc);
>  
> +     dpu_crtc->phandle = &kms->phandle;
> +
>       DPU_DEBUG("%s: successfully initialized crtc\n", dpu_crtc->name);
>       return crtc;
>  }
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h
> index 9304058..671d909 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h
> @@ -184,6 +184,7 @@ struct dpu_crtc_event {
>   * @misr_enable   : boolean entry indicates misr enable/disable status.
>   * @misr_frame_count  : misr frame count provided by client
>   * @misr_data     : store misr data before turning off the clocks.
> + * @phandle: Pointer to power handler
>   * @power_event   : registered power event handle
>   * @cur_perf      : current performance committed to clock/bandwidth driver
>   * @rp_lock       : serialization lock for resource pool
> @@ -240,6 +241,7 @@ struct dpu_crtc {
>       u32 misr_frame_count;
>       u32 misr_data[CRTC_DUAL_MIXERS];
>  
> +     struct dpu_power_handle *phandle;
>       struct dpu_power_event *power_event;
>  
>       struct dpu_core_perf_params cur_perf;
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
> index 298a6ef..11ae6cc 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
> @@ -31,7 +31,6 @@
>  #include "dpu_hw_ctl.h"
>  #include "dpu_formats.h"
>  #include "dpu_encoder_phys.h"
> -#include "dpu_power_handle.h"
>  #include "dpu_crtc.h"
>  #include "dpu_trace.h"
>  #include "dpu_core_irq.h"
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> index 9c3b220..0598cfb 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c
> @@ -1017,24 +1017,19 @@ static long dpu_kms_round_pixclk(struct msm_kms *kms, 
> unsigned long rate,
>  static void _dpu_kms_hw_destroy(struct dpu_kms *dpu_kms)
>  {
>       struct drm_device *dev;
> -     struct msm_drm_private *priv;
>       int i;
>  
>       dev = dpu_kms->dev;
>       if (!dev)
>               return;
>  
> -     priv = dev->dev_private;
> -     if (!priv)
> -             return;
> -
>       if (dpu_kms->hw_intr)
>               dpu_hw_intr_destroy(dpu_kms->hw_intr);
>       dpu_kms->hw_intr = NULL;
>  
>       if (dpu_kms->power_event)
>               dpu_power_handle_unregister_event(
> -                             &priv->phandle, dpu_kms->power_event);
> +                             &dpu_kms->phandle, dpu_kms->power_event);
>  
>       /* safe to call these more than once during shutdown */
>       _dpu_debugfs_destroy(dpu_kms);
> @@ -1067,7 +1062,8 @@ static void _dpu_kms_hw_destroy(struct dpu_kms *dpu_kms)
>       dpu_kms->catalog = NULL;
>  
>       if (dpu_kms->core_client)
> -             dpu_power_client_destroy(&priv->phandle, dpu_kms->core_client);
> +             dpu_power_client_destroy(&dpu_kms->phandle,
> +                     dpu_kms->core_client);
>       dpu_kms->core_client = NULL;
>  
>       if (dpu_kms->vbif[VBIF_NRT])
> @@ -1624,7 +1620,8 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>       }
>  #endif
>  
> -     dpu_kms->core_client = dpu_power_client_create(&priv->phandle, "core");
> +     dpu_kms->core_client = dpu_power_client_create(&dpu_kms->phandle,
> +                                     "core");
>       if (IS_ERR_OR_NULL(dpu_kms->core_client)) {
>               rc = PTR_ERR(dpu_kms->core_client);
>               if (!dpu_kms->core_client)
> @@ -1721,7 +1718,8 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>  #endif
>  
>       rc = dpu_core_perf_init(&dpu_kms->perf, dev, dpu_kms->catalog,
> -                     &priv->phandle, _dpu_kms_get_clk(dpu_kms, "core_clk"));
> +                     &dpu_kms->phandle,
> +                     _dpu_kms_get_clk(dpu_kms, "core_clk"));
>       if (rc) {
>               DPU_ERROR("failed to init perf %d\n", rc);
>               goto perf_err;
> @@ -1765,7 +1763,8 @@ static int dpu_kms_hw_init(struct msm_kms *kms)
>        * Handle (re)initializations during power enable
>        */
>       dpu_kms_handle_power_event(DPU_POWER_EVENT_POST_ENABLE, dpu_kms);
> -     dpu_kms->power_event = dpu_power_handle_register_event(&priv->phandle,
> +     dpu_kms->power_event = dpu_power_handle_register_event(
> +                     &dpu_kms->phandle,
>                       DPU_POWER_EVENT_POST_ENABLE,
>                       dpu_kms_handle_power_event, dpu_kms, "kms");
>  
> @@ -1841,6 +1840,12 @@ static int dpu_bind(struct device *dev, struct device 
> *master, void *data)
>               goto clk_rate_error;
>       }
>  
> +     ret = dpu_power_resource_init(pdev, &dpu_kms->phandle);
> +     if (ret) {
> +             pr_err("dpu power resource init failed\n");
> +             goto power_init_fail;
> +     }
> +
>       platform_set_drvdata(pdev, dpu_kms);
>  
>       msm_kms_init(&dpu_kms->base, &kms_funcs);
> @@ -1853,6 +1858,7 @@ static int dpu_bind(struct device *dev, struct device 
> *master, void *data)
>       priv->kms = &dpu_kms->base;
>       return ret;
>  
> +power_init_fail:
>  clk_rate_error:
>       msm_dss_put_clk(mp->clk_config, mp->num_clk);
>  clk_get_error:
> @@ -1867,6 +1873,7 @@ static void dpu_unbind(struct device *dev, struct 
> device *master, void *data)
>       struct dpu_kms *dpu_kms = platform_get_drvdata(pdev);
>       struct dss_module_power *mp = &dpu_kms->mp;
>  
> +     dpu_power_resource_deinit(pdev, &dpu_kms->phandle);
>       msm_dss_put_clk(mp->clk_config, mp->num_clk);
>       devm_kfree(&pdev->dev, mp->clk_config);
>       mp->num_clk = 0;
> @@ -1897,7 +1904,6 @@ static int dpu_runtime_suspend(struct device *dev)
>       struct platform_device *pdev = to_platform_device(dev);
>       struct dpu_kms *dpu_kms = platform_get_drvdata(pdev);
>       struct drm_device *ddev;
> -     struct msm_drm_private *priv;
>       struct dss_module_power *mp = &dpu_kms->mp;
>  
>       ddev = dpu_kms->dev;
> @@ -1905,10 +1911,9 @@ static int dpu_runtime_suspend(struct device *dev)
>               DPU_ERROR("invalid drm_device\n");
>               goto exit;
>       }
> -     priv = ddev->dev_private;
>  
> -     rc = dpu_power_resource_enable(&priv->phandle,
> -             dpu_kms->core_client, false);
> +     rc = dpu_power_resource_enable(&dpu_kms->phandle,
> +                     dpu_kms->core_client, false);
>       if (rc)
>               DPU_ERROR("resource disable failed: %d\n", rc);
>  
> @@ -1926,7 +1931,6 @@ static int dpu_runtime_resume(struct device *dev)
>       struct platform_device *pdev = to_platform_device(dev);
>       struct dpu_kms *dpu_kms = platform_get_drvdata(pdev);
>       struct drm_device *ddev;
> -     struct msm_drm_private *priv;
>       struct dss_module_power *mp = &dpu_kms->mp;
>  
>       ddev = dpu_kms->dev;
> @@ -1934,7 +1938,6 @@ static int dpu_runtime_resume(struct device *dev)
>               DPU_ERROR("invalid drm_device\n");
>               goto exit;
>       }
> -     priv = ddev->dev_private;
>  
>       rc = msm_dss_enable_clk(mp->clk_config, mp->num_clk, true);
>       if (rc) {
> @@ -1942,8 +1945,8 @@ static int dpu_runtime_resume(struct device *dev)
>               goto exit;
>       }
>  
> -     rc = dpu_power_resource_enable(&priv->phandle, dpu_kms->core_client,
> -             true);
> +     rc = dpu_power_resource_enable(&dpu_kms->phandle,
> +                     dpu_kms->core_client, true);
>       if (rc)
>               DPU_ERROR("resource enable failed: %d\n", rc);
>  
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
> index a8255fe..c48ed4e 100644
> --- a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
> @@ -164,6 +164,7 @@ struct dpu_kms {
>       struct dpu_mdss_cfg *catalog;
>  
>       struct msm_gem_address_space *aspace[MSM_SMMU_DOMAIN_MAX];
> +     struct dpu_power_handle phandle;
>       struct dpu_power_client *core_client;
>  #ifdef CONFIG_ION
>       struct ion_client *iclient;
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c
> new file mode 100644
> index 0000000..77be106
> --- /dev/null
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c
> @@ -0,0 +1,693 @@
> +/* Copyright (c) 2014-2018, The Linux Foundation. All rights reserved.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 and
> + * only version 2 as published by the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + */
> +
> +#define pr_fmt(fmt)  "[drm:%s:%d]: " fmt, __func__, __LINE__
> +
> +#include <linux/kernel.h>
> +#include <linux/of.h>
> +#include <linux/string.h>
> +#include <linux/of_address.h>
> +#include <linux/slab.h>
> +#include <linux/mutex.h>
> +#include <linux/of_platform.h>
> +#ifdef CONFIG_QCOM_BUS_SCALING
> +#include <linux/msm-bus.h>
> +#include <linux/msm-bus-board.h>
> +#endif
> +#include <linux/dpu_io_util.h>
> +
> +#include "dpu_power_handle.h"
> +#include "dpu_trace.h"
> +
> +static const char *data_bus_name[DPU_POWER_HANDLE_DBUS_ID_MAX] = {
> +     [DPU_POWER_HANDLE_DBUS_ID_MNOC] = "qcom,dpu-data-bus",
> +     [DPU_POWER_HANDLE_DBUS_ID_LLCC] = "qcom,dpu-llcc-bus",
> +     [DPU_POWER_HANDLE_DBUS_ID_EBI] = "qcom,dpu-ebi-bus",
> +};
> +
> +const char *dpu_power_handle_get_dbus_name(u32 bus_id)
> +{
> +     if (bus_id < DPU_POWER_HANDLE_DBUS_ID_MAX)
> +             return data_bus_name[bus_id];
> +
> +     return NULL;
> +}
> +
> +static void dpu_power_event_trigger_locked(struct dpu_power_handle *phandle,
> +             u32 event_type)
> +{
> +     struct dpu_power_event *event;
> +
> +     list_for_each_entry(event, &phandle->event_list, list) {
> +             if (event->event_type & event_type)
> +                     event->cb_fnc(event_type, event->usr);
> +     }
> +}
> +
> +struct dpu_power_client *dpu_power_client_create(
> +     struct dpu_power_handle *phandle, char *client_name)
> +{
> +     struct dpu_power_client *client;
> +     static u32 id;
> +
> +     if (!client_name || !phandle) {
> +             pr_err("client name is null or invalid power data\n");
> +             return ERR_PTR(-EINVAL);
> +     }
> +
> +     client = kzalloc(sizeof(struct dpu_power_client), GFP_KERNEL);
> +     if (!client)
> +             return ERR_PTR(-ENOMEM);
> +
> +     mutex_lock(&phandle->phandle_lock);
> +     strlcpy(client->name, client_name, MAX_CLIENT_NAME_LEN);
> +     client->usecase_ndx = VOTE_INDEX_DISABLE;
> +     client->id = id;
> +     client->active = true;
> +     pr_debug("client %s created:%pK id :%d\n", client_name,
> +             client, id);
> +     id++;
> +     list_add(&client->list, &phandle->power_client_clist);
> +     mutex_unlock(&phandle->phandle_lock);
> +
> +     return client;
> +}
> +
> +void dpu_power_client_destroy(struct dpu_power_handle *phandle,
> +     struct dpu_power_client *client)
> +{
> +     if (!client  || !phandle) {
> +             pr_err("reg bus vote: invalid client handle\n");
> +     } else if (!client->active) {
> +             pr_err("dpu power deinit already done\n");
> +             kfree(client);
> +     } else {
> +             pr_debug("bus vote client %s destroyed:%pK id:%u\n",
> +                     client->name, client, client->id);
> +             mutex_lock(&phandle->phandle_lock);
> +             list_del_init(&client->list);
> +             mutex_unlock(&phandle->phandle_lock);
> +             kfree(client);
> +     }
> +}
> +
> +#ifdef CONFIG_QCOM_BUS_SCALING
> +
> +#define MAX_AXI_PORT_COUNT 3
> +
> +static int _dpu_power_data_bus_set_quota(
> +             struct dpu_power_data_bus_handle *pdbus,
> +             u64 ab_quota_rt, u64 ab_quota_nrt,
> +             u64 ib_quota_rt, u64 ib_quota_nrt)
> +{
> +     int new_uc_idx;
> +     u64 ab_quota[MAX_AXI_PORT_COUNT] = {0, 0};
> +     u64 ib_quota[MAX_AXI_PORT_COUNT] = {0, 0};
> +     int rc;
> +
> +     if (pdbus->data_bus_hdl < 1) {
> +             pr_err("invalid bus handle %d\n", pdbus->data_bus_hdl);
> +             return -EINVAL;
> +     }
> +
> +     pdbus->ab_rt = ab_quota_rt;
> +     pdbus->ib_rt = ib_quota_rt;
> +     pdbus->ab_nrt = ab_quota_nrt;
> +     pdbus->ib_nrt = ib_quota_nrt;
> +
> +     if (pdbus->enable) {
> +             ab_quota_rt = max_t(u64, ab_quota_rt,
> +                             DPU_POWER_HANDLE_ENABLE_BUS_AB_QUOTA);
> +             ib_quota_rt = max_t(u64, ib_quota_rt,
> +                             DPU_POWER_HANDLE_ENABLE_BUS_IB_QUOTA);
> +             ab_quota_nrt = max_t(u64, ab_quota_nrt,
> +                             DPU_POWER_HANDLE_ENABLE_BUS_AB_QUOTA);
> +             ib_quota_nrt = max_t(u64, ib_quota_nrt,
> +                             DPU_POWER_HANDLE_ENABLE_BUS_IB_QUOTA);
> +     } else {
> +             ab_quota_rt = min_t(u64, ab_quota_rt,
> +                             DPU_POWER_HANDLE_DISABLE_BUS_AB_QUOTA);
> +             ib_quota_rt = min_t(u64, ib_quota_rt,
> +                             DPU_POWER_HANDLE_DISABLE_BUS_IB_QUOTA);
> +             ab_quota_nrt = min_t(u64, ab_quota_nrt,
> +                             DPU_POWER_HANDLE_DISABLE_BUS_AB_QUOTA);
> +             ib_quota_nrt = min_t(u64, ib_quota_nrt,
> +                             DPU_POWER_HANDLE_DISABLE_BUS_IB_QUOTA);
> +     }
> +
> +     if (!ab_quota_rt && !ab_quota_nrt && !ib_quota_rt && !ib_quota_nrt)  {
> +             new_uc_idx = 0;
> +     } else {
> +             int i;
> +             struct msm_bus_vectors *vect = NULL;
> +             struct msm_bus_scale_pdata *bw_table =
> +                     pdbus->data_bus_scale_table;
> +             u32 nrt_axi_port_cnt = pdbus->nrt_axi_port_cnt;
> +             u32 total_axi_port_cnt = pdbus->axi_port_cnt;
> +             u32 rt_axi_port_cnt = total_axi_port_cnt - nrt_axi_port_cnt;
> +
> +             if (!bw_table || !total_axi_port_cnt ||
> +                 total_axi_port_cnt > MAX_AXI_PORT_COUNT) {
> +                     pr_err("invalid input\n");
> +                     return -EINVAL;
> +             }
> +
> +             if (pdbus->bus_channels) {
> +                     ib_quota_rt = div_u64(ib_quota_rt,
> +                                             pdbus->bus_channels);
> +                     ib_quota_nrt = div_u64(ib_quota_nrt,
> +                                             pdbus->bus_channels);
> +             }
> +
> +             if (nrt_axi_port_cnt) {
> +
> +                     ab_quota_rt = div_u64(ab_quota_rt, rt_axi_port_cnt);
> +                     ab_quota_nrt = div_u64(ab_quota_nrt, nrt_axi_port_cnt);
> +
> +                     for (i = 0; i < total_axi_port_cnt; i++) {
> +                             if (i < rt_axi_port_cnt) {
> +                                     ab_quota[i] = ab_quota_rt;
> +                                     ib_quota[i] = ib_quota_rt;
> +                             } else {
> +                                     ab_quota[i] = ab_quota_nrt;
> +                                     ib_quota[i] = ib_quota_nrt;
> +                             }
> +                     }
> +             } else {
> +                     ab_quota[0] = div_u64(ab_quota_rt + ab_quota_nrt,
> +                                     total_axi_port_cnt);
> +                     ib_quota[0] = ib_quota_rt + ib_quota_nrt;
> +
> +                     for (i = 1; i < total_axi_port_cnt; i++) {
> +                             ab_quota[i] = ab_quota[0];
> +                             ib_quota[i] = ib_quota[0];
> +                     }
> +             }
> +
> +             new_uc_idx = (pdbus->curr_bw_uc_idx %
> +                     (bw_table->num_usecases - 1)) + 1;
> +
> +             for (i = 0; i < total_axi_port_cnt; i++) {
> +                     vect = &bw_table->usecase[new_uc_idx].vectors[i];
> +                     vect->ab = ab_quota[i];
> +                     vect->ib = ib_quota[i];
> +
> +                     pr_debug(
> +                             "%s uc_idx=%d %s path idx=%d ab=%llu ib=%llu\n",
> +                             bw_table->name,
> +                             new_uc_idx, (i < rt_axi_port_cnt) ? "rt" : "nrt"
> +                             , i, vect->ab, vect->ib);
> +             }
> +     }
> +     pdbus->curr_bw_uc_idx = new_uc_idx;
> +     pdbus->ao_bw_uc_idx = new_uc_idx;
> +
> +     DPU_ATRACE_BEGIN("msm_bus_scale_req");
> +     rc = msm_bus_scale_client_update_request(pdbus->data_bus_hdl,
> +                     new_uc_idx);
> +     DPU_ATRACE_END("msm_bus_scale_req");
> +
> +     return rc;
> +}
> +
> +int dpu_power_data_bus_set_quota(struct dpu_power_handle *phandle,
> +             struct dpu_power_client *pclient,
> +             int bus_client, u32 bus_id,
> +             u64 ab_quota, u64 ib_quota)
> +{
> +     int rc = 0;
> +     int i;
> +     u64 total_ab_rt = 0, total_ib_rt = 0;
> +     u64 total_ab_nrt = 0, total_ib_nrt = 0;
> +     struct dpu_power_client *client;
> +
> +     if (!phandle || !pclient ||
> +                     bus_client >= DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX ||
> +                     bus_id >= DPU_POWER_HANDLE_DBUS_ID_MAX) {
> +             pr_err("invalid parameters\n");
> +             return -EINVAL;
> +     }
> +
> +     mutex_lock(&phandle->phandle_lock);
> +
> +     pclient->ab[bus_client] = ab_quota;
> +     pclient->ib[bus_client] = ib_quota;
> +     trace_dpu_perf_update_bus(bus_client, ab_quota, ib_quota);
> +
> +     list_for_each_entry(client, &phandle->power_client_clist, list) {
> +             for (i = 0; i < DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX; i++) {
> +                     if (i == DPU_POWER_HANDLE_DATA_BUS_CLIENT_NRT) {
> +                             total_ab_nrt += client->ab[i];
> +                             total_ib_nrt += client->ib[i];
> +                     } else {
> +                             total_ab_rt += client->ab[i];
> +                             total_ib_rt = max(total_ib_rt, client->ib[i]);
> +                     }
> +             }
> +     }
> +
> +     if (phandle->data_bus_handle[bus_id].data_bus_hdl)
> +             rc = _dpu_power_data_bus_set_quota(
> +                     &phandle->data_bus_handle[bus_id],
> +                     total_ab_rt, total_ab_nrt,
> +                     total_ib_rt, total_ib_nrt);
> +
> +     mutex_unlock(&phandle->phandle_lock);
> +
> +     return rc;
> +}
> +
> +static void dpu_power_data_bus_unregister(
> +             struct dpu_power_data_bus_handle *pdbus)
> +{
> +     if (pdbus->data_bus_hdl) {
> +             msm_bus_scale_unregister_client(pdbus->data_bus_hdl);
> +             pdbus->data_bus_hdl = 0;
> +     }
> +}
> +
> +static int dpu_power_data_bus_parse(struct platform_device *pdev,
> +     struct dpu_power_data_bus_handle *pdbus, const char *name)
> +{
> +     struct device_node *node;
> +     int rc = 0;
> +     int paths;
> +
> +     pdbus->bus_channels = 1;
> +     rc = of_property_read_u32(pdev->dev.of_node,
> +             "qcom,dpu-dram-channels", &pdbus->bus_channels);
> +     if (rc) {
> +             pr_debug("number of channels property not specified\n");
> +             rc = 0;
> +     }
> +
> +     pdbus->nrt_axi_port_cnt = 0;
> +     rc = of_property_read_u32(pdev->dev.of_node,
> +                     "qcom,dpu-num-nrt-paths",
> +                     &pdbus->nrt_axi_port_cnt);
> +     if (rc) {
> +             pr_debug("number of axi port property not specified\n");
> +             rc = 0;
> +     }
> +
> +     node = of_get_child_by_name(pdev->dev.of_node, name);
> +     if (node) {
> +             rc = of_property_read_u32(node,
> +                             "qcom,msm-bus,num-paths", &paths);
> +             if (rc) {
> +                     pr_err("Error. qcom,msm-bus,num-paths not found\n");
> +                     return rc;
> +             }
> +             pdbus->axi_port_cnt = paths;
> +
> +             pdbus->data_bus_scale_table =
> +                             msm_bus_pdata_from_node(pdev, node);
> +             if (IS_ERR_OR_NULL(pdbus->data_bus_scale_table)) {
> +                     pr_err("reg bus handle parsing failed\n");
> +                     rc = PTR_ERR(pdbus->data_bus_scale_table);
> +                     if (!pdbus->data_bus_scale_table)
> +                             rc = -EINVAL;
> +                     goto end;
> +             }
> +             pdbus->data_bus_hdl = msm_bus_scale_register_client(
> +                             pdbus->data_bus_scale_table);
> +             if (!pdbus->data_bus_hdl) {
> +                     pr_err("data_bus_client register failed\n");
> +                     rc = -EINVAL;
> +                     goto end;
> +             }
> +             pr_debug("register %s data_bus_hdl=%x\n", name,
> +                             pdbus->data_bus_hdl);
> +     }
> +
> +end:
> +     return rc;
> +}
> +
> +static int dpu_power_reg_bus_parse(struct platform_device *pdev,
> +     struct dpu_power_handle *phandle)
> +{
> +     struct device_node *node;
> +     struct msm_bus_scale_pdata *bus_scale_table;
> +     int rc = 0;
> +
> +     node = of_get_child_by_name(pdev->dev.of_node, "qcom,dpu-reg-bus");
> +     if (node) {
> +             bus_scale_table = msm_bus_pdata_from_node(pdev, node);
> +             if (IS_ERR_OR_NULL(bus_scale_table)) {
> +                     pr_err("reg bus handle parsing failed\n");
> +                     rc = PTR_ERR(bus_scale_table);
> +                     if (!bus_scale_table)
> +                             rc = -EINVAL;
> +                     goto end;
> +             }
> +             phandle->reg_bus_hdl = msm_bus_scale_register_client(
> +                           bus_scale_table);
> +             if (!phandle->reg_bus_hdl) {
> +                     pr_err("reg_bus_client register failed\n");
> +                     rc = -EINVAL;
> +                     goto end;
> +             }
> +             pr_debug("register reg_bus_hdl=%x\n", phandle->reg_bus_hdl);
> +     }
> +
> +end:
> +     return rc;
> +}
> +
> +static void dpu_power_reg_bus_unregister(u32 reg_bus_hdl)
> +{
> +     if (reg_bus_hdl)
> +             msm_bus_scale_unregister_client(reg_bus_hdl);
> +}
> +
> +int dpu_power_data_bus_state_update(struct dpu_power_handle *phandle,
> +                                                     bool enable)
> +{
> +     int i;
> +
> +     if (!phandle) {
> +             pr_err("invalid param\n");
> +             return -EINVAL;
> +     }
> +
> +     for (i = DPU_POWER_HANDLE_DBUS_ID_MNOC;
> +                     i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++)
> +             phandle->data_bus_handle[i].enable = enable;
> +
> +     return 0;
> +}
> +
> +static int dpu_power_data_bus_update(struct dpu_power_data_bus_handle *pdbus,
> +                                                     bool enable)
> +{
> +     int rc = 0;
> +
> +     pdbus->enable = enable;
> +
> +     if (pdbus->data_bus_hdl)
> +             rc = _dpu_power_data_bus_set_quota(pdbus, pdbus->ab_rt,
> +                             pdbus->ab_nrt, pdbus->ib_rt, pdbus->ib_nrt);
> +
> +     if (rc)
> +             pr_err("failed to set data bus vote rc=%d enable:%d\n",
> +                                                     rc, enable);
> +
> +     return rc;
> +}
> +
> +static int dpu_power_reg_bus_update(u32 reg_bus_hdl, u32 usecase_ndx)
> +{
> +     int rc = 0;
> +
> +     if (reg_bus_hdl)
> +             rc = msm_bus_scale_client_update_request(reg_bus_hdl,
> +                                                             usecase_ndx);
> +     if (rc)
> +             pr_err("failed to set reg bus vote rc=%d\n", rc);
> +
> +     return rc;
> +}
> +#else
> +static int dpu_power_data_bus_parse(struct platform_device *pdev,
> +             struct dpu_power_data_bus_handle *pdbus, const char *name)
> +{
> +     return 0;
> +}
> +
> +static void dpu_power_data_bus_unregister(
> +             struct dpu_power_data_bus_handle *pdbus)
> +{
> +}
> +
> +int dpu_power_data_bus_set_quota(struct dpu_power_handle *phandle,
> +             struct dpu_power_client *pclient,
> +             int bus_client, u32 bus_id,
> +             u64 ab_quota, u64 ib_quota)
> +{
> +     return 0;
> +}
> +
> +static int dpu_power_reg_bus_parse(struct platform_device *pdev,
> +     struct dpu_power_handle *phandle)
> +{
> +     return 0;
> +}
> +
> +static void dpu_power_reg_bus_unregister(u32 reg_bus_hdl)
> +{
> +}
> +
> +static int dpu_power_reg_bus_update(u32 reg_bus_hdl, u32 usecase_ndx)
> +{
> +     return 0;
> +}
> +
> +static int dpu_power_data_bus_update(struct dpu_power_data_bus_handle *pdbus,
> +                                                     bool enable)
> +{
> +     return 0;
> +}
> +
> +int dpu_power_data_bus_state_update(struct dpu_power_handle *phandle,
> +                                                     bool enable)
> +{
> +     return 0;
> +}
> +#endif
> +
> +int dpu_power_resource_init(struct platform_device *pdev,
> +     struct dpu_power_handle *phandle)
> +{
> +     int rc = 0, i;
> +
> +     if (!phandle || !pdev) {
> +             pr_err("invalid input param\n");
> +             return -EINVAL;
> +     }
> +
> +     phandle->dev = &pdev->dev;
> +
> +     rc = dpu_power_reg_bus_parse(pdev, phandle);
> +     if (rc) {
> +             pr_err("register bus parse failed rc=%d\n", rc);
> +             return rc;
> +     }
> +
> +     for (i = DPU_POWER_HANDLE_DBUS_ID_MNOC;
> +                     i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++) {
> +             rc = dpu_power_data_bus_parse(pdev,
> +                             &phandle->data_bus_handle[i],
> +                             data_bus_name[i]);
> +             if (rc) {
> +                     pr_err("register data bus parse failed id=%d rc=%d\n",
> +                                     i, rc);
> +                     goto data_bus_err;
> +             }
> +     }
> +
> +     INIT_LIST_HEAD(&phandle->power_client_clist);
> +     INIT_LIST_HEAD(&phandle->event_list);
> +
> +     mutex_init(&phandle->phandle_lock);
> +
> +     return rc;
> +
> +data_bus_err:
> +     for (i--; i >= 0; i--)
> +             dpu_power_data_bus_unregister(&phandle->data_bus_handle[i]);
> +     dpu_power_reg_bus_unregister(phandle->reg_bus_hdl);
> +     return rc;
> +}
> +
> +void dpu_power_resource_deinit(struct platform_device *pdev,
> +     struct dpu_power_handle *phandle)
> +{
> +     struct dpu_power_client *curr_client, *next_client;
> +     struct dpu_power_event *curr_event, *next_event;
> +     int i;
> +
> +     if (!phandle || !pdev) {
> +             pr_err("invalid input param\n");
> +             return;
> +     }
> +
> +     mutex_lock(&phandle->phandle_lock);
> +     list_for_each_entry_safe(curr_client, next_client,
> +                     &phandle->power_client_clist, list) {
> +             pr_err("cliend:%s-%d still registered with refcount:%d\n",
> +                             curr_client->name, curr_client->id,
> +                             curr_client->refcount);
> +             curr_client->active = false;
> +             list_del(&curr_client->list);
> +     }
> +
> +     list_for_each_entry_safe(curr_event, next_event,
> +                     &phandle->event_list, list) {
> +             pr_err("event:%d, client:%s still registered\n",
> +                             curr_event->event_type,
> +                             curr_event->client_name);
> +             curr_event->active = false;
> +             list_del(&curr_event->list);
> +     }
> +     mutex_unlock(&phandle->phandle_lock);
> +
> +     for (i = 0; i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++)
> +             dpu_power_data_bus_unregister(&phandle->data_bus_handle[i]);
> +
> +     dpu_power_reg_bus_unregister(phandle->reg_bus_hdl);
> +}
> +
> +int dpu_power_resource_enable(struct dpu_power_handle *phandle,
> +     struct dpu_power_client *pclient, bool enable)
> +{
> +     int rc = 0, i;
> +     bool changed = false;
> +     u32 max_usecase_ndx = VOTE_INDEX_DISABLE, prev_usecase_ndx;
> +     struct dpu_power_client *client;
> +
> +     if (!phandle || !pclient) {
> +             pr_err("invalid input argument\n");
> +             return -EINVAL;
> +     }
> +
> +     mutex_lock(&phandle->phandle_lock);
> +     if (enable)
> +             pclient->refcount++;
> +     else if (pclient->refcount)
> +             pclient->refcount--;
> +
> +     if (pclient->refcount)
> +             pclient->usecase_ndx = VOTE_INDEX_LOW;
> +     else
> +             pclient->usecase_ndx = VOTE_INDEX_DISABLE;
> +
> +     list_for_each_entry(client, &phandle->power_client_clist, list) {
> +             if (client->usecase_ndx < VOTE_INDEX_MAX &&
> +                 client->usecase_ndx > max_usecase_ndx)
> +                     max_usecase_ndx = client->usecase_ndx;
> +     }
> +
> +     if (phandle->current_usecase_ndx != max_usecase_ndx) {
> +             changed = true;
> +             prev_usecase_ndx = phandle->current_usecase_ndx;
> +             phandle->current_usecase_ndx = max_usecase_ndx;
> +     }
> +
> +     pr_debug("%pS: changed=%d current idx=%d request client %s id:%u 
> enable:%d refcount:%d\n",
> +             __builtin_return_address(0), changed, max_usecase_ndx,
> +             pclient->name, pclient->id, enable, pclient->refcount);
> +
> +     if (!changed)
> +             goto end;
> +
> +     if (enable) {
> +             dpu_power_event_trigger_locked(phandle,
> +                             DPU_POWER_EVENT_PRE_ENABLE);
> +
> +             for (i = 0; i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++) {
> +                     rc = dpu_power_data_bus_update(
> +                                     &phandle->data_bus_handle[i], enable);
> +                     if (rc) {
> +                             pr_err("failed to set data bus vote id=%d 
> rc=%d\n",
> +                                             i, rc);
> +                             goto data_bus_hdl_err;
> +                     }
> +             }
> +
> +             rc = dpu_power_reg_bus_update(phandle->reg_bus_hdl,
> +                                                     max_usecase_ndx);
> +             if (rc) {
> +                     pr_err("failed to set reg bus vote rc=%d\n", rc);
> +                     goto reg_bus_hdl_err;
> +             }
> +
> +             dpu_power_event_trigger_locked(phandle,
> +                             DPU_POWER_EVENT_POST_ENABLE);
> +
> +     } else {
> +             dpu_power_event_trigger_locked(phandle,
> +                             DPU_POWER_EVENT_PRE_DISABLE);
> +
> +             dpu_power_reg_bus_update(phandle->reg_bus_hdl,
> +                                                     max_usecase_ndx);
> +
> +             for (i = 0 ; i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++)
> +                     dpu_power_data_bus_update(&phandle->data_bus_handle[i],
> +                                     enable);
> +
> +             dpu_power_event_trigger_locked(phandle,
> +                             DPU_POWER_EVENT_POST_DISABLE);
> +     }
> +
> +end:
> +     mutex_unlock(&phandle->phandle_lock);
> +     return rc;
> +
> +reg_bus_hdl_err:
> +     for (i = 0 ; i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++)
> +             dpu_power_data_bus_update(&phandle->data_bus_handle[i], 0);
> +data_bus_hdl_err:
> +     phandle->current_usecase_ndx = prev_usecase_ndx;
> +     mutex_unlock(&phandle->phandle_lock);
> +     return rc;
> +}
> +
> +struct dpu_power_event *dpu_power_handle_register_event(
> +             struct dpu_power_handle *phandle,
> +             u32 event_type, void (*cb_fnc)(u32 event_type, void *usr),
> +             void *usr, char *client_name)
> +{
> +     struct dpu_power_event *event;
> +
> +     if (!phandle) {
> +             pr_err("invalid power handle\n");
> +             return ERR_PTR(-EINVAL);
> +     } else if (!cb_fnc || !event_type) {
> +             pr_err("no callback fnc or event type\n");
> +             return ERR_PTR(-EINVAL);
> +     }
> +
> +     event = kzalloc(sizeof(struct dpu_power_event), GFP_KERNEL);
> +     if (!event)
> +             return ERR_PTR(-ENOMEM);
> +
> +     event->event_type = event_type;
> +     event->cb_fnc = cb_fnc;
> +     event->usr = usr;
> +     strlcpy(event->client_name, client_name, MAX_CLIENT_NAME_LEN);
> +     event->active = true;
> +
> +     mutex_lock(&phandle->phandle_lock);
> +     list_add(&event->list, &phandle->event_list);
> +     mutex_unlock(&phandle->phandle_lock);
> +
> +     return event;
> +}
> +
> +void dpu_power_handle_unregister_event(
> +             struct dpu_power_handle *phandle,
> +             struct dpu_power_event *event)
> +{
> +     if (!phandle || !event) {
> +             pr_err("invalid phandle or event\n");
> +     } else if (!event->active) {
> +             pr_err("power handle deinit already done\n");
> +             kfree(event);
> +     } else {
> +             mutex_lock(&phandle->phandle_lock);
> +             list_del_init(&event->list);
> +             mutex_unlock(&phandle->phandle_lock);
> +             kfree(event);
> +     }
> +}
> diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h 
> b/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h
> new file mode 100644
> index 0000000..9a6d4b9
> --- /dev/null
> +++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h
> @@ -0,0 +1,288 @@
> +/* Copyright (c) 2016-2018, The Linux Foundation. All rights reserved.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 and
> + * only version 2 as published by the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + */
> +
> +#ifndef _DPU_POWER_HANDLE_H_
> +#define _DPU_POWER_HANDLE_H_
> +
> +#define MAX_CLIENT_NAME_LEN 128
> +
> +#define DPU_POWER_HANDLE_ENABLE_BUS_AB_QUOTA 0
> +#define DPU_POWER_HANDLE_DISABLE_BUS_AB_QUOTA        0
> +#define DPU_POWER_HANDLE_ENABLE_BUS_IB_QUOTA 1600000000
> +#define DPU_POWER_HANDLE_DISABLE_BUS_IB_QUOTA        0
> +
> +#include <linux/dpu_io_util.h>
> +
> +/* event will be triggered before power handler disable */
> +#define DPU_POWER_EVENT_PRE_DISABLE  0x1
> +
> +/* event will be triggered after power handler disable */
> +#define DPU_POWER_EVENT_POST_DISABLE 0x2
> +
> +/* event will be triggered before power handler enable */
> +#define DPU_POWER_EVENT_PRE_ENABLE   0x4
> +
> +/* event will be triggered after power handler enable */
> +#define DPU_POWER_EVENT_POST_ENABLE  0x8
> +
> +/**
> + * mdss_bus_vote_type: register bus vote type
> + * VOTE_INDEX_DISABLE: removes the client vote
> + * VOTE_INDEX_LOW: keeps the lowest vote for register bus
> + * VOTE_INDEX_MAX: invalid
> + */
> +enum mdss_bus_vote_type {
> +     VOTE_INDEX_DISABLE,
> +     VOTE_INDEX_LOW,
> +     VOTE_INDEX_MAX,
> +};
> +
> +/**
> + * enum dpu_power_handle_data_bus_client - type of axi bus clients
> + * @DPU_POWER_HANDLE_DATA_BUS_CLIENT_RT: core real-time bus client
> + * @DPU_POWER_HANDLE_DATA_BUS_CLIENT_NRT: core non-real-time bus client
> + * @DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX: maximum number of bus client type
> + */
> +enum dpu_power_handle_data_bus_client {
> +     DPU_POWER_HANDLE_DATA_BUS_CLIENT_RT,
> +     DPU_POWER_HANDLE_DATA_BUS_CLIENT_NRT,
> +     DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX
> +};
> +
> +/**
> + * enum DPU_POWER_HANDLE_DBUS_ID - data bus identifier
> + * @DPU_POWER_HANDLE_DBUS_ID_MNOC: DPU/MNOC data bus
> + * @DPU_POWER_HANDLE_DBUS_ID_LLCC: MNOC/LLCC data bus
> + * @DPU_POWER_HANDLE_DBUS_ID_EBI: LLCC/EBI data bus
> + */
> +enum DPU_POWER_HANDLE_DBUS_ID {
> +     DPU_POWER_HANDLE_DBUS_ID_MNOC,
> +     DPU_POWER_HANDLE_DBUS_ID_LLCC,
> +     DPU_POWER_HANDLE_DBUS_ID_EBI,
> +     DPU_POWER_HANDLE_DBUS_ID_MAX,
> +};
> +
> +/**
> + * struct dpu_power_client: stores the power client for dpu driver
> + * @name:    name of the client
> + * @usecase_ndx: current regs bus vote type
> + * @refcount:        current refcount if multiple modules are using same
> + *              same client for enable/disable. Power module will
> + *              aggregate the refcount and vote accordingly for this
> + *              client.
> + * @id:              assigned during create. helps for debugging.
> + * @list:    list to attach power handle master list
> + * @ab:         arbitrated bandwidth for each bus client
> + * @ib:         instantaneous bandwidth for each bus client
> + * @active:  inidcates the state of dpu power handle
> + */
> +struct dpu_power_client {
> +     char name[MAX_CLIENT_NAME_LEN];
> +     short usecase_ndx;
> +     short refcount;
> +     u32 id;
> +     struct list_head list;
> +     u64 ab[DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX];
> +     u64 ib[DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX];
> +     bool active;
> +};
> +
> +/**
> + * struct dpu_power_data_handle: power handle struct for data bus
> + * @data_bus_scale_table: pointer to bus scaling table
> + * @data_bus_hdl: current data bus handle
> + * @axi_port_cnt: number of rt axi ports
> + * @nrt_axi_port_cnt: number of nrt axi ports
> + * @bus_channels: number of memory bus channels
> + * @curr_bw_uc_idx: current use case index of data bus
> + * @ao_bw_uc_idx: active only use case index of data bus
> + * @ab_rt: realtime ab quota
> + * @ib_rt: realtime ib quota
> + * @ab_nrt: non-realtime ab quota
> + * @ib_nrt: non-realtime ib quota
> + * @enable: true if bus is enabled
> + */
> +struct dpu_power_data_bus_handle {
> +     struct msm_bus_scale_pdata *data_bus_scale_table;
> +     u32 data_bus_hdl;
> +     u32 axi_port_cnt;
> +     u32 nrt_axi_port_cnt;
> +     u32 bus_channels;
> +     u32 curr_bw_uc_idx;
> +     u32 ao_bw_uc_idx;
> +     u64 ab_rt;
> +     u64 ib_rt;
> +     u64 ab_nrt;
> +     u64 ib_nrt;
> +     bool enable;
> +};
> +
> +/*
> + * struct dpu_power_event - local event registration structure
> + * @client_name: name of the client registering
> + * @cb_fnc: pointer to desired callback function
> + * @usr: user pointer to pass to callback event trigger
> + * @event: refer to DPU_POWER_HANDLE_EVENT_*
> + * @list: list to attach event master list
> + * @active: indicates the state of dpu power handle
> + */
> +struct dpu_power_event {
> +     char client_name[MAX_CLIENT_NAME_LEN];
> +     void (*cb_fnc)(u32 event_type, void *usr);
> +     void *usr;
> +     u32 event_type;
> +     struct list_head list;
> +     bool active;
> +};
> +
> +/**
> + * struct dpu_power_handle: power handle main struct
> + * @client_clist: master list to store all clients
> + * @phandle_lock: lock to synchronize the enable/disable
> + * @dev: pointer to device structure
> + * @usecase_ndx: current usecase index
> + * @reg_bus_hdl: current register bus handle
> + * @data_bus_handle: context structure for data bus control
> + * @event_list: current power handle event list
> + */
> +struct dpu_power_handle {
> +     struct list_head power_client_clist;
> +     struct mutex phandle_lock;
> +     struct device *dev;
> +     u32 current_usecase_ndx;
> +     u32 reg_bus_hdl;
> +     struct dpu_power_data_bus_handle data_bus_handle
> +             [DPU_POWER_HANDLE_DBUS_ID_MAX];
> +     struct list_head event_list;
> +};
> +
> +/**
> + * dpu_power_resource_init() - initializes the dpu power handle
> + * @pdev:   platform device to search the power resources
> + * @pdata:  power handle to store the power resources
> + *
> + * Return: error code.
> + */
> +int dpu_power_resource_init(struct platform_device *pdev,
> +     struct dpu_power_handle *pdata);
> +
> +/**
> + * dpu_power_resource_deinit() - release the dpu power handle
> + * @pdev:   platform device for power resources
> + * @pdata:  power handle containing the resources
> + *
> + * Return: error code.
> + */
> +void dpu_power_resource_deinit(struct platform_device *pdev,
> +     struct dpu_power_handle *pdata);
> +
> +/**
> + * dpu_power_client_create() - create the client on power handle
> + * @pdata:  power handle containing the resources
> + * @client_name: new client name for registration
> + *
> + * Return: error code.
> + */
> +struct dpu_power_client *dpu_power_client_create(struct dpu_power_handle 
> *pdata,
> +     char *client_name);
> +
> +/**
> + * dpu_power_client_destroy() - destroy the client on power handle
> + * @pdata:  power handle containing the resources
> + * @client_name: new client name for registration
> + *
> + * Return: none
> + */
> +void dpu_power_client_destroy(struct dpu_power_handle *phandle,
> +     struct dpu_power_client *client);
> +
> +/**
> + * dpu_power_resource_enable() - enable/disable the power resources
> + * @pdata:  power handle containing the resources
> + * @client: client information to enable/disable its vote
> + * @enable: boolean request for enable/disable
> + *
> + * Return: error code.
> + */
> +int dpu_power_resource_enable(struct dpu_power_handle *pdata,
> +     struct dpu_power_client *pclient, bool enable);
> +
> +/**
> + * dpu_power_data_bus_state_update() - update data bus state
> + * @pdata:  power handle containing the resources
> + * @enable: take enable vs disable path
> + *
> + * Return: error code.
> + */
> +int dpu_power_data_bus_state_update(struct dpu_power_handle *phandle,
> +                                                     bool enable);
> +
> +/**
> + * dpu_power_data_bus_set_quota() - set data bus quota for power client
> + * @phandle:  power handle containing the resources
> + * @client: client information to set quota
> + * @bus_client: real-time or non-real-time bus client
> + * @bus_id: identifier of data bus, see DPU_POWER_HANDLE_DBUS_ID
> + * @ab_quota: arbitrated bus bandwidth
> + * @ib_quota: instantaneous bus bandwidth
> + *
> + * Return: zero if success, or error code otherwise
> + */
> +int dpu_power_data_bus_set_quota(struct dpu_power_handle *phandle,
> +             struct dpu_power_client *pclient,
> +             int bus_client, u32 bus_id,
> +             u64 ab_quota, u64 ib_quota);
> +
> +/**
> + * dpu_power_data_bus_bandwidth_ctrl() - control data bus bandwidth enable
> + * @phandle:  power handle containing the resources
> + * @client: client information to bandwidth control
> + * @enable: true to enable bandwidth for data base
> + *
> + * Return: none
> + */
> +void dpu_power_data_bus_bandwidth_ctrl(struct dpu_power_handle *phandle,
> +             struct dpu_power_client *pclient, int enable);
> +
> +/**
> + * dpu_power_handle_register_event - register a callback function for an 
> event.
> + *   Clients can register for multiple events with a single register.
> + *   Any block with access to phandle can register for the event
> + *   notification.
> + * @phandle: power handle containing the resources
> + * @event_type:      event type to register; refer DPU_POWER_HANDLE_EVENT_*
> + * @cb_fnc:  pointer to desired callback function
> + * @usr:     user pointer to pass to callback on event trigger
> + *
> + * Return:   event pointer if success, or error code otherwise
> + */
> +struct dpu_power_event *dpu_power_handle_register_event(
> +             struct dpu_power_handle *phandle,
> +             u32 event_type, void (*cb_fnc)(u32 event_type, void *usr),
> +             void *usr, char *client_name);
> +/**
> + * dpu_power_handle_unregister_event - unregister callback for event(s)
> + * @phandle: power handle containing the resources
> + * @event:   event pointer returned after power handle register
> + */
> +void dpu_power_handle_unregister_event(struct dpu_power_handle *phandle,
> +             struct dpu_power_event *event);
> +
> +/**
> + * dpu_power_handle_get_dbus_name - get name of given data bus identifier
> + * @bus_id:  data bus identifier
> + * Return:   Pointer to name string if success; NULL otherwise
> + */
> +const char *dpu_power_handle_get_dbus_name(u32 bus_id);
> +
> +#endif /* _DPU_POWER_HANDLE_H_ */
> diff --git a/drivers/gpu/drm/msm/dpu_power_handle.c 
> b/drivers/gpu/drm/msm/dpu_power_handle.c
> deleted file mode 100644
> index 77be106..0000000
> --- a/drivers/gpu/drm/msm/dpu_power_handle.c
> +++ /dev/null
> @@ -1,693 +0,0 @@
> -/* Copyright (c) 2014-2018, The Linux Foundation. All rights reserved.
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License version 2 and
> - * only version 2 as published by the Free Software Foundation.
> - *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> - * GNU General Public License for more details.
> - *
> - */
> -
> -#define pr_fmt(fmt)  "[drm:%s:%d]: " fmt, __func__, __LINE__
> -
> -#include <linux/kernel.h>
> -#include <linux/of.h>
> -#include <linux/string.h>
> -#include <linux/of_address.h>
> -#include <linux/slab.h>
> -#include <linux/mutex.h>
> -#include <linux/of_platform.h>
> -#ifdef CONFIG_QCOM_BUS_SCALING
> -#include <linux/msm-bus.h>
> -#include <linux/msm-bus-board.h>
> -#endif
> -#include <linux/dpu_io_util.h>
> -
> -#include "dpu_power_handle.h"
> -#include "dpu_trace.h"
> -
> -static const char *data_bus_name[DPU_POWER_HANDLE_DBUS_ID_MAX] = {
> -     [DPU_POWER_HANDLE_DBUS_ID_MNOC] = "qcom,dpu-data-bus",
> -     [DPU_POWER_HANDLE_DBUS_ID_LLCC] = "qcom,dpu-llcc-bus",
> -     [DPU_POWER_HANDLE_DBUS_ID_EBI] = "qcom,dpu-ebi-bus",
> -};
> -
> -const char *dpu_power_handle_get_dbus_name(u32 bus_id)
> -{
> -     if (bus_id < DPU_POWER_HANDLE_DBUS_ID_MAX)
> -             return data_bus_name[bus_id];
> -
> -     return NULL;
> -}
> -
> -static void dpu_power_event_trigger_locked(struct dpu_power_handle *phandle,
> -             u32 event_type)
> -{
> -     struct dpu_power_event *event;
> -
> -     list_for_each_entry(event, &phandle->event_list, list) {
> -             if (event->event_type & event_type)
> -                     event->cb_fnc(event_type, event->usr);
> -     }
> -}
> -
> -struct dpu_power_client *dpu_power_client_create(
> -     struct dpu_power_handle *phandle, char *client_name)
> -{
> -     struct dpu_power_client *client;
> -     static u32 id;
> -
> -     if (!client_name || !phandle) {
> -             pr_err("client name is null or invalid power data\n");
> -             return ERR_PTR(-EINVAL);
> -     }
> -
> -     client = kzalloc(sizeof(struct dpu_power_client), GFP_KERNEL);
> -     if (!client)
> -             return ERR_PTR(-ENOMEM);
> -
> -     mutex_lock(&phandle->phandle_lock);
> -     strlcpy(client->name, client_name, MAX_CLIENT_NAME_LEN);
> -     client->usecase_ndx = VOTE_INDEX_DISABLE;
> -     client->id = id;
> -     client->active = true;
> -     pr_debug("client %s created:%pK id :%d\n", client_name,
> -             client, id);
> -     id++;
> -     list_add(&client->list, &phandle->power_client_clist);
> -     mutex_unlock(&phandle->phandle_lock);
> -
> -     return client;
> -}
> -
> -void dpu_power_client_destroy(struct dpu_power_handle *phandle,
> -     struct dpu_power_client *client)
> -{
> -     if (!client  || !phandle) {
> -             pr_err("reg bus vote: invalid client handle\n");
> -     } else if (!client->active) {
> -             pr_err("dpu power deinit already done\n");
> -             kfree(client);
> -     } else {
> -             pr_debug("bus vote client %s destroyed:%pK id:%u\n",
> -                     client->name, client, client->id);
> -             mutex_lock(&phandle->phandle_lock);
> -             list_del_init(&client->list);
> -             mutex_unlock(&phandle->phandle_lock);
> -             kfree(client);
> -     }
> -}
> -
> -#ifdef CONFIG_QCOM_BUS_SCALING
> -
> -#define MAX_AXI_PORT_COUNT 3
> -
> -static int _dpu_power_data_bus_set_quota(
> -             struct dpu_power_data_bus_handle *pdbus,
> -             u64 ab_quota_rt, u64 ab_quota_nrt,
> -             u64 ib_quota_rt, u64 ib_quota_nrt)
> -{
> -     int new_uc_idx;
> -     u64 ab_quota[MAX_AXI_PORT_COUNT] = {0, 0};
> -     u64 ib_quota[MAX_AXI_PORT_COUNT] = {0, 0};
> -     int rc;
> -
> -     if (pdbus->data_bus_hdl < 1) {
> -             pr_err("invalid bus handle %d\n", pdbus->data_bus_hdl);
> -             return -EINVAL;
> -     }
> -
> -     pdbus->ab_rt = ab_quota_rt;
> -     pdbus->ib_rt = ib_quota_rt;
> -     pdbus->ab_nrt = ab_quota_nrt;
> -     pdbus->ib_nrt = ib_quota_nrt;
> -
> -     if (pdbus->enable) {
> -             ab_quota_rt = max_t(u64, ab_quota_rt,
> -                             DPU_POWER_HANDLE_ENABLE_BUS_AB_QUOTA);
> -             ib_quota_rt = max_t(u64, ib_quota_rt,
> -                             DPU_POWER_HANDLE_ENABLE_BUS_IB_QUOTA);
> -             ab_quota_nrt = max_t(u64, ab_quota_nrt,
> -                             DPU_POWER_HANDLE_ENABLE_BUS_AB_QUOTA);
> -             ib_quota_nrt = max_t(u64, ib_quota_nrt,
> -                             DPU_POWER_HANDLE_ENABLE_BUS_IB_QUOTA);
> -     } else {
> -             ab_quota_rt = min_t(u64, ab_quota_rt,
> -                             DPU_POWER_HANDLE_DISABLE_BUS_AB_QUOTA);
> -             ib_quota_rt = min_t(u64, ib_quota_rt,
> -                             DPU_POWER_HANDLE_DISABLE_BUS_IB_QUOTA);
> -             ab_quota_nrt = min_t(u64, ab_quota_nrt,
> -                             DPU_POWER_HANDLE_DISABLE_BUS_AB_QUOTA);
> -             ib_quota_nrt = min_t(u64, ib_quota_nrt,
> -                             DPU_POWER_HANDLE_DISABLE_BUS_IB_QUOTA);
> -     }
> -
> -     if (!ab_quota_rt && !ab_quota_nrt && !ib_quota_rt && !ib_quota_nrt)  {
> -             new_uc_idx = 0;
> -     } else {
> -             int i;
> -             struct msm_bus_vectors *vect = NULL;
> -             struct msm_bus_scale_pdata *bw_table =
> -                     pdbus->data_bus_scale_table;
> -             u32 nrt_axi_port_cnt = pdbus->nrt_axi_port_cnt;
> -             u32 total_axi_port_cnt = pdbus->axi_port_cnt;
> -             u32 rt_axi_port_cnt = total_axi_port_cnt - nrt_axi_port_cnt;
> -
> -             if (!bw_table || !total_axi_port_cnt ||
> -                 total_axi_port_cnt > MAX_AXI_PORT_COUNT) {
> -                     pr_err("invalid input\n");
> -                     return -EINVAL;
> -             }
> -
> -             if (pdbus->bus_channels) {
> -                     ib_quota_rt = div_u64(ib_quota_rt,
> -                                             pdbus->bus_channels);
> -                     ib_quota_nrt = div_u64(ib_quota_nrt,
> -                                             pdbus->bus_channels);
> -             }
> -
> -             if (nrt_axi_port_cnt) {
> -
> -                     ab_quota_rt = div_u64(ab_quota_rt, rt_axi_port_cnt);
> -                     ab_quota_nrt = div_u64(ab_quota_nrt, nrt_axi_port_cnt);
> -
> -                     for (i = 0; i < total_axi_port_cnt; i++) {
> -                             if (i < rt_axi_port_cnt) {
> -                                     ab_quota[i] = ab_quota_rt;
> -                                     ib_quota[i] = ib_quota_rt;
> -                             } else {
> -                                     ab_quota[i] = ab_quota_nrt;
> -                                     ib_quota[i] = ib_quota_nrt;
> -                             }
> -                     }
> -             } else {
> -                     ab_quota[0] = div_u64(ab_quota_rt + ab_quota_nrt,
> -                                     total_axi_port_cnt);
> -                     ib_quota[0] = ib_quota_rt + ib_quota_nrt;
> -
> -                     for (i = 1; i < total_axi_port_cnt; i++) {
> -                             ab_quota[i] = ab_quota[0];
> -                             ib_quota[i] = ib_quota[0];
> -                     }
> -             }
> -
> -             new_uc_idx = (pdbus->curr_bw_uc_idx %
> -                     (bw_table->num_usecases - 1)) + 1;
> -
> -             for (i = 0; i < total_axi_port_cnt; i++) {
> -                     vect = &bw_table->usecase[new_uc_idx].vectors[i];
> -                     vect->ab = ab_quota[i];
> -                     vect->ib = ib_quota[i];
> -
> -                     pr_debug(
> -                             "%s uc_idx=%d %s path idx=%d ab=%llu ib=%llu\n",
> -                             bw_table->name,
> -                             new_uc_idx, (i < rt_axi_port_cnt) ? "rt" : "nrt"
> -                             , i, vect->ab, vect->ib);
> -             }
> -     }
> -     pdbus->curr_bw_uc_idx = new_uc_idx;
> -     pdbus->ao_bw_uc_idx = new_uc_idx;
> -
> -     DPU_ATRACE_BEGIN("msm_bus_scale_req");
> -     rc = msm_bus_scale_client_update_request(pdbus->data_bus_hdl,
> -                     new_uc_idx);
> -     DPU_ATRACE_END("msm_bus_scale_req");
> -
> -     return rc;
> -}
> -
> -int dpu_power_data_bus_set_quota(struct dpu_power_handle *phandle,
> -             struct dpu_power_client *pclient,
> -             int bus_client, u32 bus_id,
> -             u64 ab_quota, u64 ib_quota)
> -{
> -     int rc = 0;
> -     int i;
> -     u64 total_ab_rt = 0, total_ib_rt = 0;
> -     u64 total_ab_nrt = 0, total_ib_nrt = 0;
> -     struct dpu_power_client *client;
> -
> -     if (!phandle || !pclient ||
> -                     bus_client >= DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX ||
> -                     bus_id >= DPU_POWER_HANDLE_DBUS_ID_MAX) {
> -             pr_err("invalid parameters\n");
> -             return -EINVAL;
> -     }
> -
> -     mutex_lock(&phandle->phandle_lock);
> -
> -     pclient->ab[bus_client] = ab_quota;
> -     pclient->ib[bus_client] = ib_quota;
> -     trace_dpu_perf_update_bus(bus_client, ab_quota, ib_quota);
> -
> -     list_for_each_entry(client, &phandle->power_client_clist, list) {
> -             for (i = 0; i < DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX; i++) {
> -                     if (i == DPU_POWER_HANDLE_DATA_BUS_CLIENT_NRT) {
> -                             total_ab_nrt += client->ab[i];
> -                             total_ib_nrt += client->ib[i];
> -                     } else {
> -                             total_ab_rt += client->ab[i];
> -                             total_ib_rt = max(total_ib_rt, client->ib[i]);
> -                     }
> -             }
> -     }
> -
> -     if (phandle->data_bus_handle[bus_id].data_bus_hdl)
> -             rc = _dpu_power_data_bus_set_quota(
> -                     &phandle->data_bus_handle[bus_id],
> -                     total_ab_rt, total_ab_nrt,
> -                     total_ib_rt, total_ib_nrt);
> -
> -     mutex_unlock(&phandle->phandle_lock);
> -
> -     return rc;
> -}
> -
> -static void dpu_power_data_bus_unregister(
> -             struct dpu_power_data_bus_handle *pdbus)
> -{
> -     if (pdbus->data_bus_hdl) {
> -             msm_bus_scale_unregister_client(pdbus->data_bus_hdl);
> -             pdbus->data_bus_hdl = 0;
> -     }
> -}
> -
> -static int dpu_power_data_bus_parse(struct platform_device *pdev,
> -     struct dpu_power_data_bus_handle *pdbus, const char *name)
> -{
> -     struct device_node *node;
> -     int rc = 0;
> -     int paths;
> -
> -     pdbus->bus_channels = 1;
> -     rc = of_property_read_u32(pdev->dev.of_node,
> -             "qcom,dpu-dram-channels", &pdbus->bus_channels);
> -     if (rc) {
> -             pr_debug("number of channels property not specified\n");
> -             rc = 0;
> -     }
> -
> -     pdbus->nrt_axi_port_cnt = 0;
> -     rc = of_property_read_u32(pdev->dev.of_node,
> -                     "qcom,dpu-num-nrt-paths",
> -                     &pdbus->nrt_axi_port_cnt);
> -     if (rc) {
> -             pr_debug("number of axi port property not specified\n");
> -             rc = 0;
> -     }
> -
> -     node = of_get_child_by_name(pdev->dev.of_node, name);
> -     if (node) {
> -             rc = of_property_read_u32(node,
> -                             "qcom,msm-bus,num-paths", &paths);
> -             if (rc) {
> -                     pr_err("Error. qcom,msm-bus,num-paths not found\n");
> -                     return rc;
> -             }
> -             pdbus->axi_port_cnt = paths;
> -
> -             pdbus->data_bus_scale_table =
> -                             msm_bus_pdata_from_node(pdev, node);
> -             if (IS_ERR_OR_NULL(pdbus->data_bus_scale_table)) {
> -                     pr_err("reg bus handle parsing failed\n");
> -                     rc = PTR_ERR(pdbus->data_bus_scale_table);
> -                     if (!pdbus->data_bus_scale_table)
> -                             rc = -EINVAL;
> -                     goto end;
> -             }
> -             pdbus->data_bus_hdl = msm_bus_scale_register_client(
> -                             pdbus->data_bus_scale_table);
> -             if (!pdbus->data_bus_hdl) {
> -                     pr_err("data_bus_client register failed\n");
> -                     rc = -EINVAL;
> -                     goto end;
> -             }
> -             pr_debug("register %s data_bus_hdl=%x\n", name,
> -                             pdbus->data_bus_hdl);
> -     }
> -
> -end:
> -     return rc;
> -}
> -
> -static int dpu_power_reg_bus_parse(struct platform_device *pdev,
> -     struct dpu_power_handle *phandle)
> -{
> -     struct device_node *node;
> -     struct msm_bus_scale_pdata *bus_scale_table;
> -     int rc = 0;
> -
> -     node = of_get_child_by_name(pdev->dev.of_node, "qcom,dpu-reg-bus");
> -     if (node) {
> -             bus_scale_table = msm_bus_pdata_from_node(pdev, node);
> -             if (IS_ERR_OR_NULL(bus_scale_table)) {
> -                     pr_err("reg bus handle parsing failed\n");
> -                     rc = PTR_ERR(bus_scale_table);
> -                     if (!bus_scale_table)
> -                             rc = -EINVAL;
> -                     goto end;
> -             }
> -             phandle->reg_bus_hdl = msm_bus_scale_register_client(
> -                           bus_scale_table);
> -             if (!phandle->reg_bus_hdl) {
> -                     pr_err("reg_bus_client register failed\n");
> -                     rc = -EINVAL;
> -                     goto end;
> -             }
> -             pr_debug("register reg_bus_hdl=%x\n", phandle->reg_bus_hdl);
> -     }
> -
> -end:
> -     return rc;
> -}
> -
> -static void dpu_power_reg_bus_unregister(u32 reg_bus_hdl)
> -{
> -     if (reg_bus_hdl)
> -             msm_bus_scale_unregister_client(reg_bus_hdl);
> -}
> -
> -int dpu_power_data_bus_state_update(struct dpu_power_handle *phandle,
> -                                                     bool enable)
> -{
> -     int i;
> -
> -     if (!phandle) {
> -             pr_err("invalid param\n");
> -             return -EINVAL;
> -     }
> -
> -     for (i = DPU_POWER_HANDLE_DBUS_ID_MNOC;
> -                     i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++)
> -             phandle->data_bus_handle[i].enable = enable;
> -
> -     return 0;
> -}
> -
> -static int dpu_power_data_bus_update(struct dpu_power_data_bus_handle *pdbus,
> -                                                     bool enable)
> -{
> -     int rc = 0;
> -
> -     pdbus->enable = enable;
> -
> -     if (pdbus->data_bus_hdl)
> -             rc = _dpu_power_data_bus_set_quota(pdbus, pdbus->ab_rt,
> -                             pdbus->ab_nrt, pdbus->ib_rt, pdbus->ib_nrt);
> -
> -     if (rc)
> -             pr_err("failed to set data bus vote rc=%d enable:%d\n",
> -                                                     rc, enable);
> -
> -     return rc;
> -}
> -
> -static int dpu_power_reg_bus_update(u32 reg_bus_hdl, u32 usecase_ndx)
> -{
> -     int rc = 0;
> -
> -     if (reg_bus_hdl)
> -             rc = msm_bus_scale_client_update_request(reg_bus_hdl,
> -                                                             usecase_ndx);
> -     if (rc)
> -             pr_err("failed to set reg bus vote rc=%d\n", rc);
> -
> -     return rc;
> -}
> -#else
> -static int dpu_power_data_bus_parse(struct platform_device *pdev,
> -             struct dpu_power_data_bus_handle *pdbus, const char *name)
> -{
> -     return 0;
> -}
> -
> -static void dpu_power_data_bus_unregister(
> -             struct dpu_power_data_bus_handle *pdbus)
> -{
> -}
> -
> -int dpu_power_data_bus_set_quota(struct dpu_power_handle *phandle,
> -             struct dpu_power_client *pclient,
> -             int bus_client, u32 bus_id,
> -             u64 ab_quota, u64 ib_quota)
> -{
> -     return 0;
> -}
> -
> -static int dpu_power_reg_bus_parse(struct platform_device *pdev,
> -     struct dpu_power_handle *phandle)
> -{
> -     return 0;
> -}
> -
> -static void dpu_power_reg_bus_unregister(u32 reg_bus_hdl)
> -{
> -}
> -
> -static int dpu_power_reg_bus_update(u32 reg_bus_hdl, u32 usecase_ndx)
> -{
> -     return 0;
> -}
> -
> -static int dpu_power_data_bus_update(struct dpu_power_data_bus_handle *pdbus,
> -                                                     bool enable)
> -{
> -     return 0;
> -}
> -
> -int dpu_power_data_bus_state_update(struct dpu_power_handle *phandle,
> -                                                     bool enable)
> -{
> -     return 0;
> -}
> -#endif
> -
> -int dpu_power_resource_init(struct platform_device *pdev,
> -     struct dpu_power_handle *phandle)
> -{
> -     int rc = 0, i;
> -
> -     if (!phandle || !pdev) {
> -             pr_err("invalid input param\n");
> -             return -EINVAL;
> -     }
> -
> -     phandle->dev = &pdev->dev;
> -
> -     rc = dpu_power_reg_bus_parse(pdev, phandle);
> -     if (rc) {
> -             pr_err("register bus parse failed rc=%d\n", rc);
> -             return rc;
> -     }
> -
> -     for (i = DPU_POWER_HANDLE_DBUS_ID_MNOC;
> -                     i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++) {
> -             rc = dpu_power_data_bus_parse(pdev,
> -                             &phandle->data_bus_handle[i],
> -                             data_bus_name[i]);
> -             if (rc) {
> -                     pr_err("register data bus parse failed id=%d rc=%d\n",
> -                                     i, rc);
> -                     goto data_bus_err;
> -             }
> -     }
> -
> -     INIT_LIST_HEAD(&phandle->power_client_clist);
> -     INIT_LIST_HEAD(&phandle->event_list);
> -
> -     mutex_init(&phandle->phandle_lock);
> -
> -     return rc;
> -
> -data_bus_err:
> -     for (i--; i >= 0; i--)
> -             dpu_power_data_bus_unregister(&phandle->data_bus_handle[i]);
> -     dpu_power_reg_bus_unregister(phandle->reg_bus_hdl);
> -     return rc;
> -}
> -
> -void dpu_power_resource_deinit(struct platform_device *pdev,
> -     struct dpu_power_handle *phandle)
> -{
> -     struct dpu_power_client *curr_client, *next_client;
> -     struct dpu_power_event *curr_event, *next_event;
> -     int i;
> -
> -     if (!phandle || !pdev) {
> -             pr_err("invalid input param\n");
> -             return;
> -     }
> -
> -     mutex_lock(&phandle->phandle_lock);
> -     list_for_each_entry_safe(curr_client, next_client,
> -                     &phandle->power_client_clist, list) {
> -             pr_err("cliend:%s-%d still registered with refcount:%d\n",
> -                             curr_client->name, curr_client->id,
> -                             curr_client->refcount);
> -             curr_client->active = false;
> -             list_del(&curr_client->list);
> -     }
> -
> -     list_for_each_entry_safe(curr_event, next_event,
> -                     &phandle->event_list, list) {
> -             pr_err("event:%d, client:%s still registered\n",
> -                             curr_event->event_type,
> -                             curr_event->client_name);
> -             curr_event->active = false;
> -             list_del(&curr_event->list);
> -     }
> -     mutex_unlock(&phandle->phandle_lock);
> -
> -     for (i = 0; i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++)
> -             dpu_power_data_bus_unregister(&phandle->data_bus_handle[i]);
> -
> -     dpu_power_reg_bus_unregister(phandle->reg_bus_hdl);
> -}
> -
> -int dpu_power_resource_enable(struct dpu_power_handle *phandle,
> -     struct dpu_power_client *pclient, bool enable)
> -{
> -     int rc = 0, i;
> -     bool changed = false;
> -     u32 max_usecase_ndx = VOTE_INDEX_DISABLE, prev_usecase_ndx;
> -     struct dpu_power_client *client;
> -
> -     if (!phandle || !pclient) {
> -             pr_err("invalid input argument\n");
> -             return -EINVAL;
> -     }
> -
> -     mutex_lock(&phandle->phandle_lock);
> -     if (enable)
> -             pclient->refcount++;
> -     else if (pclient->refcount)
> -             pclient->refcount--;
> -
> -     if (pclient->refcount)
> -             pclient->usecase_ndx = VOTE_INDEX_LOW;
> -     else
> -             pclient->usecase_ndx = VOTE_INDEX_DISABLE;
> -
> -     list_for_each_entry(client, &phandle->power_client_clist, list) {
> -             if (client->usecase_ndx < VOTE_INDEX_MAX &&
> -                 client->usecase_ndx > max_usecase_ndx)
> -                     max_usecase_ndx = client->usecase_ndx;
> -     }
> -
> -     if (phandle->current_usecase_ndx != max_usecase_ndx) {
> -             changed = true;
> -             prev_usecase_ndx = phandle->current_usecase_ndx;
> -             phandle->current_usecase_ndx = max_usecase_ndx;
> -     }
> -
> -     pr_debug("%pS: changed=%d current idx=%d request client %s id:%u 
> enable:%d refcount:%d\n",
> -             __builtin_return_address(0), changed, max_usecase_ndx,
> -             pclient->name, pclient->id, enable, pclient->refcount);
> -
> -     if (!changed)
> -             goto end;
> -
> -     if (enable) {
> -             dpu_power_event_trigger_locked(phandle,
> -                             DPU_POWER_EVENT_PRE_ENABLE);
> -
> -             for (i = 0; i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++) {
> -                     rc = dpu_power_data_bus_update(
> -                                     &phandle->data_bus_handle[i], enable);
> -                     if (rc) {
> -                             pr_err("failed to set data bus vote id=%d 
> rc=%d\n",
> -                                             i, rc);
> -                             goto data_bus_hdl_err;
> -                     }
> -             }
> -
> -             rc = dpu_power_reg_bus_update(phandle->reg_bus_hdl,
> -                                                     max_usecase_ndx);
> -             if (rc) {
> -                     pr_err("failed to set reg bus vote rc=%d\n", rc);
> -                     goto reg_bus_hdl_err;
> -             }
> -
> -             dpu_power_event_trigger_locked(phandle,
> -                             DPU_POWER_EVENT_POST_ENABLE);
> -
> -     } else {
> -             dpu_power_event_trigger_locked(phandle,
> -                             DPU_POWER_EVENT_PRE_DISABLE);
> -
> -             dpu_power_reg_bus_update(phandle->reg_bus_hdl,
> -                                                     max_usecase_ndx);
> -
> -             for (i = 0 ; i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++)
> -                     dpu_power_data_bus_update(&phandle->data_bus_handle[i],
> -                                     enable);
> -
> -             dpu_power_event_trigger_locked(phandle,
> -                             DPU_POWER_EVENT_POST_DISABLE);
> -     }
> -
> -end:
> -     mutex_unlock(&phandle->phandle_lock);
> -     return rc;
> -
> -reg_bus_hdl_err:
> -     for (i = 0 ; i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++)
> -             dpu_power_data_bus_update(&phandle->data_bus_handle[i], 0);
> -data_bus_hdl_err:
> -     phandle->current_usecase_ndx = prev_usecase_ndx;
> -     mutex_unlock(&phandle->phandle_lock);
> -     return rc;
> -}
> -
> -struct dpu_power_event *dpu_power_handle_register_event(
> -             struct dpu_power_handle *phandle,
> -             u32 event_type, void (*cb_fnc)(u32 event_type, void *usr),
> -             void *usr, char *client_name)
> -{
> -     struct dpu_power_event *event;
> -
> -     if (!phandle) {
> -             pr_err("invalid power handle\n");
> -             return ERR_PTR(-EINVAL);
> -     } else if (!cb_fnc || !event_type) {
> -             pr_err("no callback fnc or event type\n");
> -             return ERR_PTR(-EINVAL);
> -     }
> -
> -     event = kzalloc(sizeof(struct dpu_power_event), GFP_KERNEL);
> -     if (!event)
> -             return ERR_PTR(-ENOMEM);
> -
> -     event->event_type = event_type;
> -     event->cb_fnc = cb_fnc;
> -     event->usr = usr;
> -     strlcpy(event->client_name, client_name, MAX_CLIENT_NAME_LEN);
> -     event->active = true;
> -
> -     mutex_lock(&phandle->phandle_lock);
> -     list_add(&event->list, &phandle->event_list);
> -     mutex_unlock(&phandle->phandle_lock);
> -
> -     return event;
> -}
> -
> -void dpu_power_handle_unregister_event(
> -             struct dpu_power_handle *phandle,
> -             struct dpu_power_event *event)
> -{
> -     if (!phandle || !event) {
> -             pr_err("invalid phandle or event\n");
> -     } else if (!event->active) {
> -             pr_err("power handle deinit already done\n");
> -             kfree(event);
> -     } else {
> -             mutex_lock(&phandle->phandle_lock);
> -             list_del_init(&event->list);
> -             mutex_unlock(&phandle->phandle_lock);
> -             kfree(event);
> -     }
> -}
> diff --git a/drivers/gpu/drm/msm/dpu_power_handle.h 
> b/drivers/gpu/drm/msm/dpu_power_handle.h
> deleted file mode 100644
> index 9a6d4b9..0000000
> --- a/drivers/gpu/drm/msm/dpu_power_handle.h
> +++ /dev/null
> @@ -1,288 +0,0 @@
> -/* Copyright (c) 2016-2018, The Linux Foundation. All rights reserved.
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License version 2 and
> - * only version 2 as published by the Free Software Foundation.
> - *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> - * GNU General Public License for more details.
> - *
> - */
> -
> -#ifndef _DPU_POWER_HANDLE_H_
> -#define _DPU_POWER_HANDLE_H_
> -
> -#define MAX_CLIENT_NAME_LEN 128
> -
> -#define DPU_POWER_HANDLE_ENABLE_BUS_AB_QUOTA 0
> -#define DPU_POWER_HANDLE_DISABLE_BUS_AB_QUOTA        0
> -#define DPU_POWER_HANDLE_ENABLE_BUS_IB_QUOTA 1600000000
> -#define DPU_POWER_HANDLE_DISABLE_BUS_IB_QUOTA        0
> -
> -#include <linux/dpu_io_util.h>
> -
> -/* event will be triggered before power handler disable */
> -#define DPU_POWER_EVENT_PRE_DISABLE  0x1
> -
> -/* event will be triggered after power handler disable */
> -#define DPU_POWER_EVENT_POST_DISABLE 0x2
> -
> -/* event will be triggered before power handler enable */
> -#define DPU_POWER_EVENT_PRE_ENABLE   0x4
> -
> -/* event will be triggered after power handler enable */
> -#define DPU_POWER_EVENT_POST_ENABLE  0x8
> -
> -/**
> - * mdss_bus_vote_type: register bus vote type
> - * VOTE_INDEX_DISABLE: removes the client vote
> - * VOTE_INDEX_LOW: keeps the lowest vote for register bus
> - * VOTE_INDEX_MAX: invalid
> - */
> -enum mdss_bus_vote_type {
> -     VOTE_INDEX_DISABLE,
> -     VOTE_INDEX_LOW,
> -     VOTE_INDEX_MAX,
> -};
> -
> -/**
> - * enum dpu_power_handle_data_bus_client - type of axi bus clients
> - * @DPU_POWER_HANDLE_DATA_BUS_CLIENT_RT: core real-time bus client
> - * @DPU_POWER_HANDLE_DATA_BUS_CLIENT_NRT: core non-real-time bus client
> - * @DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX: maximum number of bus client type
> - */
> -enum dpu_power_handle_data_bus_client {
> -     DPU_POWER_HANDLE_DATA_BUS_CLIENT_RT,
> -     DPU_POWER_HANDLE_DATA_BUS_CLIENT_NRT,
> -     DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX
> -};
> -
> -/**
> - * enum DPU_POWER_HANDLE_DBUS_ID - data bus identifier
> - * @DPU_POWER_HANDLE_DBUS_ID_MNOC: DPU/MNOC data bus
> - * @DPU_POWER_HANDLE_DBUS_ID_LLCC: MNOC/LLCC data bus
> - * @DPU_POWER_HANDLE_DBUS_ID_EBI: LLCC/EBI data bus
> - */
> -enum DPU_POWER_HANDLE_DBUS_ID {
> -     DPU_POWER_HANDLE_DBUS_ID_MNOC,
> -     DPU_POWER_HANDLE_DBUS_ID_LLCC,
> -     DPU_POWER_HANDLE_DBUS_ID_EBI,
> -     DPU_POWER_HANDLE_DBUS_ID_MAX,
> -};
> -
> -/**
> - * struct dpu_power_client: stores the power client for dpu driver
> - * @name:    name of the client
> - * @usecase_ndx: current regs bus vote type
> - * @refcount:        current refcount if multiple modules are using same
> - *              same client for enable/disable. Power module will
> - *              aggregate the refcount and vote accordingly for this
> - *              client.
> - * @id:              assigned during create. helps for debugging.
> - * @list:    list to attach power handle master list
> - * @ab:         arbitrated bandwidth for each bus client
> - * @ib:         instantaneous bandwidth for each bus client
> - * @active:  inidcates the state of dpu power handle
> - */
> -struct dpu_power_client {
> -     char name[MAX_CLIENT_NAME_LEN];
> -     short usecase_ndx;
> -     short refcount;
> -     u32 id;
> -     struct list_head list;
> -     u64 ab[DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX];
> -     u64 ib[DPU_POWER_HANDLE_DATA_BUS_CLIENT_MAX];
> -     bool active;
> -};
> -
> -/**
> - * struct dpu_power_data_handle: power handle struct for data bus
> - * @data_bus_scale_table: pointer to bus scaling table
> - * @data_bus_hdl: current data bus handle
> - * @axi_port_cnt: number of rt axi ports
> - * @nrt_axi_port_cnt: number of nrt axi ports
> - * @bus_channels: number of memory bus channels
> - * @curr_bw_uc_idx: current use case index of data bus
> - * @ao_bw_uc_idx: active only use case index of data bus
> - * @ab_rt: realtime ab quota
> - * @ib_rt: realtime ib quota
> - * @ab_nrt: non-realtime ab quota
> - * @ib_nrt: non-realtime ib quota
> - * @enable: true if bus is enabled
> - */
> -struct dpu_power_data_bus_handle {
> -     struct msm_bus_scale_pdata *data_bus_scale_table;
> -     u32 data_bus_hdl;
> -     u32 axi_port_cnt;
> -     u32 nrt_axi_port_cnt;
> -     u32 bus_channels;
> -     u32 curr_bw_uc_idx;
> -     u32 ao_bw_uc_idx;
> -     u64 ab_rt;
> -     u64 ib_rt;
> -     u64 ab_nrt;
> -     u64 ib_nrt;
> -     bool enable;
> -};
> -
> -/*
> - * struct dpu_power_event - local event registration structure
> - * @client_name: name of the client registering
> - * @cb_fnc: pointer to desired callback function
> - * @usr: user pointer to pass to callback event trigger
> - * @event: refer to DPU_POWER_HANDLE_EVENT_*
> - * @list: list to attach event master list
> - * @active: indicates the state of dpu power handle
> - */
> -struct dpu_power_event {
> -     char client_name[MAX_CLIENT_NAME_LEN];
> -     void (*cb_fnc)(u32 event_type, void *usr);
> -     void *usr;
> -     u32 event_type;
> -     struct list_head list;
> -     bool active;
> -};
> -
> -/**
> - * struct dpu_power_handle: power handle main struct
> - * @client_clist: master list to store all clients
> - * @phandle_lock: lock to synchronize the enable/disable
> - * @dev: pointer to device structure
> - * @usecase_ndx: current usecase index
> - * @reg_bus_hdl: current register bus handle
> - * @data_bus_handle: context structure for data bus control
> - * @event_list: current power handle event list
> - */
> -struct dpu_power_handle {
> -     struct list_head power_client_clist;
> -     struct mutex phandle_lock;
> -     struct device *dev;
> -     u32 current_usecase_ndx;
> -     u32 reg_bus_hdl;
> -     struct dpu_power_data_bus_handle data_bus_handle
> -             [DPU_POWER_HANDLE_DBUS_ID_MAX];
> -     struct list_head event_list;
> -};
> -
> -/**
> - * dpu_power_resource_init() - initializes the dpu power handle
> - * @pdev:   platform device to search the power resources
> - * @pdata:  power handle to store the power resources
> - *
> - * Return: error code.
> - */
> -int dpu_power_resource_init(struct platform_device *pdev,
> -     struct dpu_power_handle *pdata);
> -
> -/**
> - * dpu_power_resource_deinit() - release the dpu power handle
> - * @pdev:   platform device for power resources
> - * @pdata:  power handle containing the resources
> - *
> - * Return: error code.
> - */
> -void dpu_power_resource_deinit(struct platform_device *pdev,
> -     struct dpu_power_handle *pdata);
> -
> -/**
> - * dpu_power_client_create() - create the client on power handle
> - * @pdata:  power handle containing the resources
> - * @client_name: new client name for registration
> - *
> - * Return: error code.
> - */
> -struct dpu_power_client *dpu_power_client_create(struct dpu_power_handle 
> *pdata,
> -     char *client_name);
> -
> -/**
> - * dpu_power_client_destroy() - destroy the client on power handle
> - * @pdata:  power handle containing the resources
> - * @client_name: new client name for registration
> - *
> - * Return: none
> - */
> -void dpu_power_client_destroy(struct dpu_power_handle *phandle,
> -     struct dpu_power_client *client);
> -
> -/**
> - * dpu_power_resource_enable() - enable/disable the power resources
> - * @pdata:  power handle containing the resources
> - * @client: client information to enable/disable its vote
> - * @enable: boolean request for enable/disable
> - *
> - * Return: error code.
> - */
> -int dpu_power_resource_enable(struct dpu_power_handle *pdata,
> -     struct dpu_power_client *pclient, bool enable);
> -
> -/**
> - * dpu_power_data_bus_state_update() - update data bus state
> - * @pdata:  power handle containing the resources
> - * @enable: take enable vs disable path
> - *
> - * Return: error code.
> - */
> -int dpu_power_data_bus_state_update(struct dpu_power_handle *phandle,
> -                                                     bool enable);
> -
> -/**
> - * dpu_power_data_bus_set_quota() - set data bus quota for power client
> - * @phandle:  power handle containing the resources
> - * @client: client information to set quota
> - * @bus_client: real-time or non-real-time bus client
> - * @bus_id: identifier of data bus, see DPU_POWER_HANDLE_DBUS_ID
> - * @ab_quota: arbitrated bus bandwidth
> - * @ib_quota: instantaneous bus bandwidth
> - *
> - * Return: zero if success, or error code otherwise
> - */
> -int dpu_power_data_bus_set_quota(struct dpu_power_handle *phandle,
> -             struct dpu_power_client *pclient,
> -             int bus_client, u32 bus_id,
> -             u64 ab_quota, u64 ib_quota);
> -
> -/**
> - * dpu_power_data_bus_bandwidth_ctrl() - control data bus bandwidth enable
> - * @phandle:  power handle containing the resources
> - * @client: client information to bandwidth control
> - * @enable: true to enable bandwidth for data base
> - *
> - * Return: none
> - */
> -void dpu_power_data_bus_bandwidth_ctrl(struct dpu_power_handle *phandle,
> -             struct dpu_power_client *pclient, int enable);
> -
> -/**
> - * dpu_power_handle_register_event - register a callback function for an 
> event.
> - *   Clients can register for multiple events with a single register.
> - *   Any block with access to phandle can register for the event
> - *   notification.
> - * @phandle: power handle containing the resources
> - * @event_type:      event type to register; refer DPU_POWER_HANDLE_EVENT_*
> - * @cb_fnc:  pointer to desired callback function
> - * @usr:     user pointer to pass to callback on event trigger
> - *
> - * Return:   event pointer if success, or error code otherwise
> - */
> -struct dpu_power_event *dpu_power_handle_register_event(
> -             struct dpu_power_handle *phandle,
> -             u32 event_type, void (*cb_fnc)(u32 event_type, void *usr),
> -             void *usr, char *client_name);
> -/**
> - * dpu_power_handle_unregister_event - unregister callback for event(s)
> - * @phandle: power handle containing the resources
> - * @event:   event pointer returned after power handle register
> - */
> -void dpu_power_handle_unregister_event(struct dpu_power_handle *phandle,
> -             struct dpu_power_event *event);
> -
> -/**
> - * dpu_power_handle_get_dbus_name - get name of given data bus identifier
> - * @bus_id:  data bus identifier
> - * Return:   Pointer to name string if success; NULL otherwise
> - */
> -const char *dpu_power_handle_get_dbus_name(u32 bus_id);
> -
> -#endif /* _DPU_POWER_HANDLE_H_ */
> diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
> index 5c267cd..60b6919 100644
> --- a/drivers/gpu/drm/msm/msm_drv.c
> +++ b/drivers/gpu/drm/msm/msm_drv.c
> @@ -340,7 +340,6 @@ static int msm_drm_uninit(struct device *dev)
>       component_unbind_all(dev, ddev);
>  
>  #ifdef CONFIG_DRM_MSM_DPU
> -     dpu_power_resource_deinit(pdev, &priv->phandle);
>       dpu_dbg_destroy();
>  #endif
>  
> @@ -519,12 +518,6 @@ static int msm_drm_init(struct device *dev, struct 
> drm_driver *drv)
>       drm_mode_config_init(ddev);
>  
>  #ifdef CONFIG_DRM_MSM_DPU
> -     ret = dpu_power_resource_init(pdev, &priv->phandle);
> -     if (ret) {
> -             pr_err("dpu power resource init failed\n");
> -             goto power_init_fail;
> -     }
> -
>       ret = dpu_dbg_init(&pdev->dev);
>       if (ret) {
>               dev_err(dev, "failed to init dpu dbg: %d\n", ret);
> @@ -733,8 +726,6 @@ static int msm_drm_init(struct device *dev, struct 
> drm_driver *drv)
>  #ifdef CONFIG_DRM_MSM_DPU
>       dpu_dbg_destroy();
>  dbg_init_fail:
> -     dpu_power_resource_deinit(pdev, &priv->phandle);
> -power_init_fail:
>  #endif
>       if (mdss && mdss->funcs)
>               mdss->funcs->destroy(ddev);
> diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h
> index f9ae96f..27a73a8 100644
> --- a/drivers/gpu/drm/msm/msm_drv.h
> +++ b/drivers/gpu/drm/msm/msm_drv.h
> @@ -46,8 +46,6 @@
>  #include <drm/msm_drm.h>
>  #include <drm/drm_gem.h>
>  
> -#include "dpu_power_handle.h"
> -
>  #define GET_MAJOR_REV(rev)           ((rev) >> 28)
>  #define GET_MINOR_REV(rev)           (((rev) >> 16) & 0xFFF)
>  #define GET_STEP_REV(rev)            ((rev) & 0xFFFF)
> @@ -375,8 +373,6 @@ struct msm_drm_private {
>  
>       struct msm_kms *kms;
>  
> -     struct dpu_power_handle phandle;
> -
>       /* subordinate devices, if present: */
>       struct platform_device *gpu_pdev;
>  
> -- 
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
> 

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to