RE: [PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-17 Thread Adam Jackson
On Sat, 2009-02-14 at 17:02 +0800, Ma, Ling wrote: And yeah, I think frame height is the sensible representation for modes internally. Now we just need to figure out how to fix up EDID timings that are in field height, but that's a fight for another day. So your menas (If I was wrong,

RE: [PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-17 Thread Ma, Ling
OK, I got it! Thanks a lot Ma Ling -Original Message- From: Adam Jackson [mailto:a...@nwnk.net] Sent: 2009年2月18日 4:02 To: Ma, Ling Cc: xorg@lists.freedesktop.org Subject: RE: [PATCH 8/9] handle cea-ext data block and insert mode into mode list. On Sat, 2009-02-14 at 17:02 +0800, Ma

Re: [PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-13 Thread Adam Jackson
On Thu, 2009-02-12 at 15:07 +0800, Ma Ling wrote: On Thu, 2009-02-12 at 03:11 +0800, Adam Jackson wrote: This timing list actually brings up an unpleasant topic. How are interlaced modes supposed to be represented inside the server? Detailed blocks seem to mostly be written such that

Re: [PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-13 Thread Jim Gettys
It would be good if the xorg-devel list appeared on the mail list list at: http://lists.freedesktop.org/mailman/listinfo/ This is a checkbox in mailman; of course, with mailman, it can often be a maze of little passages, all different... - Jim On Fri, 2009-02-13 at 13:45

Re: [PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-13 Thread Daniel Stone
On Fri, Feb 13, 2009 at 01:47:48PM -0500, Jim Gettys wrote: It would be good if the xorg-devel list appeared on the mail list list at: http://lists.freedesktop.org/mailman/listinfo/ This is a checkbox in mailman; of course, with mailman, it can often be a maze of little passages, all

[PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-11 Thread Ma Ling
select appropriate mode form video data blocks of CEA extension, then insert them into mode list. --- hw/xfree86/ddc/interpret_edid.c | 56 + hw/xfree86/ddc/xf86DDC.h |3 + hw/xfree86/modes/xf86EdidModes.c | 102 ++ 3 files

Re: [PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-11 Thread Adam Jackson
On Wed, 2009-02-11 at 17:17 +0800, Ma Ling wrote: +void xf86FetchDataBlock(xf86MonPtr mon, +int ext_type, int data_type, +void *data) +{ +int i; +Uchar *ext; + +if (mon == NULL) + return; + +for (i = 0; i

Re: [PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-11 Thread Ma Ling
On Thu, 2009-02-12 at 03:11 +0800, Adam Jackson wrote: On Wed, 2009-02-11 at 17:17 +0800, Ma Ling wrote: +void xf86FetchDataBlock(xf86MonPtr mon, +int ext_type, int data_type, +void *data) +{ +int i; +Uchar *ext; + +if

Re: [PATCH 8/9] handle cea-ext data block and insert mode into mode list.

2009-02-11 Thread Ma Ling
Sorry, please use this one to replace last email. Thanks Ma Ling On Thu, 2009-02-12 at 03:11 +0800, Adam Jackson wrote: On Wed, 2009-02-11 at 17:17 +0800, Ma Ling wrote: +void xf86FetchDataBlock(xf86MonPtr mon, +int ext_type, int data_type, +