On Mon, 2009-06-08 at 10:04 +0800, yakui_zhao wrote:
> Hi, All
>       the following is the patch set that adds the default modes for LVDS in 
> KMS
> mode. It is to solve the following bugs:
>         http://bugs.freedesktop.org/show_bug.cgi?id=20801
>         http://bugs.freedesktop.org/show_bug.cgi?id=21094        
>         http://bugs.freedesktop.org/show_bug.cgi?id=21346
>         http://bugs.freedesktop.org/show_bug.cgi?id=21417
>         http://bugs.freedesktop.org/show_bug.cgi?id=21671
> 
Hi, Jessa/Eric

        How about this patch set?
        In this patch set the default modes are added in user space. At the
same time I also use the LVDS panel limit to check whether the modeline
is valid.The modeline added by xrandr tool can also be validated.
        
        Will you please review the patch set? Is something missing in my patch
set?
        Now it seems that this is the KMS gap. Xrandr tool will get less modes
for LVDS output device than KMS mode.

Thanks.
        
>     
>       Patch 1/3: Add a private data for LVDS
>               When the connector type is LVDS, it will add a private data 
> that 
> can be used to store the LVDS panel limit. For example: hdisplay, vdisplay.
>       
>       Patch 2/3: Get the LVDS panel limit and check whether the given modeline
> is valid
>               Traverse the mode list returned by KMS kernel to get the LVDS 
> panel
> limit. And the panel limit is used to check whether the given modeline is 
> valid.
> 
>       Patch 3/3: Add the default modes for the LVDS
>               This is similar to what we have done in UMS mode
>                 a. When there exists the EDID, either find a DS_RANGES
> block or replace a DS_VENDOR block , smashing it into a DS_RANGES block
> with open refresh to match all the default modes. 
>                 b. When there is no EDID, we will construct a bogus EDID
> and add a DS_RANGES block with open refresh to match all the default
> modes
> 
> welcome the comments.
> 
> Best regards
>     Yakui
> 
> _______________________________________________
> Intel-gfx mailing list
> intel-...@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx


------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to