<div dir="ltr"><div>Hi,</div><div><br></div><div>For letting DRM clients to select output encoding:<br></div><div>Sink can support certain display timings with high output bit-depths using multiple output encodings, e.g. sink can support a particular timing with RGB 10-bit, YCbCr422 10-bit and YCbCr420 10-bit. So DRM client may want to select YCbCr422 10-bit over RBG 10-bit output to reduce the link bandwidth (and in turn reduce power/voltage). If DRM driver automatically selects output encoding then we are restricting DRM clients from making appropriate choice.</div><div><br></div><div>For selectable output color range:</div><div>Certain applications (typically graphics) usually rendered in full range while some applications (typically video) have limited range content. Since content can change dynamically, DRM driver does not have enough information to choose correct quantization. Only DRM client can correctly select which quantization to set (to preserve artist's intent).</div><div><br></div><div>For how to use selectable output encoding with Weston: <br></div><div>I was thinking that DRM should have separate property to list the encodings supported by sink and Weston will present this list to its client. Your idea to validate encodings using TEST_ONLY commit and present a list of timings along with encodings supported by particular timing seems better. Instead of validating all possible encodings, does it make sense to validate  only those supported by sink? Irrespective of this we would anyway need some mechanism which will allow user to select particular encoding for a particular mode. I was thinking to allow this using new DRM property "Encoding". Do you have anything better in mind?</div><div><br></div><div></div><div></div><div>(Since I am using my Gmail Id, I feel I should mention that I work at Nvidia)<br></div><div></div><div><br></div><div>Thanks,</div><div>-Yogish<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 28, 2020 at 6:18 PM Pekka Paalanen <<a href="mailto:ppaalanen@gmail.com">ppaalanen@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, 28 May 2020 17:38:59 +0530<br>
Yogish Kulkarni <<a href="mailto:yogishkulkarni@gmail.com" target="_blank">yogishkulkarni@gmail.com</a>> wrote:<br>
<br>
> I am trying to find a way through Weston which will allow setting specific<br>
> encoding at display output.<br>
<br>
Hi,<br>
<br>
why do *you* want to control that?<br>
<br>
Why not let the driver always choose the highest possible encoding<br>
given the video mode and hardware capability?<br>
<br>
I can understand userspace wanting to know what it got, but why should<br>
userspace be able to control it?<br>
<br>
Would people want to pick the encoding first, and then go for the<br>
highest possible video mode?<br>
<br>
> Could you please elaborate on  why it is best<br>
> to let DRM driver automatically configure which encoding to choose rather<br>
> than making it selectable by DRM client ? I am not able to find reference<br>
> to past discussion about this. I was only able to find a proposed change -<br>
> <a href="https://lists.freedesktop.org/archives/intel-gfx/2017-April/125451.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/archives/intel-gfx/2017-April/125451.html</a> but<br>
> am not able to find why it got rejected.<br>
> <br>
> Alternatively, is there existing way through which DRM clients can specify<br>
> preference for output encoding ? Or currently it's all up to the DRM driver<br>
> to choose what output encoding to use.<br>
<br>
There must be some reason why userspace needs to be able to control it.<br>
I'm also asking as a Weston maintainer, since I'm interested in how<br>
this affects e.g. color reproduction or HDR support.<br>
<br>
One thing that comes to my mind is using atomic TEST_ONLY commits to<br>
probe all the possible video modes × encodings for presenting a list to<br>
the user to choose from, if you have a display configuration GUI. E.g<br>
with some TV use cases, maybe the user wants to avoid sub-sampling, use<br>
the native resolution, but limit refresh rate to what's actually<br>
possible. Or any other combination of the three.<br>
<br>
<br>
Thanks,<br>
pq<br>
<br>
> <br>
> Thanks,<br>
> -Yogish<br>
> <br>
> On Thu, May 28, 2020 at 1:54 PM Daniel Vetter <<a href="mailto:daniel@ffwll.ch" target="_blank">daniel@ffwll.ch</a>> wrote:<br>
> <br>
> > On Thu, May 28, 2020 at 12:29:43PM +0530, Yogish Kulkarni wrote:  <br>
> > > For creating new source property, is it good to follow<br>
> > > "drm_mode_create_hdmi_colorspace_property()"  as an example ? It seems  <br>
> > that  <br>
> > > currently there is no standard DRM property which allows DRM client to  <br>
> > set  <br>
> > > a specific output encoding (like YUV420, YUV422 etc). Also, there is no<br>
> > > standard property for letting client select YUV/RGB color range. I see<br>
> > > there are two ways to introduce new properties, 1. do something like<br>
> > > drm_mode_create_hdmi_colorspace_property 2. create custom property  <br>
> > similar  <br>
> > > to "Broadcast RGB". Is there opinion on which is a preferable way to  <br>
> > expose  <br>
> > > encoding and color rage selection property ?  <br>
> ><br>
> > I guess first question is "why?" Thus far we've gone with the opinion that<br>
> > automatically configuring output stuff as much as possible is best. What's<br>
> > the use-case where the driver can't select this?<br>
> > -Daniel<br>
</blockquote></div>