[PATCH i-g-t] tests/intel/kms_psr2_sf: Add eDP check for Early Trasport

Naladala, Ramanaidu ramanaidu.naladala at intel.com
Thu Sep 19 10:21:40 UTC 2024



On 9/19/2024 3:29 PM, Jeevan B wrote:
> On current platfroms early trasport is supported on eDP only

fix typo errors 'platforms' and 'transport'.

> so avoiding assert for other connector.
> 
> Signed-off-by: Jeevan B <jeevan.b at intel.com>
> ---
>   tests/intel/kms_psr2_sf.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/intel/kms_psr2_sf.c b/tests/intel/kms_psr2_sf.c
> index a9be1a1ec..14b3e20ff 100644
> --- a/tests/intel/kms_psr2_sf.c
> +++ b/tests/intel/kms_psr2_sf.c
> @@ -935,7 +935,8 @@ static void run(data_t *data)
>   							  data->pipe),
>   							  "FBC still disabled\n");
>   
> -	if (data->et_flag)

Add a TODO to enable early transport in feature platforms.
imho: To handle data->output = NULL cases add a check as below to handle 
crash issues.
if (data->et_flag && data->output != NULL && 
data->output->config.connector->connector_type == DRM_MODE_CONNECTOR_eDP)

> +	if (data->et_flag && data->output->config.connector->connector_type ==
> +	    DRM_MODE_CONNECTOR_eDP)
>   		igt_assert_f(early_transport_check(data->debugfs_fd),
>   			     "Early Trasport Disbaled\n");
>   



More information about the igt-dev mailing list