Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 23 Jun 2021 14:14:23 +0300
From:   Pekka Paalanen <ppaalanen@gmail.com>
To:     Werner Sembach <wse@tuxedocomputers.com>
Cc:     Simon Ser <contact@emersion.fr>, amd-gfx@lists.freedesktop.org,
        sunpeng.li@amd.com, intel-gfx@lists.freedesktop.org,
        linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
        airlied@linux.ie, tzimmermann@suse.de, rodrigo.vivi@intel.com,
        alexander.deucher@amd.com, christian.koenig@amd.com
Subject: Re: [PATCH v4 09/17] drm/uAPI: Add "active color range" drm
 property as feedback for userspace
Message-ID: <20210623141423.3f6c3e9f@eldfell>
In-Reply-To: <5947f768-dd87-6b39-ecfc-b7c4aaa88aca@tuxedocomputers.com>
References: <20210618091116.14428-1-wse@tuxedocomputers.com>
        <20210618091116.14428-10-wse@tuxedocomputers.com>
        <20210622100042.4041624a@eldfell>
        <56d079d4-841a-0ca5-b8a6-d2c10f91d211@tuxedocomputers.com>
        <k_QeCRpur3SNNIspBiTHOTT6Oj1pSteEO3MzyQFUG0lSXmEqJUI2tMXJ6wR9chzUHGahdQApJZrUxdJvIv6s8aXr2DBdoHg-2PsGH4kEMyA=@emersion.fr>
        <20210623103256.01d680ba@eldfell>
        <5947f768-dd87-6b39-ecfc-b7c4aaa88aca@tuxedocomputers.com>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="Sig_/gPEvcW1kQm+Ybq.N/v+Zg1q";
 protocol="application/pgp-signature"; micalg=pgp-sha256
Precedence: bulk

--Sig_/gPEvcW1kQm+Ybq.N/v+Zg1q
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

On Wed, 23 Jun 2021 12:17:40 +0200
Werner Sembach <wse@tuxedocomputers.com> wrote:

> Am 23.06.21 um 09:32 schrieb Pekka Paalanen:
> > On Tue, 22 Jun 2021 11:48:52 +0000
> > Simon Ser <contact@emersion.fr> wrote:
> > =20
> >> On Tuesday, June 22nd, 2021 at 11:50, Werner Sembach <wse@tuxedocomput=
ers.com> wrote:
> >> =20
> >>> Unknown is when no monitor is connected or is when the
> >>> connector/monitor is disabled.   =20
> >> I think the other connector props (link-status, non-desktop, etc) don't
> >> have a special "unset" value, and instead the value is set to a random
> >> enum entry. User-space should ignore the prop on these disconnected
> >> connectors anyways. =20
> > That sounds fine to me. =20
>=20
> Currently the only case for "not applicable" is when the monitor is
> disconnected, but sicne the properties are so interdependent, there
> might be a case in the future where e.g. a color format that has no
> differentiation between full and limited arises. When there is no
> special unset/not applicable option, the userspace has to know
> exactly when an option is valid or not, possible requiring additional
> logic.
>=20
> Setting a "not applicable" value allows userspace to be more dumb,
> without much hassle on the kernelspace side.

That's a good point too. So "not applicable" would be a value, but
"unknown" would not be.


Thanks,
pq

--Sig_/gPEvcW1kQm+Ybq.N/v+Zg1q
Content-Type: application/pgp-signature
Content-Description: OpenPGP digital signature

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEJQjwWQChkWOYOIONI1/ltBGqqqcFAmDTF48ACgkQI1/ltBGq
qqe5aBAAqjWygSQjmKD6D7wupf6+Ir2Pto1tmAW7ioCbPvk4RutUtuKzbT4WoBku
RtE2pHxKp2SiqKyo8rjRfgOR2gvDdOmfuKI5yfdKfPDZKhiyNIA2Wpyx9ntnFS+f
HGVdWVFTJh61TfNtqC0FQqvEASiRPC7mnu9GzLtb69DU/0Jdu+NlALsH2FIYo1lb
4+HpBXhq1sd3EsgqnWbnC8BquKmH0Whea27uEidyo0B1VdHdT+vDEJzc0juN7TSo
xs3us/rckFCEkhP0dASPe9ieWsYfUQIklQw1oKb/4WHHy0O9lyW5x0qABTbD34oF
jOy9byLTiWmXQuR9zLpiW24Qo6n58hRTQ2+10mFPif7j/brhobX/k1js+cO9sDRJ
+BkFsAJ0eU+cajpPi/lUGMgyTRPTT49yv1tjfzwsZihnH02Att+PtlXuCdmAei2/
6imtfO3ETUcbyqgbmOeqPkY4dXqY7bhRCt82KCmAf1Bx/gJz2rhrBOjwhwJbXTeF
NwcsWiFljcfSE118AJlkdHDGXnq+NaXNIqbfeOTYN0moUePis31A8+j78AEtFkAh
dapDvoirU08Z2MgQAElb3HfD9EZf+en8s7JvPn/VTCi7jxMVBUPqyzZf8pceQVwU
Vh7z8kFjqJKrRwBOSp6EpGnKYW244wCIc/24cUwx34e4MmOWmNc=
=r73d
-----END PGP SIGNATURE-----

--Sig_/gPEvcW1kQm+Ybq.N/v+Zg1q--