From patchwork Fri Jun 29 13:01:40 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Benjamin Gaignard X-Patchwork-Id: 140555 Delivered-To: patch@linaro.org Received: by 2002:a2e:9754:0:0:0:0:0 with SMTP id f20-v6csp815746ljj; Fri, 29 Jun 2018 06:02:07 -0700 (PDT) X-Google-Smtp-Source: ADUXVKImEGsgg2qKyWjvfrj/+mjlPtVj/fKcct61CBQxZPImNdHu5ijfilDrYtLnS6lKolEVz9eu X-Received: by 2002:a17:902:4203:: with SMTP id g3-v6mr14552340pld.315.1530277326956; Fri, 29 Jun 2018 06:02:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1530277326; cv=none; d=google.com; s=arc-20160816; b=K9wT0D51gMiqVEAuWo0+vxlGInR5xMUJYdBm+Dd9nVlsHJIrqeuFD4pK3PJxHD/I2N 6y20iwETmCQXGKpo4yQBPzALH3dtxxrrpTHHahBmmyHHxNxO26I7q3FwCRIThIFRFSVZ 3962yPP0M4LS1CcFSqdMNqbTRMZXrbTwmDIBuarkOOXN7IWFbhKM3KPYY5RfUv4+1yLz /kZxRCEoRH3K+AwTOfxUlSni8q2FEAhZM0Kq1cKTy2hAXr3jAsMRVaejW8bZn9UfPmZY XkuPDRLeVySgvH8Ncei6D38LFKKkuk2svFrlj5WqWh1w/0vx3F/vK5lMWuMq2oFAzxJ7 uo6A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :dkim-signature:arc-authentication-results; bh=eGdPVS1hmmh8A+MBC/MKcmxcBqsPEV4vpqI/JhInOdQ=; b=MhpRSvUvBnJy7WmMteENn6c9v8b3pBEFvCBqSllCCjr9Pd49jPdHP7a5hc7g8GLXfj KBx4N7184Kd/G8iO+EM4I0AOeJ2npprniCnPGfP5vjUDZqOmSZAitZQvmdhqloDvJYoY J6OCGgl1UKYv7P0/CMyapHn0f+8FJAtQy12fby3igE3izYzr3tNLpedJZ+K99gQDj7WH M3PXWoSBJ4V03f/AZohYgykS1HPecIIaK0q5hP6Rj5V2SyRSSMKXZ4QlM5+q+uCMZA77 fVfKXYfFLQoalo20zPgsmitbkqi1bPPNnKJRDcuF/Vp/Cys7/Og4R1PZdwdpP8Q/lvE9 vIxA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=Xz52LIvt; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j91-v6si9119250pld.402.2018.06.29.06.02.06; Fri, 29 Jun 2018 06:02:06 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=Xz52LIvt; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965222AbeF2NCE (ORCPT + 31 others); Fri, 29 Jun 2018 09:02:04 -0400 Received: from mail-wm0-f68.google.com ([74.125.82.68]:55717 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935222AbeF2NCC (ORCPT ); Fri, 29 Jun 2018 09:02:02 -0400 Received: by mail-wm0-f68.google.com with SMTP id v16-v6so2119991wmv.5 for ; Fri, 29 Jun 2018 06:02:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=eGdPVS1hmmh8A+MBC/MKcmxcBqsPEV4vpqI/JhInOdQ=; b=Xz52LIvtEadWImfaXrITpg5+zYdgE4cDFSgJzXuG6jYGlCE/iqA7hcv2bKvplD6JVh EQHSuP8mgxCh//ThyeI/n0cmGgDo1yrgsSszaF56WexPCnX8olNpx2dosgg6dxzwRm1u egfYTkdqYJPSzCSgBIKRRgwDWRehmcxu/r0dw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=eGdPVS1hmmh8A+MBC/MKcmxcBqsPEV4vpqI/JhInOdQ=; b=Wlrede1KY13ANOJc9dXLfPaIkUUsZBe8BeJjvvO/cFBU89ls6wxid/ruYWhBwInufk s2uimh/+tZRaaiD84rs5tYN4ccXbSAi3H+jeJ3s7ZjPCD5y1/fHTuFaZuy9uvF7kQywa 0ST9bzOPBvNS+McNpzB9nc5Oz4LNcuodL7YUNO2G8QjIAsY+CQmqt/xPtDCEo/6NG8x2 +zXdhqK/4HxNtAVUnT1/Yy8qUxBKQsfEoXgsD5hoM1waQQW79xJgfe4j02P8ZMxm1T2D UuZvI2kHPJVrSE0/LMEZYkuCftlx8lbpHS4QWRaumb7Ob2e69u+JvnqXo6RI+tQo9yBp yNrQ== X-Gm-Message-State: APt69E3xE5M9FGWkE0LI295pHZIKpcWcBI9nmcKgNnhlss6/C7thp9+a H3nUgTA97G86vzdB7Fwapz6JUf5+oY8= X-Received: by 2002:a1c:a016:: with SMTP id j22-v6mr888972wme.1.1530277321604; Fri, 29 Jun 2018 06:02:01 -0700 (PDT) Received: from lmecxl0911.lme.st.com ([80.215.34.152]) by smtp.gmail.com with ESMTPSA id q70-v6sm2218724wmd.45.2018.06.29.06.01.59 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 29 Jun 2018 06:02:00 -0700 (PDT) From: Benjamin Gaignard To: yannick.fertre@st.com, philippe.cornu@st.com, airlied@linux.ie Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Benjamin Gaignard , Benjamin Gaignard Subject: [PATCH] drm: stm: implement get_scanout_position function Date: Fri, 29 Jun 2018 15:01:40 +0200 Message-Id: <20180629130140.16004-1-benjamin.gaignard@linaro.org> X-Mailer: git-send-email 2.15.0 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hardware allow to read the position in scanout buffer so we can use this information to make wait of vblank more accurate. Active area bounds (start, end, total height) have already been computed and written in ltdc registers, read them and get the current line position to compute vpos value. Signed-off-by: Benjamin Gaignard --- drivers/gpu/drm/stm/drv.c | 2 ++ drivers/gpu/drm/stm/ltdc.c | 45 +++++++++++++++++++++++++++++++++++++++++++++ drivers/gpu/drm/stm/ltdc.h | 5 +++++ 3 files changed, 52 insertions(+) -- 2.15.0 diff --git a/drivers/gpu/drm/stm/drv.c b/drivers/gpu/drm/stm/drv.c index 8698e08313e1..ac53383350e3 100644 --- a/drivers/gpu/drm/stm/drv.c +++ b/drivers/gpu/drm/stm/drv.c @@ -72,6 +72,8 @@ static struct drm_driver drv_driver = { .gem_prime_vmap = drm_gem_cma_prime_vmap, .gem_prime_vunmap = drm_gem_cma_prime_vunmap, .gem_prime_mmap = drm_gem_cma_prime_mmap, + .get_scanout_position = ltdc_crtc_scanoutpos, + .get_vblank_timestamp = drm_calc_vbltimestamp_from_scanoutpos, }; static int drv_load(struct drm_device *ddev) diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c index d997a6014d6c..05b714673042 100644 --- a/drivers/gpu/drm/stm/ltdc.c +++ b/drivers/gpu/drm/stm/ltdc.c @@ -148,6 +148,8 @@ #define IER_TERRIE BIT(2) /* Transfer ERRor Interrupt Enable */ #define IER_RRIE BIT(3) /* Register Reload Interrupt enable */ +#define CPSR_CYPOS GENMASK(15, 0) /* Current Y position */ + #define ISR_LIF BIT(0) /* Line Interrupt Flag */ #define ISR_FUIF BIT(1) /* Fifo Underrun Interrupt Flag */ #define ISR_TERRIF BIT(2) /* Transfer ERRor Interrupt Flag */ @@ -622,6 +624,49 @@ static void ltdc_crtc_disable_vblank(struct drm_crtc *crtc) reg_clear(ldev->regs, LTDC_IER, IER_LIE); } +bool ltdc_crtc_scanoutpos(struct drm_device *ddev, unsigned int pipe, + bool in_vblank_irq, int *vpos, int *hpos, + ktime_t *stime, ktime_t *etime, + const struct drm_display_mode *mode) +{ + struct ltdc_device *ldev = ddev->dev_private; + int line, vactive_start, vactive_end, vtotal; + + if (stime) + *stime = ktime_get(); + + /* The active area starts after vsync + front porch and ends + * at vsync + front porc + display size. + * The total height also include back porch. + * We have 3 possible cases to handle: + * - line < vactive_start: vpos = line - vactive_start and will be + * negative + * - vactive_start < line < vactive_end: vpos = line - vactive_start + * and will be positive + * - line > vactive_end: vpos = line - vtotal - vactive_start + * and will negative + * + * Computation for the two first cases are identical so we can + * simplify the code and only test if line > vactive_end + */ + line = reg_read(ldev->regs, LTDC_CPSR) & CPSR_CYPOS; + vactive_start = reg_read(ldev->regs, LTDC_BPCR) & BPCR_AVBP; + vactive_end = reg_read(ldev->regs, LTDC_AWCR) & AWCR_AAH; + vtotal = reg_read(ldev->regs, LTDC_TWCR) & TWCR_TOTALH; + + if (line > vactive_end) + *vpos = line - vtotal - vactive_start; + else + *vpos = line - vactive_start; + + *hpos = 0; + + if (etime) + *etime = ktime_get(); + + return true; +} + static const struct drm_crtc_funcs ltdc_crtc_funcs = { .destroy = drm_crtc_cleanup, .set_config = drm_atomic_helper_set_config, diff --git a/drivers/gpu/drm/stm/ltdc.h b/drivers/gpu/drm/stm/ltdc.h index 1e16d6afb0d2..b8c5cc41e17a 100644 --- a/drivers/gpu/drm/stm/ltdc.h +++ b/drivers/gpu/drm/stm/ltdc.h @@ -37,6 +37,11 @@ struct ltdc_device { struct fps_info plane_fpsi[LTDC_MAX_LAYER]; }; +bool ltdc_crtc_scanoutpos(struct drm_device *dev, unsigned int pipe, + bool in_vblank_irq, int *vpos, int *hpos, + ktime_t *stime, ktime_t *etime, + const struct drm_display_mode *mode); + int ltdc_load(struct drm_device *ddev); void ltdc_unload(struct drm_device *ddev);