On Thu, 01 Oct 2020, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Just like with eDP let's wait for the power sequencer power > cycle delay before we reboot the machine, as otherwise we > can't guarantee the panel's minimum power cycle delay will > be respected. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_lvds.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_lvds.c b/drivers/gpu/drm/i915/display/intel_lvds.c > index e65c2de522c3..c6c7c0b9989b 100644 > --- a/drivers/gpu/drm/i915/display/intel_lvds.c > +++ b/drivers/gpu/drm/i915/display/intel_lvds.c > @@ -371,6 +371,15 @@ static void pch_post_disable_lvds(struct intel_atomic_state *state, > intel_disable_lvds(state, encoder, old_crtc_state, old_conn_state); > } > > +static void intel_lvds_shutdown(struct intel_encoder *encoder) > +{ > + struct drm_i915_private *dev_priv = to_i915(encoder->base.dev); > + > + if (intel_de_wait_for_clear(dev_priv, PP_STATUS(0), PP_CYCLE_DELAY_ACTIVE, 5000)) I guess we'll be seeing reports if we hit the 5s delay. :| Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > + drm_err(&dev_priv->drm, > + "timed out waiting for panel power cycle delay\n"); > +} > + > static enum drm_mode_status > intel_lvds_mode_valid(struct drm_connector *connector, > struct drm_display_mode *mode) > @@ -897,6 +906,7 @@ void intel_lvds_init(struct drm_i915_private *dev_priv) > intel_encoder->get_hw_state = intel_lvds_get_hw_state; > intel_encoder->get_config = intel_lvds_get_config; > intel_encoder->update_pipe = intel_panel_update_backlight; > + intel_encoder->shutdown = intel_lvds_shutdown; > intel_connector->get_hw_state = intel_connector_get_hw_state; > > intel_connector_attach_encoder(intel_connector, intel_encoder); -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx