[PATCH v3 0/8] drm/msm: Switch to atomic helpers

Sean Paul seanpaul at chromium.org
Mon Apr 2 15:11:24 UTC 2018


I spoke with Rob on IRC about the set and he preferred the private
object be isolated in mdp5. So here's the atomic helper set rebased on
Archit's series. 

One notable difference is that I've dropped the first patch from that
series which added a new private_obj lock to the core. Instead, it's
located in mdp5_kms along with the global state (same as the state_lock
which is currently there).

Thank you for your feedback thus far!

Sean


Archit Taneja (3):
  drm/msm/mdp5: Add global state as a private atomic object
  drm/msm/mdp5: Use the new private_obj state
  drm/msm: Don't subclass drm_atomic_state anymore

Sean Paul (5):
  drm/msm: Refactor complete_commit() to look more the helpers
  drm/msm: Mark the crtc->state->event consumed
  drm/msm: Issue queued events when disabling crtc
  drm/msm: Remove msm_commit/worker, use atomic helper commit
  drm/msm: Switch to atomic_helper_commit()

 drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c  |   1 +
 drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c  |  10 +
 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c   | 105 ++++++---
 drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.h   |  27 +--
 drivers/gpu/drm/msm/disp/mdp5/mdp5_mixer.c |  12 +-
 drivers/gpu/drm/msm/disp/mdp5/mdp5_pipe.c  |  20 +-
 drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c   |  17 +-
 drivers/gpu/drm/msm/msm_atomic.c           | 236 +--------------------
 drivers/gpu/drm/msm/msm_drv.c              |  11 +-
 drivers/gpu/drm/msm/msm_drv.h              |   7 +-
 drivers/gpu/drm/msm/msm_kms.h              |  14 --
 11 files changed, 141 insertions(+), 319 deletions(-)

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



More information about the dri-devel mailing list