public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
To: igt-dev@lists.freedesktop.org
Subject: [igt-dev] [i-g-t v6 19/50] tests/kms_pipe_crc_basic: Add support for Bigjoiner
Date: Fri, 30 Dec 2022 23:17:54 +0530	[thread overview]
Message-ID: <20221230174825.3016716-20-bhanuprakash.modem@intel.com> (raw)
In-Reply-To: <20221230174825.3016716-1-bhanuprakash.modem@intel.com>

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: Mohammed Thasleem <mohammed.thasleem@intel.com>
---
 tests/kms_pipe_crc_basic.c | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/tests/kms_pipe_crc_basic.c b/tests/kms_pipe_crc_basic.c
index 91a1b8ab..ffc77eb3 100644
--- a/tests/kms_pipe_crc_basic.c
+++ b/tests/kms_pipe_crc_basic.c
@@ -274,6 +274,21 @@ static void test_disable_crc_after_crtc(data_t *data, enum pipe pipe,
 	igt_remove_fb(data->drm_fd, &data->fb);
 }
 
+static bool pipe_output_combo_valid(igt_display_t *display,
+				    enum pipe pipe, igt_output_t *output)
+{
+	bool ret = true;
+
+	igt_display_reset(display);
+
+	igt_output_set_pipe(output, pipe);
+	if (!i915_pipe_output_combo_valid(display))
+		ret = false;
+	igt_output_set_pipe(output, PIPE_NONE);
+
+	return ret;
+}
+
 data_t data = {0, };
 
 igt_main
@@ -322,6 +337,9 @@ igt_main
 		igt_describe(tests[i].desc);
 		igt_subtest_with_dynamic(tests[i].name) {
 			for_each_pipe_with_single_output(&data.display, pipe, output) {
+				if(!pipe_output_combo_valid(&data.display, pipe, output))
+					continue;
+
 				igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), output->name) {
 					if (tests[i].flags & TEST_SUSPEND) {
 						test_read_crc(&data, pipe, output, 0);
@@ -350,6 +368,9 @@ igt_main
 		     "does not cause issues.");
 	igt_subtest_with_dynamic("disable-crc-after-crtc") {
 		for_each_pipe_with_single_output(&data.display, pipe, output) {
+			if(!pipe_output_combo_valid(&data.display, pipe, output))
+				continue;
+
 			igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), output->name)
 				test_disable_crc_after_crtc(&data, pipe, output);
 		}
@@ -358,6 +379,9 @@ igt_main
 	igt_describe("Basic sanity check for CRC mismatches");
 	igt_subtest_with_dynamic("compare-crc-sanitycheck") {
 		for_each_pipe_with_single_output(&data.display, pipe, output) {
+			if(!pipe_output_combo_valid(&data.display, pipe, output))
+				continue;
+
 			igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), output->name)
 				test_compare_crc(&data, pipe, output);
 		}
-- 
2.39.0

  parent reply	other threads:[~2022-12-30 17:52 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-30 17:47 [igt-dev] [i-g-t v6 00/50] Add IGT support for Bigjoiner Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 01/50] lib/igt_kms: Add a helper for valid pipe/output constraint Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 02/50] lib/igt_kms: Bigjoiner check in max_bpc constraint Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 03/50] tests/kms_async_flips: Add support for Bigjoiner Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 04/50] tests/kms_atomic: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 05/50] tests/kms_atomic_interruptible: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 06/50] tests/kms_atomic_transition: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 07/50] tests/kms_color_helper: Helper to check Bigjoiner constraint Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 08/50] tests/kms_color: Add support for Bigjoiner Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 09/50] tests/chamelium/kms_color_chamelium: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 10/50] tests/kms_concurrent: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 11/50] tests/kms_cursor_crc: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 12/50] tests/kms_cursor_edge_walk: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 13/50] tests/kms_cursor_legacy: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 14/50] tests/kms_display_modes: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 15/50] tests/kms_dp_tiled_display: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 16/50] tests/kms_flip_event_leak: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 17/50] tests/kms_invalid_mode: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 18/50] tests/kms_lease: " Bhanuprakash Modem
2022-12-30 17:47 ` Bhanuprakash Modem [this message]
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 20/50] tests/kms_plane: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 21/50] tests/kms_plane_alpha_blend: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 22/50] tests/kms_plane_cursor: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 23/50] tests/kms_plane_lowres: " Bhanuprakash Modem
2022-12-30 17:47 ` [igt-dev] [i-g-t v6 24/50] tests/kms_plane_multiple: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 25/50] tests/kms_plane_scaling: " Bhanuprakash Modem
2023-01-04  7:36   ` Swati Sharma
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 26/50] tests/kms_prime: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 27/50] tests/kms_properties: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 28/50] tests/kms_rmfb: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 29/50] tests/kms_rotation_crc: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 30/50] tests/kms_scaling_modes: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 31/50] tests/kms_sequence: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 32/50] tests/kms_universal_plane: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 33/50] tests/kms_vblank: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 34/50] tests/kms_vrr: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 35/50] tests/i915/kms_big_fb: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 36/50] tests/i915/kms_big_joiner: Negative tests " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 37/50] tests/i915/kms_busy: Add support " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 38/50] tests/i915/kms_ccs: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 39/50] tests/i915/kms_cdclk: " Bhanuprakash Modem
2023-01-04  7:33   ` Swati Sharma
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 40/50] tests/i915/kms_draw_crc: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 41/50] tests/i915/kms_fence_pin_leak: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 42/50] tests/i915/kms_flip_scaled_crc: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 43/50] tests/i915/kms_flip_tiling: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 44/50] tests/i915/kms_frontbuffer_tracking: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 45/50] tests/i915/kms_mmap_write_crc: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 46/50] tests/i915/kms_psr2_sf: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 47/50] tests/i915/kms_psr2_su: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 48/50] tests/i915/kms_psr: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 49/50] tests/i915/kms_psr_stress_test: " Bhanuprakash Modem
2022-12-30 17:48 ` [igt-dev] [i-g-t v6 50/50] tests/i915/kms_pwrite_crc: " Bhanuprakash Modem
2022-12-30 18:30 ` [igt-dev] ✓ Fi.CI.BAT: success for Add IGT support for Bigjoiner (rev17) Patchwork
2022-12-30 20:01 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20221230174825.3016716-20-bhanuprakash.modem@intel.com \
    --to=bhanuprakash.modem@intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox