[Freedreno] [DPU PATCH 00/19] drm/msm: dpu: Clean up dpu_dbg

Sean Paul seanpaul at chromium.org
Wed Jun 20 20:48:22 UTC 2018


Well hello,
This patchset attempts to clean up some of dpu_dbg that was duplicating
functionality already upstream. The majority of the set is replacing
all instances of DPU_EVT* functions with proper tracepoints instead of
the obfuscated ones that exist now. The second part of the set disects
dpu_dbg.[ch] a bit and removes the arbitrary register reads, and evtlog
duplication. Finally, the dpu_dbg files are moved into the dpu1
directory.

Sean

Sean Paul (19):
  drm/msm: dpu_encoder: Replace DPU_EVT with tracepoints
  drm/msm: dpu_crtc: Replace DPU_EVT with tracepoints
  drm/msm: dpu_plane: Replace DPU_EVT with tracepoints
  drm/msm: dpu_rm: Replace DPU_EVT with tracepoints
  drm/msm: dpu_kms: Replace DPU_EVT with tracepoints
  drm/msm: dpu_encoder_phys_cmd: Replace DPU_EVT with tracepoints
  drm/msm: dpu_encoder_phys_vid: Replace DPU_EVT with tracepoints
  drm/msm: dpu_vbif: Replace DPU_EVT with tracepoints
  drm/msm: dpu_pingpong: Replace DPU_EVT with tracepoints
  drm/msm: dpu_core_irq: Replace DPU_EVT with tracepoints
  drm/msm: dpu_core_perf: Replace DPU_EVT with tracepoints
  drm/msm: dpu_mdss: Replace DPU_EVT with DRM_ERROR
  drm/msm: dpu: Remove dpu evtlog
  drm/msm: dpu_dbg: Remove dump_all option for dumping registers
  drm/msm: dpu: Remove arbitrary register dumps
  drm/msm: dpu: Remove panic from dpu debug dump
  drm/msm: dpu: Remove dsi debug block name
  drm/msm: dpu_dbg: Remove string parsing from DBG_DUMP
  drm/msm: dpu: Move dpu_dbg into dpu1 directory

 drivers/gpu/drm/msm/Makefile                  |   3 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c  |  52 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c |   2 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c      | 122 +--
 drivers/gpu/drm/msm/{ => disp/dpu1}/dpu_dbg.c | 944 +-----------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_dbg.h       | 116 +++
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c   | 292 +++---
 .../drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c  |  81 +-
 .../drm/msm/disp/dpu1/dpu_encoder_phys_vid.c  |  38 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_cdm.c    |   3 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c    |   3 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c   |   3 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c     |   3 -
 .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.c   |   6 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c   |  14 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c    |   3 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c       |  46 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c      |   5 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c     |  19 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_rm.c        |  65 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h     | 836 +++++++++++++++-
 drivers/gpu/drm/msm/disp/dpu1/dpu_vbif.c      |   7 +-
 drivers/gpu/drm/msm/dpu_dbg.h                 | 406 --------
 drivers/gpu/drm/msm/dpu_dbg_evtlog.c          | 306 ------
 24 files changed, 1251 insertions(+), 2124 deletions(-)
 rename drivers/gpu/drm/msm/{ => disp/dpu1}/dpu_dbg.c (70%)
 create mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_dbg.h
 delete mode 100644 drivers/gpu/drm/msm/dpu_dbg.h
 delete mode 100644 drivers/gpu/drm/msm/dpu_dbg_evtlog.c

-- 
Sean Paul, Software Engineer, Google / Chromium OS



More information about the Freedreno mailing list