[PATCH] drm/edid: Fix the HDTV hack yet more.

[Andrew Shadura]

Hello,

On Wed, 22 Feb 2012 13:05:34 -0500
Adam Jackson <ajax at redhat.com> wrote:

> On Wed, 2012-02-22 at 19:08 +0300, Andrew Shadura wrote:
> > Checking for just two variants of standard timings for
> > 1366x768 isn't quite correct, let's check for ranges
> > instead.

> I... doubt it?  Standard timing descriptors are a tuple of

> * width in character cells (ie, 8 pixels)
> * one of four aspect ratios
> * refresh rate

> So you're only ever going to have 1360 or 1368, and then in integer
> math I think you could only possibly ever get:

> a) 1360 * 9 / 16 = 765
> b) 1368 * 9 / 16 = 769

> If you're managing to get something else I'm a little concerned, and
> would really be delighted to see the EDID block for it.

Well, my fault, I haven't read the code 10 lines above :) Indeed, that's
not STD, it's DTD. And I'm not sure how to fix it then, but DTD gives
1360x768, and that isn't quite correct.

-- 
WBR, Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20120222/ca3678c2/attachment.pgp>