Re: [PATCH v2 5/5] drm/i915/adlp+/dp_mst: Align master transcoder disabling with spec wrt. DP2 config

[Imre Deak <imre.deak@intel.com>]

On Thu, Oct 31, 2024 at 12:56:15PM +0200, Luca Coelho wrote:
> On Wed, 2024-10-30 at 21:23 +0200, Imre Deak wrote:
> > On ADLP+ during modeset disabling, disable the DP2 configuration for MST
> > master transcoders as required by the specification.
> > 
> > Bspec: 55424, 54128, 65448, 68849
> 
> Just curious how this change is related to so many bspecs...?

The Display Port modeset sequences - where the programming step added
here is described - are different or at least described separately for
different platforms on the above pages.

> > Signed-off-by: Imre Deak <imre.deak@intel.com>
> > ---
> >  drivers/gpu/drm/i915/display/intel_ddi.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c b/drivers/gpu/drm/i915/display/intel_ddi.c
> > index d82bc1bf8b68f..6adbc7d0b90d9 100644
> > --- a/drivers/gpu/drm/i915/display/intel_ddi.c
> > +++ b/drivers/gpu/drm/i915/display/intel_ddi.c
> > @@ -3062,6 +3062,8 @@ static void intel_ddi_post_disable_dp(struct intel_atomic_state *state,
> >  
> >  	intel_dp_sink_set_fec_ready(intel_dp, old_crtc_state, false);
> >  
> > +	intel_ddi_config_transcoder_dp2(old_crtc_state, false);
> > +
> >  	/*
> >  	 * From TGL spec: "If single stream or multi-stream master transcoder:
> >  	 * Configure Transcoder Clock select to direct no clock to the
> 
> Regardless:
> 
> Reviewed-by: Luca Coelho <luciano.coelho@intel.com>
> 
> --
> Cheers,
> Luca.