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 <[email protected]> > > Bspec: 50351 > Cc: Ville Syrjälä <[email protected]> > Fixes: ba3b049f4774 ("drm/i915/adl_p: Allow underrun recovery when possible") > Signed-off-by: Matt Roper <[email protected]> > --- > 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; > } >
