Received: by 2002:a89:d88:0:b0:1fa:5c73:8e2d with SMTP id eb8csp2227317lqb; Mon, 27 May 2024 12:01:45 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCXM/EUZMetysyEaqPFz75O42btFSVxsLTjYNWXU0Fn0mLX4CKyPILx71syT5I83P3ReA8bANiZZKnZucHuMmitVwOxYsQKiA89G6nDg3g== X-Google-Smtp-Source: AGHT+IEf79YXxAMcqIYqCr0bTaG3uHPBNLQrFcai6KrcMOlRkYFhSjBuN1/SO0cLdouzFcS4xDVd X-Received: by 2002:ac2:4433:0:b0:51a:d08d:bab4 with SMTP id 2adb3069b0e04-52966e9b049mr8083348e87.55.1716836505216; Mon, 27 May 2024 12:01:45 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1716836505; cv=pass; d=google.com; s=arc-20160816; b=aIQszPtCn3i3oJqR83Xz7pTVWPCWaHW9PIwz0Ig5VK/jVF8fnfW5ltKl8mFPaMx9X/ af0g3i4A2w3JUuklBXYTujqK/UkA8yArYH/vvsP0jdwPtChNdge1D10fBEyEM+1sFb+A awC4P2t68yXV9o/7ciwake8Lw4764zY0cnpU/a11p9b2sEjYOKKHaJlRy2jU1BNVp8qn +4sjZcc6mxYpmLeXTzPpGHrshRiNAoTxiBAa5Zj+l8gy7QL6TxdfM+eSUEJbfV4850ms bdW6cKBUxiSlSn4bpVV9Yn06UpQTqoT025lGKv6Xs5ThgABvB+YeZRj3M86UutGyKz/d EymA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:list-unsubscribe:list-subscribe :list-id:precedence:dkim-signature; bh=6Ja7vlfEqAriKa+FwY5Dmg/udaegLnmPF1+eRGrrskg=; fh=JRZKAiZoV3e+571j8r/V+Dvdaqpvz25w+GwAMEJ8s5E=; b=qssh756jYFF7levvfC4ESDR1a0QXTu1dCjJAy8Vti2tr3RPuBfgD+rdkqNV1B6P7Y5 y6YtVNir8kapS0R+l16I3TcTXtH+180FflgofvwI2kRhQD55nsmqfBy97zbpzp1TUyIa A8rhpwS5fphBZ+6oDgH1EXFZfzhkfZVuJrfN4yHQ4ganJtD+Udvtbg5zOB0cMc6+21l2 2kvT0luIHJAFZUqrXGOq+LiepLDSVtlKP2OcUicye+HiPGintWpDic9bO2KblDS5Th/M FsK8p0kAwKfKgAASrcbkwJMpczRVl/KPlwHe7MfY8mvHscTlLJblp63ANpnDYQd/XfF0 ivBQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=x6Lx3xLL; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-191354-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-191354-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id a640c23a62f3a-a626cc37323si416602166b.361.2024.05.27.12.01.45 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 27 May 2024 12:01:45 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-191354-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=x6Lx3xLL; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-191354-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-191354-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id B91A51F22629 for ; Mon, 27 May 2024 19:01:44 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id DB69B15FD12; Mon, 27 May 2024 19:00:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="x6Lx3xLL" Received: from mail-yb1-f178.google.com (mail-yb1-f178.google.com [209.85.219.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 464BD161313 for ; Mon, 27 May 2024 19:00:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716836450; cv=none; b=grUBe1UeaMECOLHDJCbP1SNcwiRDmADZFWFna4U4YXohL0EIk7dHhc+TByp1zNfQgmrHr5GZQvm2TQQ44VgWw48n+s3JQ3Lh5ov9JDBAsgfNHSN4EsNGEGWslMSgVdI5vxpQcxVUuKp+NIwkeb1u9ojMTG7cJ1iYvfAml5/wj7s= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716836450; c=relaxed/simple; bh=QtTJK4e+xezrmSqjZutOBCe4DS0fPdLTmIIG4CLzy38=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=hYq4TECypH5+JkG5tQ1z8Wfd3HR7smzcm9IRdCD848GY2rrNuBHP6jxPYIyR4Eooe6eBwg7iM2qAGiyp1P1TCHUVKDSrNxHSodhofkZktKCwdnlUDa5Fcbm2ZGE6m7hbP0sU8SBQQq+C4P+Tjb7rZAl34YTgeUDKulPlsrseHtI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=x6Lx3xLL; arc=none smtp.client-ip=209.85.219.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Received: by mail-yb1-f178.google.com with SMTP id 3f1490d57ef6-df771fa2b8bso62992276.2 for ; Mon, 27 May 2024 12:00:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1716836448; x=1717441248; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=6Ja7vlfEqAriKa+FwY5Dmg/udaegLnmPF1+eRGrrskg=; b=x6Lx3xLL8+EQaTx+zSZWL1QUgU5MY1zgVdw8T1xlhHKEWQcXKRdxB/wq6UATfjK23O tyxIw/roNaAXLIk254luWcJXamEqqdBrmg5VxkO2jprdJmCkDazzb7Vd4SWh4J9YC3Sr VHfwzuAYA0IktWtjhUlwrFvohWQWJrp4SWPnPSFTqgQIAmAzPEegs7wV3tyMUzwQyKpc CxLAs5986CP3NfN3Z6Dcok7l6v9PfKt8pXGSanou3dudkkQKIPJ+yKY2WGaPrmKM8u8t ZUeAz+UzrQYH4Nsjn0DBsA5ZL/lbYML9z+zMTSJ04L32u1xSzjrWIFZAXUYtHu6GIXiW qvLQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1716836448; x=1717441248; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=6Ja7vlfEqAriKa+FwY5Dmg/udaegLnmPF1+eRGrrskg=; b=qWNBlP7ehnEMGJ49q9sGvdBKpLjo7kDMH65FXAy4HdGO75tEi3LwhAszKqgnP0jFdU RNNsGAudfKA4jw6YqZdTbSNLP1i/QLozQL+ys0qdX2kuY92uKIIzNEOt91P4FBE5hptK pLB0zF+KPJgCe9hiYgJYGYg9uyap5rkdycv2z5YAQqJXxqDS9u79MNZf5MDWSlKryHve 5/R+QKbjfbsRKFJPGQbJiLom8ybthwS/Sy3DDSv8yh3iZl9Ghi/WeoymHdCVlKxmoPaH b4bW0IoEO8lnxMR6cR2xtwvrn9xNnnNYMBQAx2ikOnEPRFth+BGJ5mxlhSfnc7v5mKJZ EP6g== X-Forwarded-Encrypted: i=1; AJvYcCWqHcVKqA1R3yz2PGCRGj6sM1S4seo7EZGTGw6OAqoNV7KrxIzYL0rZytDYFVOwR7hsLqR4aD0U2XQsIxc8i+Mr3L9DkGpmLgMJ0JRd X-Gm-Message-State: AOJu0Yyi3rmrtnz+3ewgYgL3GrqCd77JXFPQOOc5Us/JGKKSXE0jIgmw L4CbiilZGHRXRSkYwbIwRxIc8fvpafC+cB1bUPXu4zs43w3r3ziFlT606lBxoqS+2D0kvUqBn7+ x8OWm9j0yqk6oFtxDsk87gSWyMYsQFAK6wHOofg== X-Received: by 2002:a25:86c7:0:b0:df7:94d3:5555 with SMTP id 3f1490d57ef6-df794d357fcmr5601696276.41.1716836447989; Mon, 27 May 2024 12:00:47 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20240521-kms-hdmi-connector-state-v14-0-51950db4fedb@kernel.org> <20240521-kms-hdmi-connector-state-v14-17-51950db4fedb@kernel.org> <4n55dbl2h275z5ubebigri4xjtjsvt44w62n656srdgndcav2j@rrq5haucqx5u> <20240527-adamant-just-spaniel-ebed4e@houat> <20240527-colorful-orange-bug-bbeefe@houat> In-Reply-To: <20240527-colorful-orange-bug-bbeefe@houat> From: Dmitry Baryshkov Date: Mon, 27 May 2024 22:00:36 +0300 Message-ID: Subject: Re: [PATCH v14 17/28] drm/connector: hdmi: Add Broadcast RGB property To: Maxime Ripard Cc: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Daniel Vetter , Jonathan Corbet , Sandy Huang , =?UTF-8?Q?Heiko_St=C3=BCbner?= , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Andy Yan , Hans Verkuil , Sebastian Wick , =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-sunxi@lists.linux.dev, Dave Stevenson , Pekka Paalanen Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, 27 May 2024 at 16:30, Maxime Ripard wrote: > > Hi, > > On Mon, May 27, 2024 at 12:43:18PM GMT, Dmitry Baryshkov wrote: > > On Mon, May 27, 2024 at 11:02:13AM +0200, Maxime Ripard wrote: > > > Hi, > > > > > > Thanks again for that thorough review :) > > > > > > On Thu, May 23, 2024 at 01:22:56PM GMT, Dmitry Baryshkov wrote: > > > > On Tue, May 21, 2024 at 12:13:50PM +0200, Maxime Ripard wrote: > > > > > The i915 driver has a property to force the RGB range of an HDMI = output. > > > > > The vc4 driver then implemented the same property with the same > > > > > semantics. KWin has support for it, and a PR for mutter is also t= here to > > > > > support it. > > > > > > > > > > Both drivers implementing the same property with the same semanti= cs, > > > > > plus the userspace having support for it, is proof enough that it= 's > > > > > pretty much a de-facto standard now and we can provide helpers fo= r it. > > > > > > > > > > Let's plumb it into the newly created HDMI connector. > > > > > > > > > > Reviewed-by: Dave Stevenson > > > > > Acked-by: Pekka Paalanen > > > > > Reviewed-by: Sebastian Wick > > > > > Signed-off-by: Maxime Ripard > > > > > --- > > > > > Documentation/gpu/kms-properties.csv | 1 - > > > > > drivers/gpu/drm/display/drm_hdmi_state_helper.c | 4 +- > > > > > drivers/gpu/drm/drm_atomic.c | 2 + > > > > > drivers/gpu/drm/drm_atomic_uapi.c | 4 ++ > > > > > drivers/gpu/drm/drm_connector.c | 88 +++++++++++= ++++++++++++++ > > > > > include/drm/drm_connector.h | 36 ++++++++++ > > > > > 6 files changed, 133 insertions(+), 2 deletions(-) > > > > > > > > > > diff --git a/Documentation/gpu/kms-properties.csv b/Documentation= /gpu/kms-properties.csv > > > > > index 0f9590834829..caef14c532d4 100644 > > > > > --- a/Documentation/gpu/kms-properties.csv > > > > > +++ b/Documentation/gpu/kms-properties.csv > > > > > @@ -15,11 +15,10 @@ Owner Module/Drivers,Group,Property Name,Type= ,Property Values,Object attached,De > > > > > ,,=E2=80=9Csaturation=E2=80=9D,RANGE,"Min=3D0, Max=3D100",Connec= tor,TBD > > > > > ,,=E2=80=9Chue=E2=80=9D,RANGE,"Min=3D0, Max=3D100",Connector,TBD > > > > > ,Virtual GPU,=E2=80=9Csuggested X=E2=80=9D,RANGE,"Min=3D0, Max= =3D0xffffffff",Connector,property to suggest an X offset for a connector > > > > > ,,=E2=80=9Csuggested Y=E2=80=9D,RANGE,"Min=3D0, Max=3D0xffffffff= ",Connector,property to suggest an Y offset for a connector > > > > > ,Optional,"""aspect ratio""",ENUM,"{ ""None"", ""4:3"", ""16:9""= }",Connector,TDB > > > > > -i915,Generic,"""Broadcast RGB""",ENUM,"{ ""Automatic"", ""Full""= , ""Limited 16:235"" }",Connector,"When this property is set to Limited 16:= 235 and CTM is set, the hardware will be programmed with the result of the = multiplication of CTM by the limited range matrix to ensure the pixels norm= ally in the range 0..1.0 are remapped to the range 16/255..235/255." > > > > > > > > Should it still be defined as a generic property? > > > > > > I'm not sure what you mean here, sorry. It's being documented as a > > > connector property now, so it's very much still listed as a generic > > > property? > > > > I didn't perform my duty well enough and I didn't check the file for > > other instances of the property. Now I indeed see a generic "Broadcast > > RGB" property, but to me it looks like having a different set of values= : > > > > ,,"""Broadcast RGB""",ENUM,"{ =E2=80=9Coff=E2=80=9D, =E2=80=9Cauto=E2= =80=9D, =E2=80=9Con=E2=80=9D }",Connector,TBD > > That's not really what I meant: that file is deprecated now and it's not > where we document the properties anymore. This patch has improved that > documentation and moved it to the new place, and removed the deprecated > part. > > However, this line shouldn't be there at all. I'll add a patch to remove > it. SG > > Thanks! > Maxime --=20 With best wishes Dmitry