Re: [PATCH v2 09/12] drm/i915/xe3: Underrun recovery does not exist post Xe2

["Pottumuttu, Sai Teja" <sai.teja.pottumuttu@intel.com>]

On 24-10-2024 03:16, Clint Taylor wrote:
> From: Ravi Kumar Vodapalli <ravi.kumar.vodapalli@intel.com>
>
>  From platforms xe3 Underrun recovery does not exist
>
> v2: improve DISPLAY_VER checking

Lets add a line here to separate commit message from git trailer and bspec.

With that addressed, LGTM

Reviewed-by: Sai Teja Pottumuttu <sai.teja.pottumuttu@intel.com>

> BSpec: 68849
> Signed-off-by: Ravi Kumar Vodapalli <ravi.kumar.vodapalli@intel.com>
> Signed-off-by: Matt Atwood <matthew.s.atwood@intel.com>
> Signed-off-by: Clint Taylor <Clinton.A.Taylor@intel.com>
> ---
>   drivers/gpu/drm/i915/display/intel_display.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
> index 01466611eebe..124e094a9999 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -861,7 +861,7 @@ static void icl_set_pipe_chicken(const struct intel_crtc_state *crtc_state)
>   	 */
>   	if (IS_DG2(dev_priv))
>   		tmp &= ~UNDERRUN_RECOVERY_ENABLE_DG2;
> -	else if (DISPLAY_VER(dev_priv) >= 13)
> +	else if ((DISPLAY_VER(dev_priv) >= 13) && (DISPLAY_VER(dev_priv) < 30))
>   		tmp |= UNDERRUN_RECOVERY_DISABLE_ADLP;
>   
>   	/* Wa_14010547955:dg2 */