> -----Original Message----- > From: Intel-gfx [mailto:[email protected]] On Behalf > Of [email protected] > Sent: Saturday, June 28, 2014 12:04 AM > To: [email protected] > Subject: [Intel-gfx] [PATCH 10/40] drm/i915: Call encoder->post_disable() in > intel_sanitize_encoder() > > From: Ville Syrjälä <[email protected]> > > VLV and CHV disable the DP port only in the .post_disable() hook, so we > need to make intel_sanitize_encoder() call that when it's trying to > disable encoders without an active pipes. > > My bsw actaully hits this when an external display is connected. The > BIOS still likes to turn on the eDP port, but leaves the pipe disabled. > > Signed-off-by: Ville Syrjälä <[email protected]> > --- > drivers/gpu/drm/i915/intel_display.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index 3e4d570..a16f635 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -12772,6 +12772,8 @@ static void intel_sanitize_encoder(struct > intel_encoder *encoder) > encoder->base.base.id, > encoder->base.name); > encoder->disable(encoder); > + if (encoder->post_disable) > + encoder->post_disable(encoder); > } > encoder->base.crtc = NULL; > encoder->connectors_active = false; > -- > 1.8.5.5 >
I haven't done as much DP testing on my CHT but that makes sense. Reviewed-by: Rafael Barbalho <[email protected]> > _______________________________________________ > Intel-gfx mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
