On Fri, Aug 14, 2015 at 10:53:17AM +0300, Jani Nikula wrote: > There is currently conflicting documentation on which steppings the > workaround is needed, up to C vs. forever. However there is post-C > stepping hardware that doesn't report port presence on DDI A, leading to > black screen on eDP. Assume the strap isn't connected, and try to enable > DDI A on these machines. (We'll still check the VBT for the info in DDI > init.) > > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > Cc: Mika Westerberg <mika.westerberg@xxxxxxxxx> > Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> Queued for -next, thanks for the patch. -Daniel > --- > drivers/gpu/drm/i915/intel_display.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index 21aa745caed1..aa208a2cfafc 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -13949,8 +13949,7 @@ static void intel_setup_outputs(struct drm_device *dev) > */ > found = I915_READ(DDI_BUF_CTL_A) & DDI_INIT_DISPLAY_DETECTED; > /* WaIgnoreDDIAStrap: skl */ > - if (found || > - (IS_SKYLAKE(dev) && INTEL_REVID(dev) < SKL_REVID_D0)) > + if (found || IS_SKYLAKE(dev)) > intel_ddi_init(dev, PORT_A); > > /* DDI B, C and D detection is indicated by the SFUSE_STRAP > -- > 2.1.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
