Re: [igt-dev] [PATCH i-g-t] tests/kms_panel_fitting: test requires eDP

["Ville Syrjälä" <ville.syrjala@linux.intel.com>]

On Mon, Jan 25, 2021 at 12:33:36PM +0530, Nidhi Gupta wrote:
> Added a condition to check the availability of eDP
> before execution of the test. Skip the test with
> proper debug messsage if eDP is not there
> 
> Signed-off-by: Nidhi Gupta <nidhi1.gupta@intel.com>
> ---
>  tests/kms_panel_fitting.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/kms_panel_fitting.c b/tests/kms_panel_fitting.c
> index 4f4c5608..9f6b00e5 100644
> --- a/tests/kms_panel_fitting.c
> +++ b/tests/kms_panel_fitting.c
> @@ -254,7 +254,10 @@ igt_main
>  	igt_fixture {
>  		data.drm_fd = drm_open_driver_master(DRIVER_ANY);
>  		igt_display_require(&data.display, data.drm_fd);
> -		igt_display_require_output(&data.display);
> +
> +		if (data.display.outputs->config.connector->connector_type !=
> +		    DRM_MODE_CONNECTOR_eDP)
> +			igt_skip("Can't test: no usable eDP screen.\n");

No, it just wants an output type where we've hooked up the pfit.
It already checks for that by looking for the scaling mode prop.

>  	}
>  
>  	igt_subtest("legacy")
> -- 
> 2.26.2
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev

-- 
Ville Syrjälä
Intel
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev