From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933769AbcIUSSr (ORCPT ); Wed, 21 Sep 2016 14:18:47 -0400 Received: from mail.kmu-office.ch ([178.209.48.109]:55312 "EHLO mail.kmu-office.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933024AbcIUSSo (ORCPT ); Wed, 21 Sep 2016 14:18:44 -0400 MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Date: Wed, 21 Sep 2016 11:13:12 -0700 From: Stefan Agner To: Meng Yi Cc: dri-devel@lists.freedesktop.org, alison.wang@freescale.com, jianwei.wang.chn@gmail.com, linux-kernel@vger.kernel.org Subject: RE: [PATCH 0/4] drm/fsl-dcu: add overlay and cursor plane support In-Reply-To: References: <20160906040233.32471-1-stefan@agner.ch> Message-ID: User-Agent: Roundcube Webmail/1.1.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016-09-07 01:43, Meng Yi wrote: > Hi Stefan, > > I had tested the patches on LS1021A-TWR board using drmlib. > > Like set three overlays: > root@ls1021atwr:~# ./modetest -P 39:900x100+10+10@RG24 -P > 39:200x200+300+0@RG24 -P 39:200x200+400+300@RG24 > > How did you test the overlays and cursor layer, I mean I see you using > x-window like thing in the video. I did use X with the modesetting driver. You just need to choose modesetting in your xorg.conf, and it should make use of the cursor layer automatically. >> This patchset adds overlay and cursor plane support. It also fixes some issues >> uncovered during implementation of this. >> >> However, the plane updates currently causes the display to flicker for unknown >> reasons. As far as I can tell, the CRTC atomic_flush should trigger the update >> correctly via READREG, which according to >> documentation: >> The READREG bit causes a single transfer to begin at the next frame blanking >> period. This bit is cleared when the transfer is complete. >> >> I made a video how that looks: >> https://cloud.agner.ch/index.php/s/Yfqa2u7UBEWUT8N It would be interesting whether you see that on LS1021a too. -- Stefan >> >> Any ideas? >> >> Stefan Agner (4): >> drm/fsl-dcu: support overlay and cursor planes >> drm/fsl-dcu: respect pos/size register sizes >> drm/fsl-dcu: update all registers on flush >> drm/fsl-dcu: do not update when modifying irq registers >> >> drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c | 50 >> ++++++++++++++++++++++++----- >> drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 4 --- >> drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h | 8 ++--- >> drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 42 +++++++++++------------- >> drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.h | 3 +- >> 5 files changed, 67 insertions(+), 40 deletions(-) >> >> -- >> 2.9.3