<div dir="ltr">I can confirm that this v2 patch, combined with the change to the xorg<div>driver is enough for X to report the correct display dimensions for</div><div>my Surface 3 device.</div><div><div class="gmail_extra"><br></div><div class="gmail_extra">Thank you,</div><div class="gmail_extra">Stephen</div><div class="gmail_extra">
<br><div class="gmail_quote">On Tue, May 31, 2016 at 3:08 AM,  <span dir="ltr"><<a href="mailto:ville.syrjala@linux.intel.com" target="_blank">ville.syrjala@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a>><br>
<br>
The VBT has these mysterious H/V image sizes as part of the display<br>
timings. Looking at some dumps those appear to be the physical<br>
dimensions in mm. Which makes sense since the timing descriptor matches<br>
the format used by EDID detailed timing descriptor, which defines these<br>
as "H/V Addressable Video Image Size in mm".<br>
<br>
So let's use that information from the panel fixed mode to get the<br>
physical dimensions for LVDS/eDP/DSI displays. And with that we can<br>
fill out the display_info so that userspace can get at it via<br>
GetConnector.<br>
<br>
v2: Use (hi<<8)|lo instead of broken (hi<<4)+lo<br>
    Handle LVDS and eDP too<br>
<br>
Cc: Stephen Just <<a href="mailto:stephenjust@gmail.com">stephenjust@gmail.com</a>><br>
Tested-by: Stephen Just <<a href="mailto:stephenjust@gmail.com">stephenjust@gmail.com</a>> (v1)<br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=96255" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=96255</a><br>
Signed-off-by: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a>><br></blockquote></div></div></div></div>