From mboxrd@z Thu Jan 1 00:00:00 1970 From: Inki Dae Subject: Re: [PATCH 4/4] drm/exynos/decon5433: signal frame done interrupt at VSYNC Date: Tue, 07 Mar 2017 18:14:09 +0900 Message-ID: <58BE79E1.6000108@samsung.com> References: <1487779505-4062-1-git-send-email-a.hajda@samsung.com> <1487779505-4062-5-git-send-email-a.hajda@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Return-path: Received: from mailout3.samsung.com ([203.254.224.33]:32800 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754884AbdCGJS1 (ORCPT ); Tue, 7 Mar 2017 04:18:27 -0500 Received: from epcas1p1.samsung.com (unknown [182.195.41.45]) by mailout3.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0OMF032P6TTUU750@mailout3.samsung.com> for linux-samsung-soc@vger.kernel.org; Tue, 07 Mar 2017 18:17:54 +0900 (KST) In-reply-to: <1487779505-4062-5-git-send-email-a.hajda@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Andrzej Hajda Cc: Bartlomiej Zolnierkiewicz , Marek Szyprowski , dri-devel@lists.freedesktop.org, linux-samsung-soc@vger.kernel.org, Krzysztof Kozlowski , Javier Martinez Canillas 2017년 02월 23일 01:05에 Andrzej Hajda 이(가) 쓴 글: > DECON in case of video mode generates interrupt by default at start > of vertical back porch. As this interrupt is used to generate VBLANK > events more optimal point is start of vertical front porch. > > Signed-off-by: Andrzej Hajda > --- > drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 2 +- > include/video/exynos5433_decon.h | 4 ++++ > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/exynos/exynos5433_drm_decon.c b/drivers/gpu/drm/exynos/exynos5433_drm_decon.c > index bfa9396..2694b32 100644 > --- a/drivers/gpu/drm/exynos/exynos5433_drm_decon.c > +++ b/drivers/gpu/drm/exynos/exynos5433_drm_decon.c > @@ -105,7 +105,7 @@ static int decon_enable_vblank(struct exynos_drm_crtc *crtc) > if (ctx->out_type & IFTYPE_I80) > val |= VIDINTCON0_FRAMEDONE; > else > - val |= VIDINTCON0_INTFRMEN; > + val |= VIDINTCON0_INTFRMEN | VIDINTCON0_FRAMESEL_FP; > > writel(val, ctx->addr + DECON_VIDINTCON0); > } > diff --git a/include/video/exynos5433_decon.h b/include/video/exynos5433_decon.h > index beefc62..2391b03 100644 > --- a/include/video/exynos5433_decon.h > +++ b/include/video/exynos5433_decon.h > @@ -128,6 +128,10 @@ > > /* VIDINTCON0 */ > #define VIDINTCON0_FRAMEDONE (1 << 17) > +#define VIDINTCON0_FRAMESEL_BP (0 << 15) > +#define VIDINTCON0_FRAMESEL_VS (1 << 15) > +#define VIDINTCON0_FRAMESEL_AC (2 << 15) Above definitions are not used. Thanks. > +#define VIDINTCON0_FRAMESEL_FP (3 << 15) > #define VIDINTCON0_INTFRMEN (1 << 12) > #define VIDINTCON0_INTEN (1 << 0) > >