From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-12.0 required=3.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED,DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7B395C43603 for ; Tue, 13 Apr 2021 11:37:04 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 171F161370 for ; Tue, 13 Apr 2021 11:37:04 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 171F161370 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 3FB5F89DBC; Tue, 13 Apr 2021 11:37:02 +0000 (UTC) Received: from mail-wm1-x336.google.com (mail-wm1-x336.google.com [IPv6:2a00:1450:4864:20::336]) by gabe.freedesktop.org (Postfix) with ESMTPS id 30D7E89D57; Tue, 13 Apr 2021 11:37:01 +0000 (UTC) Received: by mail-wm1-x336.google.com with SMTP id s10-20020a05600c318ab029012d7c83637bso19903wmp.0; Tue, 13 Apr 2021 04:37:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=d8prWghJv+BkiPsQ41cXnPAO/NBhcn05YDXuxcrhtEc=; b=iff6X5Dx5VtccL1uM8p2zLSXcrty+PuUGC7J7z8zXEhInb/+xp6sCmr4WpXqZMvgb8 U/sI9sbF3KF8QWGS2ZH58/wbAC/7sFyazmiDGUsMMbiZdsdaQsyFRpSyaG/tncNYD7s3 b6F6wj3u3w1W3ik+XIzTB254U47VrGsIrfJCf1oonuFB3mpok5f2SQ1oAuc1yPmn3Ski DYYNM2390gZtJEKQy/7PKGmwKrmIIzmf+Qjm0+BnJb5dxaY9IWr3JG1W0V+CDbW4edn9 5m022ytrZ/AQR+nTw4hpDAfaI8luLhP1/kebH6wChvq4bm0gWtj+Dc6Q+lYkhQqBRZyR sW8A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=d8prWghJv+BkiPsQ41cXnPAO/NBhcn05YDXuxcrhtEc=; b=L3CrXmLH3EYfBbaYsIqRS8atoX3dfLJTb76H93pNGf3AuMMZxoCuXYWZ4A6x03w/c8 Gsd3uofJvsBILT8Du1iaBrTxRUeZoxTUGBE4pDdlIYVjgI1P9KMRgYELpu7JCy2vzFF3 N+N6/prxt0H1wlFXwZvgy598yF8WoqUeV+fO8WGWxUO++tCCX7eOM4SV2Rx4+S3vDVSc xjEKds0ZHZc40DWUDMzFDRgA7V9mPc/1gXV+vVOM0uJa2+7DTPvhtJD7upnhFkD+pTmZ kPv/LMdlb87IenZ9UNob+AJ3yotMCgo50D46zDloyUV6WyNHXdiT4rw16TyInwDERLQ6 5aLQ== X-Gm-Message-State: AOAM530fls2TbPezvH9Wk7s4ud3Eg/IhO80ysMJWrZI8Uqh4IbNReL7/ yMHbyX9LFCTjZmfNeVNs1pI= X-Google-Smtp-Source: ABdhPJxeKrK3zuVlsI2tuMeQzuZfRj5oEbBzKypg8YvEhMsDsu8BDty5SEpGMzwepcRaCUlSBf8+IA== X-Received: by 2002:a1c:7311:: with SMTP id d17mr3559703wmb.183.1618313819915; Tue, 13 Apr 2021 04:36:59 -0700 (PDT) Received: from localhost ([62.96.65.119]) by smtp.gmail.com with ESMTPSA id v18sm2141913wmh.28.2021.04.13.04.36.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 13 Apr 2021 04:36:58 -0700 (PDT) Date: Tue, 13 Apr 2021 13:37:38 +0200 From: Thierry Reding To: Daniel Vetter Subject: Re: [PATCH 10/12] drm/tegra: Don't set allow_fb_modifiers explicitly Message-ID: References: <20210413094904.3736372-1-daniel.vetter@ffwll.ch> <20210413094904.3736372-10-daniel.vetter@ffwll.ch> MIME-Version: 1.0 In-Reply-To: <20210413094904.3736372-10-daniel.vetter@ffwll.ch> User-Agent: Mutt/2.0.6 (98f8cb83) (2021-03-06) X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Pekka Paalanen , Intel Graphics Development , stable@vger.kernel.org, Jonathan Hunter , DRI Development , linux-tegra@vger.kernel.org, Daniel Vetter Content-Type: multipart/mixed; boundary="===============1619246116==" Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" --===============1619246116== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="3zY2OYTARJAQNWvh" Content-Disposition: inline --3zY2OYTARJAQNWvh Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Apr 13, 2021 at 11:49:01AM +0200, Daniel Vetter wrote: > Since >=20 > commit 890880ddfdbe256083170866e49c87618b706ac7 > Author: Paul Kocialkowski > Date: Fri Jan 4 09:56:10 2019 +0100 >=20 > drm: Auto-set allow_fb_modifiers when given modifiers at plane init >=20 > this is done automatically as part of plane init, if drivers set the > modifier list correctly. Which is the case here. >=20 > It was slightly inconsistently though, since planes with only linear > modifier support haven't listed that explicitly. Fix that, and cc: > stable to allow userspace to rely on this. Again don't backport > further than where Paul's patch got added. >=20 > Cc: stable@vger.kernel.org # v5.1 + > Cc: Pekka Paalanen > Signed-off-by: Daniel Vetter > Cc: Thierry Reding > Cc: Jonathan Hunter > Cc: linux-tegra@vger.kernel.org > --- > drivers/gpu/drm/tegra/dc.c | 10 ++++++++-- > drivers/gpu/drm/tegra/drm.c | 2 -- > 2 files changed, 8 insertions(+), 4 deletions(-) >=20 > diff --git a/drivers/gpu/drm/tegra/dc.c b/drivers/gpu/drm/tegra/dc.c > index c9385cfd0fc1..f9845a50f866 100644 > --- a/drivers/gpu/drm/tegra/dc.c > +++ b/drivers/gpu/drm/tegra/dc.c > @@ -959,6 +959,11 @@ static const struct drm_plane_helper_funcs tegra_cur= sor_plane_helper_funcs =3D { > .atomic_disable =3D tegra_cursor_atomic_disable, > }; > =20 > +static const uint64_t linear_modifiers[] =3D { > + DRM_FORMAT_MOD_LINEAR, > + DRM_FORMAT_MOD_INVALID > +}; > + > static struct drm_plane *tegra_dc_cursor_plane_create(struct drm_device = *drm, > struct tegra_dc *dc) > { > @@ -987,7 +992,7 @@ static struct drm_plane *tegra_dc_cursor_plane_create= (struct drm_device *drm, > =20 > err =3D drm_universal_plane_init(drm, &plane->base, possible_crtcs, > &tegra_plane_funcs, formats, > - num_formats, NULL, > + num_formats, linear_modifiers, > DRM_PLANE_TYPE_CURSOR, NULL); > if (err < 0) { > kfree(plane); > @@ -1106,7 +1111,8 @@ static struct drm_plane *tegra_dc_overlay_plane_cre= ate(struct drm_device *drm, > =20 > err =3D drm_universal_plane_init(drm, &plane->base, possible_crtcs, > &tegra_plane_funcs, formats, > - num_formats, NULL, type, NULL); > + num_formats, linear_modifiers, > + type, NULL); I think we can do better than linear_modifiers for overlay planes, but given that this doesn't change existing behaviour, I'll do that in a separate patch. Acked-by: Thierry Reding --3zY2OYTARJAQNWvh Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAmB1goAACgkQ3SOs138+ s6HiPg/9F2IhGz7bj0R3eFPU3cBNVrP3Fi7Q48YLUZofkiN0ka7nGaDGknhgaupF bE3N2Yk9Qp+tsN82z3R7PsFfmmPfy3uBKp8lcXV2jMJtb/sqH+JmRZAUl3wzT0qb ZwnaBs5xjoPATt8oFC8/8Y1sQ2GW7+6yWCWIaVC8PqiQ5g78L6gsu2N7Ig/rB+P6 VAI6t1MGjdfiF8lrMF3+kkvbpxrqIybiAHXVhsUxViXH88AWq56lu9LRUWZTk3T9 eLyBcbyth7XrO4KJOMuOFqXYjPn8f7dKittgCwSacGkItWmQv0+GBrdFeA+i0WZW b0SRu1QdGY9ClU0Z+rs9YCRV9P853vCUQs2E6rPEpO9Ul8tP/OKoGu0/DUFV98gJ iReAe4bP6CSJN96oJ+FUunvOyJ40gGWToWCmZ37fM1Vt/qH2+LkPAkkbdAPkCqBx 1frCx2qFhxRNL+laXN7tkkmjg+ZPBqK8VjEQJ+dl5gBQsM8g1o4cliI6KSKgjm4B oAArUFz3u9zYzT0sFuqj+lyh9/Tu+gevrUNkcyAkUrngNaEZkCgSL6A6rJLKSOLY ZH964cF3UDvzZGwuOaVgR9agCBXh2i/WFa1AtE6HsvB/HELLxg3nUHYe5CBAGrjM 4BLaGiVxVaYx73qjnaB2LKcqybVO8S+NEP48pWjwnGXz1fu0z/8= =0rrw -----END PGP SIGNATURE----- --3zY2OYTARJAQNWvh-- --===============1619246116== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel --===============1619246116==--