On 2018-11-12 11:42, Sean Paul wrote:
From: Sean Paul <seanp...@chromium.org>

It's only used for debugfs, so just output the enum value instead.

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

Reviewed-by: Jeykumar Sankaran <jsa...@codeaurora.org>

 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c         |  6 ++----
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c | 14 --------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h |  7 -------
 3 files changed, 2 insertions(+), 25 deletions(-)

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
index ed84cf44a222..e09209d6c469 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
@@ -1381,11 +1381,9 @@ static int dpu_crtc_debugfs_state_show(struct
seq_file *s, void *v)
                        dpu_crtc->cur_perf.core_clk_rate);
        for (i = DPU_POWER_HANDLE_DBUS_ID_MNOC;
                        i < DPU_POWER_HANDLE_DBUS_ID_MAX; i++) {
-               seq_printf(s, "bw_ctl[%s]: %llu\n",
-                               dpu_power_handle_get_dbus_name(i),
+               seq_printf(s, "bw_ctl[%d]: %llu\n", i,
                                dpu_crtc->cur_perf.bw_ctl[i]);
-               seq_printf(s, "max_per_pipe_ib[%s]: %llu\n",
-                               dpu_power_handle_get_dbus_name(i),
+               seq_printf(s, "max_per_pipe_ib[%d]: %llu\n", i,
                                dpu_crtc->cur_perf.max_per_pipe_ib[i]);
        }

diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c
b/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c
index fc14116789f2..8c6f92aaaf87 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c
@@ -24,20 +24,6 @@
 #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)
 {
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h
b/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h
index a65b7a297f21..f627ae28ec68 100644
--- a/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h
+++ b/drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h
@@ -207,11 +207,4 @@ struct dpu_power_event
*dpu_power_handle_register_event(
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_ */

--
Jeykumar S
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to