On Thu, 03 Jun 2021, "Kulkarni, Vandita" <[email protected]> wrote: >> -----Original Message----- >> From: Saarinen, Jani <[email protected]> >> Sent: Thursday, June 3, 2021 1:07 PM >> To: Kulkarni, Vandita <[email protected]>; intel- >> [email protected] >> Cc: Nikula, Jani <[email protected]> >> Subject: RE: [Intel-gfx] [PATCH] drm/i915/dsc: Remove redundant checks in >> DSC disable >> >> Hi, >> > -----Original Message----- >> > From: Intel-gfx <[email protected]> On Behalf Of >> > Vandita Kulkarni >> > Sent: torstai 3. kesäkuuta 2021 9.54 >> > To: [email protected] >> > Cc: Nikula, Jani <[email protected]> >> > Subject: [Intel-gfx] [PATCH] drm/i915/dsc: Remove redundant checks in >> > DSC disable >> > >> > There can be a chance that pre os has enabled DSC and driver's compute >> > config would not need dsc to be enabled, in such case if we check on >> > compute config's compression state to disable, we might end up in state >> mismatch. >> >> I assume this fixes real gitlab issue too? > Okay, will add the tag > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/3537
See https://lore.kernel.org/r/[email protected] The problem is with ->bigjoiner, not the entire statement. BR, Jani. > > Thanks, > Vandita >> >> > >> > Signed-off-by: Vandita Kulkarni <[email protected]> >> > --- >> > drivers/gpu/drm/i915/display/intel_vdsc.c | 4 ---- >> > 1 file changed, 4 deletions(-) >> > >> > diff --git a/drivers/gpu/drm/i915/display/intel_vdsc.c >> > b/drivers/gpu/drm/i915/display/intel_vdsc.c >> > index 19cd9531c115..b05a96011d93 100644 >> > --- a/drivers/gpu/drm/i915/display/intel_vdsc.c >> > +++ b/drivers/gpu/drm/i915/display/intel_vdsc.c >> > @@ -1161,10 +1161,6 @@ void intel_dsc_disable(const struct >> > intel_crtc_state >> > *old_crtc_state) >> > struct intel_crtc *crtc = to_intel_crtc(old_crtc_state->uapi.crtc); >> > struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); >> > >> > - if (!(old_crtc_state->dsc.compression_enable && >> > - old_crtc_state->bigjoiner)) >> > - return; >> > - >> > intel_de_write(dev_priv, dss_ctl1_reg(old_crtc_state), 0); >> > intel_de_write(dev_priv, dss_ctl2_reg(old_crtc_state), 0); } >> > -- >> > 2.21.0.5.gaeb582a >> > >> > _______________________________________________ >> > Intel-gfx mailing list >> > [email protected] >> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
