drm/msm: Various dpu locking and legacy cleanups

Submitted by Sean Paul on Nov. 16, 2018, 6:42 p.m.

Details

Reviewer None
Submitted Nov. 16, 2018, 6:42 p.m.
Last Updated Nov. 30, 2018, 10 p.m.
Revision 2

Cover Letter(s)

Revision 1
      From: Sean Paul <seanpaul@chromium.org>

This was originally 3 patchsets, but none have gotten full review, so I
figured I'd package the v2's all up into one set so it's easier to track.

Set 1- https://lists.freedesktop.org/archives/dri-devel/2018-November/196170.html
Set 2- https://lists.freedesktop.org/archives/dri-devel/2018-November/196184.html
Set 3- https://lists.freedesktop.org/archives/dri-devel/2018-November/196276.html

Thanks,
Sean


Sean Paul (24):
  drm/msm: dpu: Remove dpu_power_handle_get_dbus_name()
  drm/msm: dpu: Remove unused trace_dpu_perf_update_bus()
  drm/msm: dpu: Remove dpu_power_client
  drm/msm: dpu: Don't use power_event for vbif_init_memtypes
  drm/msm: dpu: Handle crtc pm_runtime_resume() directly
  drm/msm: dpu: Remove power_handle from core_perf
  drm/msm: dpu: Include dpu_io_util.h directly in dpu_kms.h
  drm/msm: dpu: Move DPU_POWER_HANDLE_DBUS_ID to core_perf
  drm/msm: dpu: Remove dpu_power_handle
  drm/msm: dpu: Fix typo in dpu_encoder
  drm/msm: dpu: Add ->enabled to dpu_encoder_virt
  drm/msm: dpu: Move crtc runtime resume to encoder
  drm/msm: dpu: Don't drop locks in crtc_vblank_enable
  drm/msm: dpu: Grab the modeset locks in frame_event
  drm/msm: dpu: Stop using encoder->crtc pointer
  drm/msm: dpu: Add modeset lock checks where applicable
  drm/msm: dpu: Move pm_runtime_(get|put) from vblank_enable
  drm/msm: dpu: Remove crtc_lock from setup_mixers
  drm/msm: dpu: Remove vblank_callback from encoder
  drm/msm: dpu: Use atomic_disable for dpu_crtc_disable
  drm/msm: dpu: Don't bother checking ->enabled in dpu_crtc_vblank
  drm/msm: dpu: Separate crtc assignment from vblank enable
  drm/msm: dpu: Remove vblank_requested flag from dpu_crtc
  drm/msm: dpu: Remove crtc_lock

 drivers/gpu/drm/msm/Makefile                  |   1 -
 drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c |  37 ++-
 drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.h |  22 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c      | 216 +++++-----------
 drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h      |  15 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c   |  97 ++++---
 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h   |  24 +-
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c       |  76 ++----
 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h       |   6 +-
 .../gpu/drm/msm/disp/dpu1/dpu_power_handle.c  | 240 ------------------
 .../gpu/drm/msm/disp/dpu1/dpu_power_handle.h  | 217 ----------------
 drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h     |  43 +---
 12 files changed, 204 insertions(+), 790 deletions(-)
 delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.c
 delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_power_handle.h
    

Revisions

Patches download mbox

# Name Submitter State A F R T
[v2,01/24] drm/msm: dpu: Remove dpu_power_handle_get_dbus_name() Sean Paul New 1
[v2,02/24] drm/msm: dpu: Remove unused trace_dpu_perf_update_bus() Sean Paul New 1
[v2,03/24] drm/msm: dpu: Remove dpu_power_client Sean Paul New 1
[v2,04/24] drm/msm: dpu: Don't use power_event for vbif_init_memtypes Sean Paul New 1
[v2,05/24] drm/msm: dpu: Handle crtc pm_runtime_resume() directly Sean Paul New 1
[v2,06/24] drm/msm: dpu: Remove power_handle from core_perf Sean Paul New 1
[v2,07/24] drm/msm: dpu: Include dpu_io_util.h directly in dpu_kms.h Sean Paul New 1
[v2,08/24] drm/msm: dpu: Move DPU_POWER_HANDLE_DBUS_ID to core_perf Sean Paul New 1
[v2,09/24] drm/msm: dpu: Remove dpu_power_handle Sean Paul New 1
[v2,10/24] drm/msm: dpu: Fix typo in dpu_encoder Sean Paul New 1
[v2,11/24] drm/msm: dpu: Add ->enabled to dpu_encoder_virt Sean Paul New 1
[v2,12/24] drm/msm: dpu: Move crtc runtime resume to encoder Sean Paul New 1
[v2,13/24] drm/msm: dpu: Don't drop locks in crtc_vblank_enable Sean Paul New 1
[v2,14/24] drm/msm: dpu: Grab the modeset locks in frame_event Sean Paul New 1
[v2,15/24] drm/msm: dpu: Stop using encoder->crtc pointer Sean Paul New 1
[v2,16/24] drm/msm: dpu: Add modeset lock checks where applicable Sean Paul New 1
[v2,17/24] drm/msm: dpu: Move pm_runtime_(get|put) from vblank_enable Sean Paul New 1
[v2,18/24] drm/msm: dpu: Remove crtc_lock from setup_mixers Sean Paul New 1
[v2,19/24] drm/msm: dpu: Remove vblank_callback from encoder Sean Paul New 1
[v2,20/24] drm/msm: dpu: Use atomic_disable for dpu_crtc_disable Sean Paul New 1
[v2,21/24] drm/msm: dpu: Don't bother checking ->enabled in dpu_crtc_vblank Sean Paul New 1
[v2,22/24] drm/msm: dpu: Separate crtc assignment from vblank enable Sean Paul New 1
[v2,23/24] drm/msm: dpu: Remove vblank_requested flag from dpu_crtc Sean Paul New 1
[v2,24/24] drm/msm: dpu: Remove crtc_lock Sean Paul New 1

Patches download mbox

# Name Submitter State A F R T
[v2,01/24] drm/msm: dpu: Remove dpu_power_handle_get_dbus_name() Sean Paul New 1
[v2,02/24] drm/msm: dpu: Remove unused trace_dpu_perf_update_bus() Sean Paul New 1
[v2,03/24] drm/msm: dpu: Remove dpu_power_client Sean Paul New 1
[v2,04/24] drm/msm: dpu: Don't use power_event for vbif_init_memtypes Sean Paul New 1
[v2,05/24] drm/msm: dpu: Handle crtc pm_runtime_resume() directly Sean Paul New 1
[v2,06/24] drm/msm: dpu: Remove power_handle from core_perf Sean Paul New 1
[v2,07/24] drm/msm: dpu: Include dpu_io_util.h directly in dpu_kms.h Sean Paul New 1
[v2,08/24] drm/msm: dpu: Move DPU_POWER_HANDLE_DBUS_ID to core_perf Sean Paul New 1
[v2,09/24] drm/msm: dpu: Remove dpu_power_handle Sean Paul New 1
[v2,10/24] drm/msm: dpu: Fix typo in dpu_encoder Sean Paul New 1
[v2,11/24] drm/msm: dpu: Add ->enabled to dpu_encoder_virt Sean Paul New 1
[v2,12/24] drm/msm: dpu: Move crtc runtime resume to encoder Sean Paul New 1
[v2,13/24] drm/msm: dpu: Don't drop locks in crtc_vblank_enable Sean Paul New 1
[v3,14/24] drm/msm: dpu: Grab the modeset locks in frame_event Sean Paul New 1 1
[v2,15/24] drm/msm: dpu: Stop using encoder->crtc pointer Sean Paul New 1
[v2,16/24] drm/msm: dpu: Add modeset lock checks where applicable Sean Paul New 1
[v2,17/24] drm/msm: dpu: Move pm_runtime_(get|put) from vblank_enable Sean Paul New 1
[v2,18/24] drm/msm: dpu: Remove crtc_lock from setup_mixers Sean Paul New 1
[v2,19/24] drm/msm: dpu: Remove vblank_callback from encoder Sean Paul New 1
[v2,20/24] drm/msm: dpu: Use atomic_disable for dpu_crtc_disable Sean Paul New 1
[v2,21/24] drm/msm: dpu: Don't bother checking ->enabled in dpu_crtc_vblank Sean Paul New 1
[v2,22/24] drm/msm: dpu: Separate crtc assignment from vblank enable Sean Paul New 1
[v2,23/24] drm/msm: dpu: Remove vblank_requested flag from dpu_crtc Sean Paul New 1
[v2,24/24] drm/msm: dpu: Remove crtc_lock Sean Paul New 1