Bug 49911 - panel height reported x100 for LPL on DELL XPS m1710
Summary: panel height reported x100 for LPL on DELL XPS m1710
Status: RESOLVED MOVED
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-14 06:54 UTC by Olivier Fourdan
Modified: 2018-12-13 22:26 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Xorg.log (99.50 KB, application/octet-stream)
2012-05-14 06:54 UTC, Olivier Fourdan
no flags Details
xrandr --verbose output (5.91 KB, text/plain)
2012-05-14 06:55 UTC, Olivier Fourdan
no flags Details

Description Olivier Fourdan 2012-05-14 06:54:36 UTC
Created attachment 61620 [details]
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.
Comment 1 Olivier Fourdan 2012-05-14 06:55:14 UTC
Created attachment 61621 [details]
xrandr --verbose output
Comment 2 GitLab Migration User 2018-12-13 22:26:33 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/xorg/xserver/issues/422.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.