Re: [PATCH i-g-t 2/2] Revert "tests/kms_display_modes: Skip test if joiner display is connected"

[Karthik B S <karthik.b.s@intel.com>]

On 4/3/2025 4:11 PM, Jeevan B wrote:
> bigjoiner realted checks should be handled in igt_check_bigjoiner_support
> which is now added so removing the checks which is no more valid.

Nit: Although the commit message conveys the required info, please make 
it more readable.

With this,

Reviewed-by: Karthik B S <karthik.b.s@intel.com>

>
> This reverts commit 5f7cde6a98783c97649bdb3879bc016b3bf38297
>
> Signed-off-by: Jeevan B <jeevan.b@intel.com>
> ---
>   tests/kms_display_modes.c | 9 ---------
>   1 file changed, 9 deletions(-)
>
> diff --git a/tests/kms_display_modes.c b/tests/kms_display_modes.c
> index 588820170..d53b83a86 100644
> --- a/tests/kms_display_modes.c
> +++ b/tests/kms_display_modes.c
> @@ -184,11 +184,6 @@ static void run_extendedmode_test(data_t *data) {
>   	for_each_pipe(display, pipe1) {
>   		for_each_valid_output_on_pipe(display, pipe1, output1) {
>   
> -			if (is_joiner_mode(data->drm_fd, output1)) {
> -				igt_info("Skipping joiner output %s", output1->name);
> -				continue;
> -			}
> -
>   			for_each_pipe(display, pipe2) {
>   				if (pipe1 == pipe2)
>   					continue;
> @@ -238,10 +233,6 @@ igt_main
>   		igt_display_require_output(&data.display);
>   
>   		for_each_connected_output(&data.display, output) {
> -			if (is_joiner_mode(data.drm_fd, output)) {
> -				igt_info("Skipping joiner output %s", output->name);
> -				continue;
> -			}
>   			data.mst_output[count++] = output;
>   			if (output_is_dp_mst(&data, output, dp_mst_outputs))
>   				dp_mst_outputs++;