Re: [v3 6/7] drm: Add Client Cap for advance gamma mode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 4/15/2019 7:42 PM, Lankhorst, Maarten wrote:
mån 2019-04-15 klockan 19:26 +0530 skrev Sharma, Shashank:
-----Original Message-----
From: Lankhorst, Maarten
Sent: Monday, April 15, 2019 4:28 PM
To: Shankar, Uma <uma.shankar@xxxxxxxxx>; intel-gfx@lists.freedeskt
op.org; dri-
devel@xxxxxxxxxxxxxxxxxxxxx
Cc: Syrjala, Ville <ville.syrjala@xxxxxxxxx>; emil.l.velikov@gmail.
com;
sam@xxxxxxxxxxxx; Roper, Matthew D <matthew.d.roper@xxxxxxxxx>;
seanpaul@xxxxxxxxxxxx; brian.starkey@xxxxxxx; dcastagna@xxxxxxxxxxx
g;
Sharma, Shashank <shashank.sharma@xxxxxxxxx>
Subject: Re: [v3 6/7] drm: Add Client Cap for advance gamma mode

fre 2019-04-12 klockan 15:51 +0530 skrev Uma Shankar:
Introduced a client cap for advance cap mode
capability. Userspace should set this to get
to be able to use the new gamma_mode property.

If this is not set, driver will work in legacy
mode.

Suggested-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
Signed-off-by: Uma Shankar <uma.shankar@xxxxxxxxx>
Nack, this doesn't seem like a sensible idea. We already guard it
behind the gamma mode property. Userspace shouldn't set the gamma
mode
to a value it doesn't understand.

~Maarten
Hey Maarten,
In that case, what do you suggest should be the right way to do this
?

@Ville, any comments here ?

I would say drop this patch, and just enable segmented gamma
unconditionally, it's not the first property that can cause trouble
when not understood.

Honestly, I kindof like this simple approach suggestion, which makes the property focused, and easy to use. But one of the benefits I see for this new gamma mode property was, that we can handle other advanced gamma modes like 10/12/split and multi-segmented gamma modes too, using this. So even though the single property is easy to use, but if we try to cover each of the gamma modes per single property, it might get confusing to see 4 different gamma mode properties, and how to set one of this.

Another problem I see is, the precision and no of entries in the LUT, for multi-segment gamma is different than traditional gamma, and we might break some (most ?) of the old userspaces. Do you think is there any way in which we can avoid this ?

- Shashank

~Maarten
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux