Re: [PATCH v2 00/24] drm/msm: Various dpu locking and legacy cleanups

From: Sean Paul <sean@poorly.run>
To: dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org,
	linux-arm-msm@vger.kernel.org
Cc: Sean Paul <seanpaul@chromium.org>
Subject: Re: [PATCH v2 00/24] drm/msm: Various dpu locking and legacy cleanups
Date: Tue, 4 Dec 2018 10:15:19 -0500	[thread overview]
Message-ID: <20181204151519.GU154160@art_vandelay> (raw)
In-Reply-To: <20181116184238.170034-1-sean@poorly.run>

On Fri, Nov 16, 2018 at 01:42:10PM -0500, Sean Paul wrote:
> 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 for the reviews, I've pushed it to dpu-staging/for-next

Sean

> 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
> 
> -- 
> Sean Paul, Software Engineer, Google / Chromium OS
> 

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel