Re: [PATCH] drm: edid revision 0 is valid

[Jamie Lokier]

Kyle McMartin wrote:
 - if (edid-revision = 0 || edid-revision  3) {
 + if (edid-revision  3) {
   DRM_ERROR(EDID has minor version %d, which is not between 
 0-3\n, edid-revision);
   goto bad;

Ahem.  EDID version 1.4 exists too, in fact it's the current version.
EDID 1.5+ doesn't exist, but it should be backward compatible when it does.

I'm doing a lot of work on another EDID parser, including full DMT and
CEA-861 mode tables, monitor hotplug, all quirks from Xorg and the
kernel, EDID 1.4 support w/ CVT calculation, etc. right now for an
embedded Linux video project.  Would there be any interest
incorporating parts of that work?

-- Jamie

--
Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are
powering Web 2.0 with engaging, cross-platform capabilities. Quickly and
easily build your RIAs with Flex Builder, the Eclipse(TM)based development
software that enables intelligent coding and step-through debugging.
Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

[Dri-devel] Re: [PATCH] CodingStyle fixes for drm_agpsupport

[Jamie Lokier]

Werner Almesberger wrote:
 stmt ::= if (expression) stmt
 stmt ::= { newline stmts } newline
 stmt ::= expression ; newline
 etc.
 
 Perfectly consistent. (Always end a statement with a newline.)

So you agree with Javascript that the semicolons aren't necessary :)

-- Jamie


---
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa0013ave/direct;at.aspnet_072303_01/01
___
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel