At Mon, 02 Jul 2012 15:46:29 -0400, Adam Jackson wrote:
On 6/26/12 3:21 AM, Takashi Iwai wrote:
From: Takashi Iwai tiwai@suse.de Subject: [PATCH] drm: edid: Don't add inferred modes with higher resolution
When a monitor EDID doesn't give the preferred bit, driver assumes that the mode with the higest resolution and rate is the preferred mode. Meanwhile the recent changes for allowing more modes in the GFT/CVT ranges give actually more modes, and some modes may be over the native size. Thus such a mode would be picked up as the preferred mode although it's no native resolution.
For avoiding such a problem, this patch limits the addition of inferred modes by checking not to be greater than other modes. Also, it checks the duplicated mode entry at the same time.
This is a little aggressive on CRTs, but whatever, better than what's there.
Reviewed-by: Adam Jackson ajax@redhat.com
Thanks. I'm going to resend the patch with your tag.
Takashi