Re: [igt-dev] [i-g-t v5 42/52] tests/i915/kms_draw_crc: Add support for Bigjoiner

["Gupta, Nidhi1" <nidhi1.gupta@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1692 bytes --]

On Tue, 2022-11-15 at 08:43 +0530, Bhanuprakash Modem wrote:
>This patch will add a check to Skip the subtest if a selected pipe/output
>combo won't support Bigjoiner or 8K mode.
>
>Example:
>* Pipe-D wont support a mode > 5K
>* To use 8K mode on a pipe then consecutive pipe must be available & free.
>
>V2: - Use updated helper name
>
>Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
>---
Reviewed-by: Nidhi Gupta <nidhi1.gupta@intel.com>
>tests/i915/kms_draw_crc.c | 10 ++++------
>1 file changed, 4 insertions(+), 6 deletions(-)
>diff --git a/tests/i915/kms_draw_crc.c b/tests/i915/kms_draw_crc.c
>index c2ac4721..01c3808c 100644
>--- a/tests/i915/kms_draw_crc.c
>+++ b/tests/i915/kms_draw_crc.c
>@@ -59,16 +59,14 @@  static void find_modeset_params(void)
> {
>             enum pipe pipe;
>
>-           igt_display_reset(&display);
>-           igt_display_commit(&display);
>-
>             for_each_pipe_with_valid_output(&display, pipe, output) {
>-                          igt_output_set_pipe(output, pipe);
>+                          igt_display_reset(&display);
>
>-                          mode = igt_output_get_mode(output);
>-                          if (!mode)
>+                          igt_output_set_pipe(output, pipe);
>+                          if (!i915_pipe_output_combo_valid(&display))
>                                           continue;
>
>+                          mode = igt_output_get_mode(output);
>                            pipe_crc = igt_pipe_crc_new(drm_fd, pipe, IGT_PIPE_CRC_SOURCE_AUTO);
>                            /*Only one pipe/output is enough*/
>                            break;

[-- Attachment #2: Type: text/html, Size: 6747 bytes --]