Re: [Intel-gfx] [PATCH] drm/i915/adl_p: Also disable underrun recovery with MSO

["Souza, Jose" <jose.souza@intel.com>]

On Mon, 2021-08-16 at 13:41 -0700, Matt Roper wrote:
> One of the cases that the bspec lists for when underrun recovery must be
> disabled is "COG;" that note actually refers to eDP multi-segmented
> operation (MSO).  Let's ensure the this additional restriction is
> honored by the driver.

Reviewed-by: José Roberto de Souza <jose.souza@intel.com>

> 
> Bspec: 50351
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Fixes: ba3b049f4774 ("drm/i915/adl_p: Allow underrun recovery when possible")
> Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_display.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> index a257e5dc381c..134a6acbd8fb 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -2226,6 +2226,9 @@ static bool underrun_recovery_supported(const struct intel_crtc_state *crtc_stat
>  	if (crtc_state->has_psr2)
>  		return false;
>  
> +	if (crtc_state->splitter.enable)
> +		return false;
> +
>  	return true;
>  }
>