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@xxxxxxxxx>
>
> Bspec: 50351
> Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> Fixes: ba3b049f4774 ("drm/i915/adl_p: Allow underrun recovery when possible")
> Signed-off-by: Matt Roper <matthew.d.roper@xxxxxxxxx>
> ---
> 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;
> }
>
