[Intel-gfx] [Resend2] [PATCH 0/3] Add the default mode for LVDS in KMS mode

[yakui_zhao]

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-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx