On Mon, Oct 19, 2020 at 06:39:59PM +0300, Imre Deak wrote: > On Tue, Oct 06, 2020 at 09:58:08PM +0300, Ville Syrjala wrote: > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > LSPCON requires HPD detection logic to be enabled when we call > > its .reset() hook during resume, to check the live state of the > > pin. To that end let's reorder the resume sequence such that > > we do HPD init before the encoder reset. > > > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx> I think I'll just drop this patch entirely. AFAICS no longer needed once the lspcon resume is moved out of .reset(). > > > --- > > drivers/gpu/drm/i915/i915_drv.c | 7 +++---- > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > > index b2a050d916e3..66ddd4161885 100644 > > --- a/drivers/gpu/drm/i915/i915_drv.c > > +++ b/drivers/gpu/drm/i915/i915_drv.c > > @@ -1213,21 +1213,20 @@ static int i915_drm_resume(struct drm_device *dev) > > * GPU will hang. i915_gem_init_hw() will initiate batches to > > * update/restore the context. > > * > > - * drm_mode_config_reset() needs AUX interrupts. > > - * > > * Modeset enabling in intel_modeset_init_hw() also needs working > > * interrupts. > > */ > > intel_runtime_pm_enable_interrupts(dev_priv); > > > > - drm_mode_config_reset(dev); > > - > > i915_gem_resume(dev_priv); > > > > intel_modeset_init_hw(dev_priv); > > intel_init_clock_gating(dev_priv); > > intel_hpd_init(dev_priv); > > > > + /* May need AUX interrupts and HPD detection enabled */ > > + drm_mode_config_reset(dev); > > + > > /* MST sideband requires HPD interrupts enabled */ > > intel_dp_mst_resume(dev_priv); > > intel_display_resume(dev); > > -- > > 2.26.2 > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
