[PATCH] xfree86: Add quirk for Acer AL1923 monitor

[Tormod Volden]

On Mon, Feb 9, 2009 at 11:54 AM, Julien Cristau <jcristau at debian.org> wrote:
> On Sun, 2009-02-08 at 22:36 +0100, Tormod Volden wrote:
>> From: Tormod Volden <debian.tormod at gmail.com>
>>
>> The monitor reports aspect ratio instead of image size in the detailed
>> timings. https://bugs.launchpad.net/bugs/320930
>> Thanks to Sorin Enache for testing.
>
> Hi Tormod,
>
> Why is this needed on top of 0660dd9d7009147c395b9ea904539f76f55b9a7f?
>
> Cheers,
> Julien
>

The 0660dd9d7009147c395b9ea904539f76f55b9a7f checks the "Max Image
Size" i.e. DDC->features.hsize for typical aspect ratios. OTOH, this
monitor reports "Max Image Size" correctly, but puts the aspect ratio
in the detailed timing "Image Size" i.e.
det_mon->section.d_timings.h_size. So it's the other way around.

See an example log here:
http://launchpadlibrarian.net/21796076/E%3A%5Cbug%5CXorg.0.log.radeon

Maybe similar autodetection of aspect ratio in the detailed timings
can be added, instead of my patch, if this is seen with many monitors.

BTW, the whole truth is that I also overlooked the 0660dd quirk
because it was in another file than the other quirks. Although I
remember reading the funny commit message, but I was not so into EDID
at the time.

Cheers,
Tormod