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:   Thu, 16 Sep 2021 12:24:06 +0300
From:   Pekka Paalanen <ppaalanen@gmail.com>
To:     Werner Sembach <wse@tuxedocomputers.com>
Cc:     "Deucher, Alexander" <alexander.deucher@amd.com>,
        amd-gfx list <amd-gfx@lists.freedesktop.org>,
        Ville =?UTF-8?B?U3lyasOkbMOk?= <ville.syrjala@linux.intel.com>,
        Intel Graphics Development <intel-gfx@lists.freedesktop.org>,
        Maling list - DRI developers 
        <dri-devel@lists.freedesktop.org>,
        LKML <linux-kernel@vger.kernel.org>, sebastian@sebastianwick.net
Subject: Re: New uAPI for color management proposal and feedback request v2
Message-ID: <20210916122406.7c132525@eldfell>
In-Reply-To: <e452775c-5b95-bbfd-e818-f1480f556336@tuxedocomputers.com>
References: <e452775c-5b95-bbfd-e818-f1480f556336@tuxedocomputers.com>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="Sig_/m2oIbyLut0gDnAcjrS61zLj";
 protocol="application/pgp-signature"; micalg=pgp-sha256
Precedence: bulk

--Sig_/m2oIbyLut0gDnAcjrS61zLj
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

On Tue, 3 Aug 2021 11:38:19 +0200
Werner Sembach <wse@tuxedocomputers.com> wrote:

> Greetings,
>=20
> Original proposal: https://www.mail-archive.com/amd-gfx@lists.freedesktop=
.org/msg62387.html
>=20
> Abstract: Add "preferred color format", "active color format", "active bp=
c", and "active Broadcast RGB" drm properties,
> to control color information send to the monitor.
>=20
> It seems that the "preferred-" properties is not what is actually the mos=
t useful for the userspace devs.
>=20
> Preferable (Note: with only a sample size of 2 people) would be a "force =
color format" property. If the color format is
> not available for the current Monitor and GPU combo. the TEST_ONLY check =
should fail and the property should not be setable.
>=20
> This however opens another problem: When a Monitor is disconnected and a =
new one is connected, the drm properties do not
> get resetted. So if the old monitor did allow to set for example ycbcr420=
, but the new monitor does not support this
> color format at all, it will stay permanently black until the drm propert=
y is set to a correct value by hand. This is
> not an expected behavior imho.
>=20
> So a discussion questions: Does it make sense that connector properties a=
re keep for different Monitors?
>=20
> If no: On connecting a new Monitor all atomic drm properties should be re=
set to a default value.
>=20
> I have an idea how this could be implemented (correct me if i'm wrong): W=
hen an atomic property is attached it get
> assigned an inital value. But if I understood the docu correctly, this va=
lue is ignored because atomic properties use
> the getter and setter methods when their values are read or written. My i=
mplementation suggestion would be to iterate
> over all attached atomic properties once a new monitor is connected and r=
eset them to this initial value, which should
> be unchanged since initialization? This assumes that besides the initial =
value being unused it's still a sane default
> for all drivers.
>=20
> Kind Regards,
>=20
> Werner Sembach
>=20

Hi Werner,

I just wanted to say that I appreciate the effort and something like
these are things I would likely want to use some day in Weston, but
currently I can't spend time on this since it's still so far in the
future for me. I also feel that I've said what I can without spending a
significant amount of time thinking how it should actually work.

The same goes with the thing pointed out by Sebastian, the KMS property
reset idea. Since I can't really work on it now, I'll stop shouting
about it and wait for problems to arise in the wild and see where that
leads. (Probably just patching the KMS client of the day to set a
couple more KMS properties.)


Thanks,
pq

--Sig_/m2oIbyLut0gDnAcjrS61zLj
Content-Type: application/pgp-signature
Content-Description: OpenPGP digital signature

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

iQIzBAEBCAAdFiEEJQjwWQChkWOYOIONI1/ltBGqqqcFAmFDDTYACgkQI1/ltBGq
qqfxZg//bxjNceiMruwsAHhjW56idiGLy07lSyp5IItUXdTQHZer4dERd/jmd5ee
4ln2oszEfUjJDOc+MgtyuarG/5XwD6riUu353cvGAO+Ugd66yRxuDT4sGe2ZcZOs
hhkH6AT/qKJioQ3CWPLpXX5oK9i/wrzSx55TIZyPvhDkaGIpEwFbJ843evdmURGZ
vclCG0f4y5Uk976QUttDRl+llhPK8vWbMwNWzchNWroyTuaYeLU7fWM9jrycO3wo
44hNJatGdW+gqZBM98VaL5p7OAw0VMo9S/hDAOnXCXp79U+fUWWI5bArcjEsbhA/
vD4oCte3YfQ51l3Ihv3NLUkiMKmwrpQ9xiQp+pNqj2lAu/AzpsC1HAyLBEZaxA6U
YzvW55ONMvVgVy1TMPp4BatC+WP/XyFZrKzsG4gNzyDpc64Az9HmXRoeF241Obmr
/LxbJ7f9uihfLlwguwliyAaoMaZNzMYpKXE153KDbRCv0JwYNG5i0rU5Bywx+CNW
r/1eMvelNkxvKzbEtpY5Wl0vjvZllq8HE1dZ5/+EOUV35lnh/XEq9j0lr8d/BHsw
3DPqCwrNVxqvgvv4yutr4QeCFjbfs+hx98/KI4pX0imxbGCG/lPhK4i8a7P1LVK5
dd2zzBPmB4evvh7DTEiq1V7M5VQgJHmHUXdv+BoTEcCBOJN+K/E=
=Ldod
-----END PGP SIGNATURE-----

--Sig_/m2oIbyLut0gDnAcjrS61zLj--