[Intel-gfx] [PATCH] drm/i915: rip out edp special case from dp_link_down

Adam Jackson ajax at redhat.com
Mon Sep 10 18:15:40 CEST 2012


On 9/9/12 1:58 PM, Daniel Vetter wrote:

> The really interesting question is how this special cases survived
> this long in the code. The first step is declaring the pch port D as
> eDP if it's used for an internal panel:
>
> commit b329530ca7cdf6bf014f2124efd983e01265d623
> Author: Adam Jackson <ajax at redhat.com>
> Date:   Fri Jul 16 14:46:28 2010 -0400
>
>      drm/i915/dp: Correctly report eDP in the core connector type
>
> This commit unfortunately failed to notice that not all edp ports are
> created equal. Then follow a flurry of refactorings, culminating in a
> patch from Keith Packard which resulted in the current logic (by
> making it "correct" for all platforms that have edp):

The motivation for this patch was simply making eDP connectors be 
reported as eDP, since that has the same "probably special" properties 
as LVDS (correlation with lid state, probably the primary if no better 
guidance given, etc).

- ajax



More information about the Intel-gfx mailing list