On Wed, May 16, 2018 at 10:50:38AM +0200, Maarten Lankhorst wrote: > Normally this is called on a modeset, but the call is missing when > we inherit the mode from the BIOS, so make sure it's called somewhere > in hardware readout. > > Changes since v1: > - Unconditionally call intel_opregion_notify_encoder. (Ville) > > Signed-off-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_display.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index 501b83a9a128..e9dc72341e5f 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -15176,6 +15176,9 @@ static void intel_sanitize_encoder(struct intel_encoder *encoder) > connector->base.dpms = DRM_MODE_DPMS_OFF; > connector->base.encoder = NULL; > } > + > + /* notify opregion of the sanitized encoder state */ > + intel_opregion_notify_encoder(encoder, connector && has_active_crtc); Seems reasonable. The only minor nit is that we call this before the encoder hooks in the normal modeset path, here we call it after the hooks. Not sure it really matters though, so Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > } > > void i915_redisable_vga_power_on(struct drm_i915_private *dev_priv) > -- > 2.17.0 -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
