From: Adam Jackson <ajax@redhat.com>
To: Florian Mickler <florian@mickler.org>
Cc: intel-gfx@lists.freedesktop.org, xorg-devel@lists.x.org
Subject: Re: [PATCH] change initial modesetting if outputs are aligned in 1 dimension
Date: Wed, 01 Jun 2011 15:30:03 -0400 [thread overview]
Message-ID: <4DE6933B.4000000@redhat.com> (raw)
In-Reply-To: <1306922798-29344-1-git-send-email-florian@mickler.org>
On 6/1/11 6:06 AM, Florian Mickler wrote:
> Recently the kernel started reporting my outputs in a different ordering due to
>
> commit cb0953d734
> (drm/i915: Initialize LVDS and eDP outputs before anything else)
>
> Which made X choose a "wrong" resolution for my VGA display. Since they are
> aligned horizontally, I wish them to be aligned in vertical
> Resolution only.
>
> Before this patch, the sum of squared distances would force my VGA display
> (1680x1050 native resolution) to 1280x1024 (non-native) due to my internal
> display beeing considered first and 1400x1050 as native resolution.
>
> This was not an issue the other way around (VGA beeing first) because 1400x1050
> is nearest to 1680x1050 anyway.
>
> This patch changes the heuristic to only align resolution vertically if the
> displays are horizontally aligned, or vice versa.
>
> Signed-off-by: Florian Mickler<florian@mickler.org>
> ---
>
> Ok, Adam... seems I lost the staring contest... :)
> What about something like this?
At this point, given the near-unity overlap of "RANDRful drivers" and
"usable KMS support", I think I'd prefer something more like:
http://pkgs.fedoraproject.org/gitweb/?p=xorg-x11-server.git;a=blob_plain;f=xserver-1.6.99-right-of.patch;h=a0c9e7f64b98a091f64faaf5a8a432e2122e25f9;hb=HEAD
particularly once per-crtc pixmaps land.
- ajax
next prev parent reply other threads:[~2011-06-01 19:30 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-18 13:41 since kernel commit cb0953d734: X chooses wrong resolution at startup Florian Mickler
2011-05-18 15:17 ` Adam Jackson
2011-05-18 16:15 ` Florian Mickler
2011-05-18 17:03 ` Florian Mickler
2011-06-01 10:06 ` [PATCH] change initial modesetting if outputs are aligned in 1 dimension Florian Mickler
2011-06-01 19:30 ` Adam Jackson [this message]
2011-06-01 20:44 ` Florian Mickler
2011-06-01 20:48 ` Adam Jackson
2011-06-01 21:06 ` Florian Mickler
2011-06-01 21:30 ` Adam Jackson
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=4DE6933B.4000000@redhat.com \
--to=ajax@redhat.com \
--cc=florian@mickler.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=xorg-devel@lists.x.org \
/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.