From: Adam Jackson <ajax@redhat.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: Sven Joachim <svenjoac@gmx.de>,
dri-devel@lists.freedesktop.org,
Rodrigo Vivi <rodrigo.vivi@intel.com>,
Dave Airlie <airlied@redhat.com>,
linux-kernel@vger.kernel.org
Subject: Re: Bogus video resolution in Linux 3.5-rc4
Date: Mon, 02 Jul 2012 15:46:29 -0400 [thread overview]
Message-ID: <4FF1FA95.7000404@redhat.com> (raw)
In-Reply-To: <s5hsjdi1qtv.wl%tiwai@suse.de>
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>
- ajax
next prev parent reply other threads:[~2012-07-02 19:46 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-25 14:03 Bogus video resolution in Linux 3.5-rc4 Sven Joachim
2012-06-25 15:53 ` Takashi Iwai
2012-06-25 15:57 ` Takashi Iwai
2012-06-25 17:40 ` Sven Joachim
2012-06-25 19:22 ` Adam Jackson
2012-06-25 19:24 ` Takashi Iwai
2012-06-25 19:38 ` Sven Joachim
2012-06-26 7:21 ` Takashi Iwai
2012-06-30 18:46 ` Calvin Owens
2012-06-30 19:24 ` Takashi Iwai
2012-06-30 21:14 ` Sven Joachim
2012-06-30 21:14 ` Sven Joachim
2012-07-02 19:46 ` Adam Jackson [this message]
2012-07-03 9:20 ` Takashi Iwai
2012-06-25 20:25 ` Andy Furniss
2012-06-25 21:03 ` Andy Furniss
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FF1FA95.7000404@redhat.com \
--to=ajax@redhat.com \
--cc=airlied@redhat.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rodrigo.vivi@intel.com \
--cc=svenjoac@gmx.de \
--cc=tiwai@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.