Re: [PATCH v4 09/17] drm/uAPI: Add "active color range" drm property as feedback for userspace

From: Werner Sembach
Date: Wed Jun 23 2021 - 06:17:44 EST



Am 23.06.21 um 09:32 schrieb Pekka Paalanen:
> On Tue, 22 Jun 2021 11:48:52 +0000
> Simon Ser <contact@xxxxxxxxxxx> wrote:
>
>> On Tuesday, June 22nd, 2021 at 11:50, Werner Sembach <wse@xxxxxxxxxxxxxxxxxxx> wrote:
>>
>>> Unknown is when no monitor is connected or is when the
>>> connector/monitor is disabled.
>> 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.
> That sounds fine to me.

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.

Setting a "not applicable" value allows userspace to be more dumb, without much hassle on the kernelspace side.

>
>
> Thanks,
> pq
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx