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.2 required=3.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED,DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 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 AE726C07E95 for ; Tue, 13 Jul 2021 07:54:22 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 7398C61279 for ; Tue, 13 Jul 2021 07:54:22 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7398C61279 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Y9tezBN499+t8VfEj8AfWl217bTb3XTug+goL3kLdcc=; b=vl2CyQoTNcH7G8B4ZQhsQYl5pm qulwBt9es6+ovFUZLthqNf9dixxNGigQ0MVlgp4WRg0IEA+F6OejF7cr/PV/LdQOuRfqejdDVApvZ 7biq9T+tBatNaX3Om/uyRobLqfkmIUyLkAGi4qCbBkJVI3YZ+Q78J81POAuLv+AfT5Fi4CWA+jxB3 GxpA5OcQeXBPZCPozEMpg00PJ8rhgU4sb+VhC4CA9gnfr29dvEs6cV0pwt/54DJdrzgpK2oNrxtqW 3qEASpBfscp4WhmmbdkKQiu4y48pwVMV8vKodzRZwf+WR2ekqHX9voA/5CY1Tz8LA19as9CWrW9MH iiA6LgZA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m3DDV-009Pek-4J; Tue, 13 Jul 2021 07:52:33 +0000 Received: from mail-lj1-x22c.google.com ([2a00:1450:4864:20::22c]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m3DDR-009PeP-LJ for linux-arm-kernel@lists.infradead.org; Tue, 13 Jul 2021 07:52:31 +0000 Received: by mail-lj1-x22c.google.com with SMTP id h19so17718139ljl.4 for ; Tue, 13 Jul 2021 00:52:28 -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:in-reply-to:references :mime-version; bh=umfPdu1lrU6B3okQ5unfcwiBa6kdfuAx9EF1yLDSE/k=; b=kqN1K0NCZuOLIo66DGBW7GvDESI7wJ3Vo7DAytXYWMbEkDjw4qFRe5kbnbKUBChtzz a3moe+7hMFmfPN4Y8dACZeQdQkDsP32F8RXcdtiH6B7CEVFdezNPNfMoqCu6czfxuv8s 4M0pf60qTKDi5iMmkrE3UgfY/1vL1HnXZXxi+x7RqXHtSQy2bSUGWrT5keFpGdiySVue hnhLj8C5cE2rHYoTbg36cB76Ekks9U9VlvpCvPiem5nHl0eD3rAINObvWWp9gruP/RU9 DrwVk8ko6BgWxBUCLyNZy91RhNKW6E73kqtnv60k3D+GbdpsYQOvI9c2C70QYcdZdh4u pSKQ== 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:in-reply-to :references:mime-version; bh=umfPdu1lrU6B3okQ5unfcwiBa6kdfuAx9EF1yLDSE/k=; b=VokIC+9hBmUbNGFFYG7JvskEaF4PW9Hognv2QgRkINijBtSyyA8HfHijw+pAQbs7eC HKn4IN5NyIzclaqspr26JsME+t9pjAHyX/Xsn808kAFyaGDBg1iauKBMrngUTPvhuoje rVqby3ORhWpcKeY+naTUWYUFECtjSLyr7DKf6CcNwYHmLmKgEB/9bHDjvshC2XuqDZaX 61QEIatrWYTBUeU06QB7247mh4zw6CLZDpHayzNXfW6fj9TgfJotLk6Dnb9+AwA/Hmnk JnP5ShBC+VdJqH7QeOP8gZ106Fbk4EPAgUL8kp2pqMBnZ+CzwifrUqG11m0Qq90LcqqT JVAQ== X-Gm-Message-State: AOAM5303rr1vRV9taVmmJgtCq9KQQvihUyW1b1bS8fYFwXe1dIdQGLtv eY9Gx0oxHZZJvwtPOnI2LIA= X-Google-Smtp-Source: ABdhPJymvkBefEBGF3M0rdNw7gmRquJu+GkPGC0OJkiXTav8YVwN0NE1AqUrDfq1zhWggpGqPVhLHw== X-Received: by 2002:a2e:99cc:: with SMTP id l12mr3082563ljj.358.1626162747533; Tue, 13 Jul 2021 00:52:27 -0700 (PDT) Received: from eldfell ([194.136.85.206]) by smtp.gmail.com with ESMTPSA id r5sm662018ljn.28.2021.07.13.00.52.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 13 Jul 2021 00:52:27 -0700 (PDT) Date: Tue, 13 Jul 2021 10:52:14 +0300 From: Pekka Paalanen To: Harry Wentland Cc: Raphael Gallais-Pou , "dri-devel@lists.freedesktop.org" , Maxime Coquelin , Benjamin Gaignard , Thomas Zimmermann , Raphael GALLAIS-POU , David Airlie , Yannick FERTRE - foss , Alexandre TORGUE - foss , "linux-kernel@vger.kernel.org" , Yannick FERTRE , Philippe CORNU - foss , Philippe CORNU , "linux-stm32@st-md-mailman.stormreply.com" , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH 1/2] drm: add crtc background color property Message-ID: <20210713105214.5730c959@eldfell> In-Reply-To: References: <20210707084557.22443-1-raphael.gallais-pou@foss.st.com> <20210707084557.22443-2-raphael.gallais-pou@foss.st.com> <20210709110459.79ca406a@eldfell> <3c8331cf-fded-b6e6-3e25-666634f4b87a@foss.st.com> <20210712110310.540df27d@eldfell> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210713_005229_774796_76838808 X-CRM114-Status: GOOD ( 35.12 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============2059027656228733230==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2059027656228733230== Content-Type: multipart/signed; boundary="Sig_/7sIoOLLn+H1OrBMUDsRwLWM"; protocol="application/pgp-signature"; micalg=pgp-sha256 --Sig_/7sIoOLLn+H1OrBMUDsRwLWM Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Mon, 12 Jul 2021 12:15:59 -0400 Harry Wentland wrote: > On 2021-07-12 4:03 a.m., Pekka Paalanen wrote: > > On Fri, 9 Jul 2021 18:23:26 +0200 > > Raphael Gallais-Pou wrote: > > =20 > >> On 7/9/21 10:04 AM, Pekka Paalanen wrote: =20 > >>> On Wed, 7 Jul 2021 08:48:47 +0000 > >>> Raphael GALLAIS-POU - foss wrote: > >>> =20 > >>>> Some display controllers can be programmed to present non-black colo= rs > >>>> for pixels not covered by any plane (or pixels covered by the > >>>> transparent regions of higher planes). Compositors that want a UI w= ith > >>>> a solid color background can potentially save memory bandwidth by > >>>> setting the CRTC background property and using smaller planes to dis= play > >>>> the rest of the content. > >>>> > >>>> To avoid confusion between different ways of encoding RGB data, we > >>>> define a standard 64-bit format that should be used for this propert= y's > >>>> value. Helper functions and macros are provided to generate and dis= sect > >>>> values in this standard format with varying component precision valu= es. > >>>> > >>>> Signed-off-by: Raphael Gallais-Pou > >>>> Signed-off-by: Matt Roper > >>>> --- > >>>> drivers/gpu/drm/drm_atomic_state_helper.c | 1 + > >>>> drivers/gpu/drm/drm_atomic_uapi.c | 4 +++ > >>>> drivers/gpu/drm/drm_blend.c | 34 +++++++++++++++++++= ++-- > >>>> drivers/gpu/drm/drm_mode_config.c | 6 ++++ > >>>> include/drm/drm_blend.h | 1 + > >>>> include/drm/drm_crtc.h | 12 ++++++++ > >>>> include/drm/drm_mode_config.h | 5 ++++ > >>>> include/uapi/drm/drm_mode.h | 28 +++++++++++++++++++ > >>>> 8 files changed, 89 insertions(+), 2 deletions(-) ... > >>> The question about full vs. limited range seems unnecessary to me, as > >>> the background color will be used as-is in the blending stage, so > >>> userspace can just program the correct value that fits the pipeline it > >>> is setting up. > >>> > >>> One more question is, as HDR exists, could we need background colors > >>> with component values greater than 1.0? =20 > >> > >> AR4H color format should cover that case, isn't it ? =20 > >=20 > > Yes, but with the inconvenience I mentioned. > >=20 > > This is a genuine question though, would anyone actually need > > background color values > 1.0. I don't know of any case yet where it > > would be required. It would imply that plane blending happens in a > > color space where >1.0 values are meaningful. I'm not even sure if any > > hardware supporting that exists. > >=20 > > Maybe it would be best to assume that only [0.0, 1.0] pixel value range > > is useful, and mention in the commit message that if someone really > > needs values outside of that, they should create another background > > color property. Then, you can pick a simple unsigned integer pixel > > format, too. (I didn't see any 16 bit-per-channel formats like that in > > drm_fourcc.h though.) > > =20 >=20 > I don't think we should artificially limit this to [0.0, 1.0]. As you > mentioned above when talking about full vs limited, the userspace > understands what's the correct value that fits the pipeline. If that > pipeline is FP16 with > 1.0 values then it would make sense that the > background color can be > 1.0. Ok. The standard FP32 format then for ease of use and guaranteed enough range and precision for far into the future? Or do you want to keep it in 64 bits total, so the UABI can pack everything into a u64 instead of needing to create a blob? I don't mind as long as it's clearly documented what it is and how it works, and it carries enough precision. But FP16 with its 10 bits of precision might be too little for integer 12-16 bpc pipelines and sinks? If the values can go beyond [0.0, 1.0] range, then does the blending hardware and the degamma/ctm/gamma coming afterwards cope with them, or do they get clamped anyway? Thanks, pq --Sig_/7sIoOLLn+H1OrBMUDsRwLWM Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEJQjwWQChkWOYOIONI1/ltBGqqqcFAmDtRi4ACgkQI1/ltBGq qqeaYxAAlKsXIdhtMGdhLiHKsFuM2ocvI/5alaiQD0SWSgB2OJuz7wltI5aV53Am LhPDY1h0zPnpT5v3WvvJZSSCHSSwlm5TkJX24DOuqwFJWINcmBYWQsMb0/ab7f76 fZERpWuaiB2rpOMOzAzgKMXI8Ss1gVVdztQmKZj3WrTwnu+6p2LAQn8NwYgH7Fi9 ZL8+acw2A3xPuT5gkgk8IosBVt5WGeeBY0R92uLRU47V7nzbtJADziLlkv+pM8dL 3o+Tdglgxnlb4SsEYwv/saJmAAe5Uyzst4vOBfI6BqbxegNeU+gsJUVqpTv5z+tj NoMVzxXIzZPY6vD0fgCvZWiEECR+kmc/CVY/kmwKjoZhaWRuCPEzNz2QjH3ETM9f AMl+V5/uLpvgpTlvi/HLrcapU89/7tw5drqjcmacygejDWz2BH+KWwB5uXYNRFkQ QzUhY0pV+fY4O+hMtNVWEfHjZr9qCzD9dyfDz5EUI5I1LJyAZLUnSmzWsXaNh+yA LmbJuLu3XtKECtGsTnSYWO52g8f/ihqE+vBgEzm+t05r3MSV/WXAaLrtWmWh4cUc wrQFi4EmblkUrd3gi2okv9AWZ3Gqzo25qYIMHswndLVdMc1QborQO1bTSctH9xFv aw8ajlVrK9YiZ6723kVTsYaWAGq4aalnOQw7mW2xdGxlWFvXI/I= =O57I -----END PGP SIGNATURE----- --Sig_/7sIoOLLn+H1OrBMUDsRwLWM-- --===============2059027656228733230== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============2059027656228733230==--