panel height reported x100 for LPL on DELL XPS m1710
@ofourdan
Submitted by Olivier Fourdan Assigned to Xorg Project Team
Description
Created attachment 61620 Xorg.log
Trying to get the physical panel size from xrandr, the reported panel height is wrong by a factor x100.
xrandr gives:
LVDS-1 connected 1920x1200+0+0 (normal left inverted right x axis y axis) 367mm x 23000mm
Checking the Xorg.0.log:
X.Org X Server 1.12.0
Release Date: 2012-03-04
[ 47.319] X Protocol Version 11, Revision 0
...
[ 48.916] (II) NOUVEAU(0): Max Image Size [cm]: horiz.: 37 vert.: 23
...
[ 48.917] (II) NOUVEAU(0): EDID vendor "LPL", prod id 0
[ 48.917] (II) NOUVEAU(0): EDID quirk: Detailed timings give vertical size in cm.
The Max image size seems correct.
Checking at "Detailed timings give vertical size in cm", it comes from:
http://cgit.freedesktop.org/xorg/xserver/tree/hw/xfree86/modes/xf86EdidModes.c#n239
{
quirk_detailed_v_in_cm, DDC_QUIRK_DETAILED_V_IN_CM,
"Detailed timings give vertical size in cm."},
{
So DDC_QUIRK_DETAILED_V_IN_CM is set, so the size is x10:
if (quirks & DDC_QUIRK_DETAILED_V_IN_CM)
det_mon->section.d_timings.v_size *= 10;
And that comes from:
http://cgit.freedesktop.org/xorg/xserver/tree/hw/xfree86/modes/xf86EdidModes.c#n133
/* Bug #10304: LGPhilipsLCD LP154W01-A5 */
if (memcmp(DDC->vendor.name, "LPL", 4) == 0 && DDC->vendor.prod_id == 0)
return TRUE;
Because the reported product_id is indeed 0:
[ 48.916] (II) NOUVEAU(0): Manufacturer: LPL Model: 0 Serial#: 0
So I suspect the quirk for bug #10304 could make things worse in the case of this panel/laptop.
Attachment 61620, "Xorg.log":
Xorg.0.log
Version: git