Re: [i-g-t 2/2] Revert "lib/igt_psr: Fix PR state check for DisplayPort connectors"

["Naladala, Ramanaidu" <ramanaidu.naladala@intel.com>]

Acked-by: Naladala Ramanaidu <ramanaidu.naladala@intel.com>

On 9/11/2024 1:27 PM, Bhanuprakash Modem wrote:
> Causing regression in CI BAT.
> 
> This reverts commit 196c2cc9c945a66bb542fdefcf491ea62c1f885e.
> 
> Issue: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/2748
> Cc: Naladala Ramanaidu <ramanaidu.naladala@intel.com>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
> ---
>   lib/igt_psr.c | 5 ++---
>   1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/igt_psr.c b/lib/igt_psr.c
> index 4e44f44e6..e3e7577eb 100644
> --- a/lib/igt_psr.c
> +++ b/lib/igt_psr.c
> @@ -79,9 +79,8 @@ static bool psr_active_check(int debugfs_fd, enum psr_mode mode, igt_output_t *o
>   		c = output->config.connector;
>   	}
>   
> -	if (c->connector_type == DRM_MODE_CONNECTOR_DisplayPort)
> -		state = "SRDENT_ON";
> -	else if (mode == PR_MODE || mode == PR_MODE_SEL_FETCH)
> +	if ((mode == PR_MODE || mode == PR_MODE_SEL_FETCH) &&
> +	    c && c->connector_type == DRM_MODE_CONNECTOR_eDP)
>   		state = "SLEEP";
>   	else if (mode == PSR_MODE_1 || mode == PR_MODE || mode == PR_MODE_SEL_FETCH)
>   		state = "SRDENT";