[PATCH i-g-t] tests/intel/kms_pipe_stress: Skip invalid pipe-output combinations

[Sharma, Swati2]

Hi Jeevan,

On 11-07-2025 02:50 pm, Jeevan B wrote:
> Add intel_pipe_output_combo_valid check to skip invalid pipe-output
> combinations which are not valid.
>
> Signed-off-by: Jeevan B <jeevan.b at intel.com>
> ---
>   tests/intel/kms_pipe_stress.c | 5 +++++
>   1 file changed, 5 insertions(+)
>
> diff --git a/tests/intel/kms_pipe_stress.c b/tests/intel/kms_pipe_stress.c
> index 296c93b58..a896c830b 100644
> --- a/tests/intel/kms_pipe_stress.c
> +++ b/tests/intel/kms_pipe_stress.c
> @@ -578,6 +578,11 @@ static void stress_pipes(struct data *data, struct timespec *start,
>   		if (!data->highest_mode[pipe])
>   			continue;
>   
> +		if (!intel_pipe_output_combo_valid(&data->display)) {
> +			igt_output_set_pipe(output, PIPE_NONE);
> +			continue;
> +		}
We are setting pipe in pipe_stress(); how are we checking valid_combo() 
before?
First, we need to set pipe then only we can check valid_combo.

Also, please add which bug is getting closed with this fix.
> +
>   		igt_assert_f(data->display.pipes[pipe].n_planes < MAX_PLANES,
>   			    "Currently we don't support more than %d planes!",
>   			     MAX_PLANES);