| Message ID | 20230908030521.236309-1-dmitry.baryshkov@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | drm/msm: cleanup private obj handling | expand |
On Fri, 8 Sep 2023 06:05:16 +0300, Dmitry Baryshkov wrote: > While debugging one of the features in DRM/MSM I noticed that MSM > subdrivers still wrap private object access with manual modeset locking. > Since commit b962a12050a3 ("drm/atomic: integrate modeset lock with > private objects") this is no longer required, as the DRM framework > handles private objects internally. Drop these custom locks, while also > > [ ... ] Reviewed-by: Maxime Ripard <mripard@kernel.org> Thanks! Maxime
While debugging one of the features in DRM/MSM I noticed that MSM subdrivers still wrap private object access with manual modeset locking. Since commit b962a12050a3 ("drm/atomic: integrate modeset lock with private objects") this is no longer required, as the DRM framework handles private objects internally. Drop these custom locks, while also cleaning up the surrounding code. Dmitry Baryshkov (5): drm/atomic: add private obj state to state dump drm/msm/dpu: finalise global state object drm/msm/dpu: drop global_state_lock drm/msm/mdp5: migrate SMP dumping to using atomic_print_state drm/msm/mdp5: drop global_state_lock drivers/gpu/drm/drm_atomic.c | 9 ++++ drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 14 +++--- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 1 - drivers/gpu/drm/msm/disp/mdp5/mdp5_irq.c | 2 - drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 54 +++++------------------- drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.h | 1 - drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c | 12 +----- drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.h | 4 +- 8 files changed, 31 insertions(+), 66 deletions(-)