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

Adam Jackson ajax at nwnk.net
Tue Feb 17 12:02:13 PST 2009


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, please correct me): we directly
> duplicate interlaced  mode from CEA table  which contains original
> height (like 1080i), then mode_valide function in driver will chose it
> or not ?

Yes, the CEA mode table should give frame height (eg 1920x1080) not
field height (1920x540).  So the mode table as you have it is fine.

As of commit bcafdfbe... in xserver, the driver's mode_valid() hook
won't need to check for interlace; the server will filter all modes for
it based on whether interlaceAllowed was set (and likewise for
doublescan).

Drivers that want to be compatible with pre-1.7 servers can still check
in mode_valid() if they want, it's harmless, but it won't be necessary.

- ajax
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg/attachments/20090217/6c2f9fe4/attachment.pgp>


More information about the xorg mailing list