From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 2/2] drm/i915: move panel connectors to the front Date: Thu, 1 Nov 2012 22:16:55 +0100 Message-ID: <20121101211655.GD5755@phenom.ffwll.local> References: <1351345925-17510-1-git-send-email-daniel.vetter@ffwll.ch> <1351345925-17510-2-git-send-email-daniel.vetter@ffwll.ch> <5091848D.9040408@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <5091848D.9040408@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Adam Jackson Cc: Daniel Vetter , Intel Graphics Development , DRI Development List-Id: dri-devel@lists.freedesktop.org On Wed, Oct 31, 2012 at 04:05:33PM -0400, Adam Jackson wrote: > On 10/27/12 9:52 AM, Daniel Vetter wrote: > >This essentially reverts > > > >commit cb0953d734348e8862d6d7edc666cfb3bf6d8fae > >Author: Adam Jackson > >Date: Fri Jul 16 14:46:29 2010 -0400 > > > > drm/i915: Initialize LVDS and eDP outputs before anything else > > > >simply because it doesn't scale: It misses SDVO and DVO panels, > >and now with DDI encoders on haswell this is becoming unmanageable. > > > >Instead we simply sort the connector list after everything is > >set up. > > > >Signed-off-by: Daniel Vetter > > Reviewed-by: Adam Jackson Slurped both this and the drm helper patch into dinq, thanks for the review. I'll annoy Dave about including the doc patches into his tree directly, he should get the pull which has the drm_dp_helper.c rename tomorrow. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch