From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 0/9] drm/msm/mdp5: Cursor plane stuff Date: Mon, 19 Dec 2016 16:50:37 +0100 Message-ID: <20161219155037.ffejtdirfeph65mf@phenom.ffwll.local> References: <1482149338-586-1-git-send-email-architt@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-wm0-f67.google.com ([74.125.82.67]:32870 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755120AbcLSPum (ORCPT ); Mon, 19 Dec 2016 10:50:42 -0500 Received: by mail-wm0-f67.google.com with SMTP id u144so19337325wmu.0 for ; Mon, 19 Dec 2016 07:50:41 -0800 (PST) Content-Disposition: inline In-Reply-To: <1482149338-586-1-git-send-email-architt@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Archit Taneja Cc: robdclark@gmail.com, linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, daniel@ffwll.ch, maarten.lankhorst@linux.intel.com On Mon, Dec 19, 2016 at 05:38:49PM +0530, Archit Taneja wrote: > This series does some mdp5_plane related clean ups (use plane helpers > for clipping etc), adds MDP5 bits needed for cursor plane blocks, and > then add cursor planes. > > On older MDP5 versions, we had cursor HW in Layer Mixer blocks, and > that's implemented in mdp5_crtc.c. With newer hardware, the cursor > blocks look exactly like MDP5 pipes (SSPPs). > > The "faster than vblank cursor position update stuff" has been > copied from vc4 and the patches posted recently by Maarten for Intel: > > https://patchwork.kernel.org/patch/9466417/ > > I've tried this with some X11 UI and modetest -C, and it seems to > work okay. Are there any tests that mix up atomic commits and legacy > cursor updates a lot to identify issues? kms_legacy_cursor mixes cursor updates with page_flips (and if you use atomic for those, atomic flips ofcourse). But the test is a bit too oveager and even expects cursor updates vs. modesets to not block. Not sure Maarten fixed that already. -Daniel > > Archit Taneja (9): > drm/msm/mdp5: cfg: Add pipe_cursor block > drm/msm/mdp5: Update generated headers > drm/msm/mdp5: Prepare CRTC/LM for empty stages > drm/msm/mdp5: Use plane helpers to configure src/dst rectangles > drm/msm/mdp5: Configure COLOR3_OUT propagation > drm/msm/mdp5: Misc cursor plane bits > drm/msm/mdp5: Refactor mdp5_plane_atomic_check > HACK: drm/msm/mdp5: Add support for legacy cursor updates > drm/msm/mdp5: Add cursor planes > > drivers/gpu/drm/msm/mdp/mdp5/mdp5.xml.h | 30 +++-- > drivers/gpu/drm/msm/mdp/mdp5/mdp5_cfg.c | 10 ++ > drivers/gpu/drm/msm/mdp/mdp5/mdp5_cfg.h | 1 + > drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 73 +++++++++++- > drivers/gpu/drm/msm/mdp/mdp5/mdp5_ctl.c | 14 ++- > drivers/gpu/drm/msm/mdp/mdp5/mdp5_ctl.h | 4 +- > drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c | 38 +++++-- > drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.h | 8 +- > drivers/gpu/drm/msm/mdp/mdp5/mdp5_pipe.c | 8 ++ > drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c | 181 ++++++++++++++++++++++++++---- > drivers/gpu/drm/msm/mdp/mdp_kms.h | 1 + > drivers/gpu/drm/msm/msm_atomic.c | 26 +++-- > 12 files changed, 334 insertions(+), 60 deletions(-) > > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > hosted by The Linux Foundation > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch