From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1BEB410E195 for ; Thu, 12 Jan 2023 10:57:05 +0000 (UTC) From: Dnyaneshwar Bhadane To: igt-dev@lists.freedesktop.org Date: Thu, 12 Jan 2023 16:27:02 +0530 Message-Id: <20230112105702.19502-1-dnyaneshwar.bhadane@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [i-g-t] tests/kms_cursor_crc: skip pipe on invalid connector in cursor size test List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: suresh.kumar.kurmi@intel.com, dnyaneshwar.bhadane@intel.com, juha-pekka.heikkila@intel.com Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Only the valid pipe connector combination reach to the igt commit. Cursor max-size test will not affect existing flow as only skip for invalid connector. For cursor-dpms and cursor-suspend not require to check require_cursor_size becuase the cursor height and width used from drm capablities. Signed-off-by: Dnyaneshwar Bhadane --- tests/kms_cursor_crc.c | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index d5a4b30b..4ab4c005 100644 --- a/tests/kms_cursor_crc.c +++ b/tests/kms_cursor_crc.c @@ -752,6 +752,13 @@ static void run_size_tests(data_t *data, int w, int h) for_each_pipe(&data->display, pipe) { data->pipe = pipe; + if(!igt_pipe_connector_valid(pipe, data->output)) { + igt_debug("Invalid connector on pipe-%s-%s\n", + kmstest_pipe_name(pipe), + data->output->name); + continue; + } + if (require_cursor_size(data, w, h)) { igt_info("Cursor size %dx%d not supported by driver\n", w, h); continue; @@ -854,9 +861,10 @@ static void run_tests_on_pipe(data_t *data) data->pipe = pipe; data->flags = TEST_DPMS; - if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) { - igt_debug("Cursor size %dx%d not supported by driver\n", - data->cursor_max_w, data->cursor_max_h); + if(!igt_pipe_connector_valid(pipe, data->output)) { + igt_debug("Invalid connector on pipe-%s-%s\n", + kmstest_pipe_name(pipe), + data->output->name); continue; } @@ -875,9 +883,10 @@ static void run_tests_on_pipe(data_t *data) data->pipe = pipe; data->flags = TEST_SUSPEND; - if (require_cursor_size(data, data->cursor_max_w, data->cursor_max_h)) { - igt_debug("Cursor size %dx%d not supported by driver\n", - data->cursor_max_w, data->cursor_max_h); + if(!igt_pipe_connector_valid(pipe, data->output)) { + igt_debug("Invalid connector on pipe-%s-%s\n", + kmstest_pipe_name(pipe), + data->output->name); continue; } -- 2.35.1