Received: by 2002:a89:d88:0:b0:1fa:5c73:8e2d with SMTP id eb8csp1928754lqb; Mon, 27 May 2024 02:04:30 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCWP2Xu+Bn1mNzfytzCUIm22ysjvSMI80ClvrvQ8ogISeQoYVDdj0tAWI0N5lKb4MzemIcVcZsYOBjF/So8e4IWKmQ670o4Q3P6YGOjR1g== X-Google-Smtp-Source: AGHT+IGOf0IsTuYd0umYBIEvat5Ybo+HgU1Bjv6/IwJ5AYC06Up7ufUgH+5HKUr30/N8ryM/jPWl X-Received: by 2002:a17:902:e5d0:b0:1f4:9474:e44d with SMTP id d9443c01a7336-1f4947509b0mr36580715ad.21.1716800670525; Mon, 27 May 2024 02:04:30 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1716800670; cv=pass; d=google.com; s=arc-20160816; b=ShI7QVUqWeldIdQ4p+nUfG7oQx7lQfQtKduGrOBu/eizkF/xELsgv7tWQFIlpcdCxy ZlYsE9IEXsFU0Rj1mSyEZMRPKtC0O60nj3FHA07I9xU+YqR0aFlPB5wKPfhjtarGmXQm 1e90BD5YEVDcZRaQlghILdC5uAIXBIeC9BWf6TvbqNhAoE7Yd61+jUqoGDo/zimLwxF2 B8WTXW9bkTXmmpA7KkF6+B6zKwh+2m1XCBamC8rWm8883ddLy9tKa2CIG84AtxnYV7EK 4rKp4tCW3r5DwjOwfzeg2E9LeJCpovzcIFFVP1f6yxnPR+YdmuNstwCwVIz83Iv2wnIn owZg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=03fWaQYzaZV/c1B0t2NogE6hpI6rV1xFosMoQFHu0ec=; fh=h4X/nf4IW7NfPdiqlv4AGCn2SUNlaTA8xb6pbxt6Twg=; b=k2OW5Mtvv6dy+kNikRE867AEake+TGMZJA44XbPtekxszAPxv1n5XXZmUeBj7Wr4wa X2521vGFZdORLalIx2GUWTDBmr3SkzlO4ilKbWocrbEot5Lo01+E40lCJE4ubmTSm8dg Bo1U75lpnen41SAZSZ45uB1/yLLnjy2tH1l7P3V7lMdpRSDy3ojF1b0SusoR3QCoHUkJ SEKyPPrbJBUNLOvp3oAfcym+/w+N5X0cejuIosL1c84iXw2lEWKZpjZqqUiSwey8Vbvm 3Bv8cCgRVGr3vZDeoqsh+YHsiMFH7lB6t8qxjf3cIZE3jmjH2l+js0L8YkxXC+nN/aE9 tHXA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=TRMPiNiI; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-190281-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-190281-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [2604:1380:40f1:3f00::1]) by mx.google.com with ESMTPS id d9443c01a7336-1f44c970f57si57114585ad.274.2024.05.27.02.04.29 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 27 May 2024 02:04:30 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-190281-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=TRMPiNiI; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-190281-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-190281-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.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 sy.mirrors.kernel.org (Postfix) with ESMTPS id 22488B21914 for ; Mon, 27 May 2024 09:02:30 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 31072763EC; Mon, 27 May 2024 09:02:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="TRMPiNiI" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 43B121A2C35; Mon, 27 May 2024 09:02:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716800537; cv=none; b=AFnNrIlAWRYd6C2XWAEU86VTnnBmgUlk6pwyJjyqWAoyxVjBwVuVt5X01lszfQ42OKSfaeP/j4jI6Mdi+fVAqeuNiGxLUNtMmxrPN4hIAjAg95WA+RdxyPw8bMMlBXsVGsSr11JJaez7w4q7jM1EiyxaxJUkoAIW9BipUxuy/CI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716800537; c=relaxed/simple; bh=uYAFBgtM5vXI9g7/LeVQ+pczQ1yooR+keQpxvhcuA8k=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=IZdjyuWLoq6ME7fBeVZyjbZ2iamWHglfoZ91kYg8aftspZaf3qB2ZrFM2LvuKihQ/xANSsLSHgQUBbMIAD2id36ig8jjtdhrDAVeARwK4OiDvXGatlHL091CPcN4TfFwmiLJrka8PEfIl7XY6mHgYB8+b7bGE9MIyEywyU6Q0NM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=TRMPiNiI; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4A944C2BBFC; Mon, 27 May 2024 09:02:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1716800536; bh=uYAFBgtM5vXI9g7/LeVQ+pczQ1yooR+keQpxvhcuA8k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=TRMPiNiIHap5cUpLkC3niSfLnS5r5ay/7/AKD7dAXmxw4VtzFUte71pZ8s4wYxN6C 5N7gB4+obnBGFvTNNxSuXzK/U5TbAzL9ZplGMilaZr1KdblJ6LczlqxIQ8GuuBy7nc 9i1Xee/2KLe/dEp3NpfRWXYMXpQGjWGZRqeXwpsjJf83/GO2GxAql5Y4nbHCV2YGjz rYh02qQutduLCSqGeaeXpUEYU9uSWnlHpo/iwn2FqW2k+BzSyYmty//oZ6FRorGIBN +xcOBIRc0ZuQyvFii3U1AO5zNKn47fmXCnneBR7Hq+ycIaKg4tSKiWLgwSvWA5gvkB +hbDCAwtJxCnQ== Date: Mon, 27 May 2024 11:02:13 +0200 From: Maxime Ripard To: Dmitry Baryshkov Cc: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Daniel Vetter , Jonathan Corbet , Sandy Huang , Heiko =?utf-8?Q?St=C3=BCbner?= , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Andy Yan , Hans Verkuil , Sebastian Wick , Ville =?utf-8?B?U3lyasOkbMOk?= , 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 Subject: Re: [PATCH v14 17/28] drm/connector: hdmi: Add Broadcast RGB property Message-ID: <20240527-adamant-just-spaniel-ebed4e@houat> References: <20240521-kms-hdmi-connector-state-v14-0-51950db4fedb@kernel.org> <20240521-kms-hdmi-connector-state-v14-17-51950db4fedb@kernel.org> <4n55dbl2h275z5ubebigri4xjtjsvt44w62n656srdgndcav2j@rrq5haucqx5u> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha384; protocol="application/pgp-signature"; boundary="wqcwbvnihqwt2l7e" Content-Disposition: inline In-Reply-To: <4n55dbl2h275z5ubebigri4xjtjsvt44w62n656srdgndcav2j@rrq5haucqx5u> --wqcwbvnihqwt2l7e Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable 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 there to > > support it. > >=20 > > Both drivers implementing the same property with the same semantics, > > 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 for it. > >=20 > > Let's plumb it into the newly created HDMI connector. > >=20 > > 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(-) > >=20 > > diff --git a/Documentation/gpu/kms-properties.csv b/Documentation/gpu/k= ms-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,Prope= rty Values,Object attached,De > > ,,=E2=80=9Csaturation=E2=80=9D,RANGE,"Min=3D0, Max=3D100",Connector,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=3D0xfff= fffff",Connector,property to suggest an X offset for a connector > > ,,=E2=80=9Csuggested Y=E2=80=9D,RANGE,"Min=3D0, Max=3D0xffffffff",Conn= ector,property to suggest an Y offset for a connector > > ,Optional,"""aspect ratio""",ENUM,"{ ""None"", ""4:3"", ""16:9"" }",Co= nnector,TDB > > -i915,Generic,"""Broadcast RGB""",ENUM,"{ ""Automatic"", ""Full"", ""Li= mited 16:235"" }",Connector,"When this property is set to Limited 16:235 an= d CTM is set, the hardware will be programmed with the result of the multip= lication of CTM by the limited range matrix to ensure the pixels normally i= n the range 0..1.0 are remapped to the range 16/255..235/255." >=20 > 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? Maxime --wqcwbvnihqwt2l7e Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iJUEABMJAB0WIQTkHFbLp4ejekA/qfgnX84Zoj2+dgUCZlRMEAAKCRAnX84Zoj2+ dhGAAYDS1llqP+ICZyWXx+g6L1EiXrG7OIB0z7t7KF3yJW5Y1zN4+dIvVNhFS6LU gAXKTp4BgKUepq65FIlFjrRN3gyXTVxmbiIMJd5mPgjySS4scgX65VNBO5cc0Qvt qThCHUrQ2A== =K55A -----END PGP SIGNATURE----- --wqcwbvnihqwt2l7e--