Received: by 2002:ac0:a581:0:0:0:0:0 with SMTP id m1-v6csp6847412imm; Wed, 27 Jun 2018 14:39:41 -0700 (PDT) X-Google-Smtp-Source: AAOMgpcVr7rYocbRePPBqQrP/lJ76Jp31+3LDhBlN1Hto5QaEAyXUgZdgTregxFHG5RqBLBQJTyp X-Received: by 2002:a62:d455:: with SMTP id u21-v6mr7562547pfl.143.1530135581487; Wed, 27 Jun 2018 14:39:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1530135581; cv=none; d=google.com; s=arc-20160816; b=Qp7K+kxtboqQmriPv4qVGEhckAOfJ7IY4yJstiQQsGj6lCGXMQIcXZiW3B5cfXSaOu +O1dSCrf/j+YskyT7UdQvl4vJ/31f+4MWccO3sg2rNUKAzJrc4RnMAtC0L7XGd0MMeUy zmTBHz2M89TSez8FfBNlkU8SEKxeH1+KMHWqlSEYG7xusY6dW31uLCwyqgCMkturskqC +8/06wnIj6ZWTW8ZWnT6ziWrHfBbCZeFHEZ47RdNPGT/WrePHmy5DcjDU1sZStOxokWs Iwc6FeWmFrvibvg0PT5ufTQKPFxFhBGM8atC/CxxSi+H36Y0kC3Nzm8DxnJzvk9/Jqlb JbJw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:arc-authentication-results; bh=uQL2BmpgOuRbOEEJzxIMW6v0tzLkHsqelgI7AceDHhY=; b=JRbuha/Dbc21htzdvHzV+HCsWLxK8AZm7SydlXVnGtxz+d1hgLcciKDv5GshDgHqSX +vuWWbZLvpsy5raaxbSce3MYgMMYodl4b7HTfVMMBDvPCGOUQjngvPeADoUXfW8rOKtH QB+HF0d89UARzQJGUkWeLlPFYcC0WJj31ZHDRNOhiI9OpQ3xikHyu+3BWsCJkPIx/nwv /m0130apy/m1FDbQFYwqltfXmzbJqv6rGmSmT8NcqPdkkLHXBNlJgT11Wh6JkBokCC1o 77DG+7704Q+1aQuAs45HduN7VGL3yUNl81OFkUG96GiA3RpomBon+RpDN+fMUnZHQyUZ WF/A== ARC-Authentication-Results: i=1; mx.google.com; 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=fail (p=NONE sp=NONE dis=NONE) header.from=collabora.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p61-v6si4942054plb.472.2018.06.27.14.39.27; Wed, 27 Jun 2018 14:39:41 -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; 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=fail (p=NONE sp=NONE dis=NONE) header.from=collabora.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965874AbeF0VO5 (ORCPT + 99 others); Wed, 27 Jun 2018 17:14:57 -0400 Received: from bhuna.collabora.co.uk ([46.235.227.227]:55864 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965801AbeF0VOz (ORCPT ); Wed, 27 Jun 2018 17:14:55 -0400 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: eballetbo) with ESMTPSA id 83AB1263BFB From: Enric Balletbo i Serra To: Sandy Huang , =?UTF-8?q?Heiko=20St=C3=BCbner?= , David Airlie Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-rockchip@lists.infradead.org, Gustavo Padovan , Tomasz Figa , Sean Paul , kernel@collabora.com, =?UTF-8?q?St=C3=A9phane=20Marchesin?= Subject: [RFC RESEND PATCH] drm/rockchip: update cursors asynchronously through atomic. Date: Wed, 27 Jun 2018 23:14:47 +0200 Message-Id: <20180627211447.20927-1-enric.balletbo@collabora.com> X-Mailer: git-send-email 2.18.0 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Add support to async updates of cursors by using the new atomic interface for that. Signed-off-by: Enric Balletbo i Serra --- I am sending this as RFC because I still don't have a deep knowledge of the hw and I am not sure if the vop_plane_update function can be reused in both cases, atomic_updates and atomic_async_updates. I think that someone with more knowledge should take a look. The patch was tested on a Samsung Chromebook Plus in two ways. 1. Running all igt kms_cursor_legacy and kms_atomic@plane_cursor_legacy tests and see that there is no regression after the patch. 2. Running weston using the atomic API. Best regards, Enric drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 80 ++++++++++++++++----- 1 file changed, 64 insertions(+), 16 deletions(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c index 53d4afe15278..1eb6bda924af 100644 --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c @@ -688,8 +688,7 @@ static void vop_plane_atomic_disable(struct drm_plane *plane, spin_unlock(&vop->reg_lock); } -static void vop_plane_atomic_update(struct drm_plane *plane, - struct drm_plane_state *old_state) +static void vop_plane_update(struct drm_plane *plane) { struct drm_plane_state *state = plane->state; struct drm_crtc *crtc = state->crtc; @@ -710,20 +709,6 @@ static void vop_plane_atomic_update(struct drm_plane *plane, bool rb_swap; int format; - /* - * can't update plane when vop is disabled. - */ - if (WARN_ON(!crtc)) - return; - - if (WARN_ON(!vop->is_enabled)) - return; - - if (!state->visible) { - vop_plane_atomic_disable(plane, old_state); - return; - } - obj = rockchip_fb_get_gem_obj(fb, 0); rk_obj = to_rockchip_obj(obj); @@ -794,10 +779,73 @@ static void vop_plane_atomic_update(struct drm_plane *plane, spin_unlock(&vop->reg_lock); } +static void vop_plane_atomic_update(struct drm_plane *plane, + struct drm_plane_state *old_state) +{ + struct drm_plane_state *state = plane->state; + struct vop *vop = to_vop(state->crtc); + + /* + * can't update plane when vop is disabled. + */ + if (WARN_ON(!state->crtc)) + return; + + if (WARN_ON(!vop->is_enabled)) + return; + + if (!state->visible) { + vop_plane_atomic_disable(plane, old_state); + return; + } + + vop_plane_update(plane); +} + +static int vop_plane_atomic_async_check(struct drm_plane *plane, + struct drm_plane_state *state) +{ + struct drm_crtc_state *crtc_state; + + crtc_state = drm_atomic_get_existing_crtc_state(state->state, + state->crtc); + if (WARN_ON(!crtc_state)) + return -EINVAL; + + if (!crtc_state->active) + return -EINVAL; + + if (plane->state->crtc != state->crtc || + plane->state->src_w != state->src_w || + plane->state->src_h != state->src_h || + plane->state->crtc_w != state->crtc_w || + plane->state->crtc_h != state->crtc_h || + !plane->state->fb || + plane->state->fb != state->fb) + return -EINVAL; + + return 0; +} + +static void vop_plane_atomic_async_update(struct drm_plane *plane, + struct drm_plane_state *new_state) +{ + plane->state->src_x = new_state->src_x; + plane->state->src_y = new_state->src_y; + plane->state->crtc_x = new_state->crtc_x; + plane->state->crtc_y = new_state->crtc_y; + plane->state->fb = new_state->fb; + *plane->state = *new_state; + + vop_plane_update(plane); +} + static const struct drm_plane_helper_funcs plane_helper_funcs = { .atomic_check = vop_plane_atomic_check, .atomic_update = vop_plane_atomic_update, .atomic_disable = vop_plane_atomic_disable, + .atomic_async_check = vop_plane_atomic_async_check, + .atomic_async_update = vop_plane_atomic_async_update, }; static const struct drm_plane_funcs vop_plane_funcs = { -- 2.18.0