On Wed, 09 Dec 2020, Imre Deak <imre.deak@xxxxxxxxx> wrote: > For an enabled DSC during HW readout the corresponding power reference > is taken along the CRTC power domain references in > get_crtc_power_domains(). Remove the incorrect get ref from the DSI > encoder hook. Does this fix [1] which is v5.11-rc2 on TGL DSI? Should we pick this up for fixes? BR, Jani. [1] https://intel-gfx-ci.01.org/tree/drm-intel-fixes/CI_DIF_538/fi-tgl-dsi/igt@gem_exec_suspend@xxxxxxxxxxxxx > > Cc: Vandita Kulkarni <vandita.kulkarni@xxxxxxxxx> > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx> > --- > drivers/gpu/drm/i915/display/icl_dsi.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/icl_dsi.c b/drivers/gpu/drm/i915/display/icl_dsi.c > index a9439b415603..b3533a32f8ba 100644 > --- a/drivers/gpu/drm/i915/display/icl_dsi.c > +++ b/drivers/gpu/drm/i915/display/icl_dsi.c > @@ -1616,10 +1616,6 @@ static void gen11_dsi_get_power_domains(struct intel_encoder *encoder, > > get_dsi_io_power_domains(i915, > enc_to_intel_dsi(encoder)); > - > - if (crtc_state->dsc.compression_enable) > - intel_display_power_get(i915, > - intel_dsc_power_domain(crtc_state)); > } > > static bool gen11_dsi_get_hw_state(struct intel_encoder *encoder, -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
