Bug 49911

Summary:

panel height reported x100 for LPL on DELL XPS m1710

Product:

xorg

Reporter:

Olivier Fourdan <fourdan>

Component:

Server/General

Assignee:

Xorg Project Team <xorg-team>

Status:

RESOLVED MOVED

QA Contact:

Xorg Project Team <xorg-team>

Severity:

normal

Priority:

medium

Version:

git

Hardware:

Other

OS:

All

Whiteboard:

i915 platform:

i915 features:

Attachments:

Description	Flags
Xorg.log	none
xrandr --verbose output	none

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.