From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06b.intel.com [134.134.136.31]) by gabe.freedesktop.org (Postfix) with ESMTPS id F1C9210E33D for ; Tue, 31 Jan 2023 12:07:02 +0000 (UTC) From: Ville Syrjala To: igt-dev@lists.freedesktop.org Date: Tue, 31 Jan 2023 14:06:44 +0200 Message-Id: <20230131120646.27434-5-ville.syrjala@linux.intel.com> In-Reply-To: <20230131120646.27434-1-ville.syrjala@linux.intel.com> References: <20230131120646.27434-1-ville.syrjala@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t 4/6] tests/kms_async_flips: Run the test only once per pipe List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Arun R Murthy Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: From: Ville Syrjälä No real reason to iterate over all the outputs here. Just do the test once per pipe. Cc: Arun R Murthy Signed-off-by: Ville Syrjälä --- tests/kms_async_flips.c | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/tests/kms_async_flips.c b/tests/kms_async_flips.c index f8c97e029c59..54611f66350d 100644 --- a/tests/kms_async_flips.c +++ b/tests/kms_async_flips.c @@ -544,20 +544,12 @@ static void test_crc(data_t *data) static void run_test(data_t *data, void (*test)(data_t *)) { - igt_output_t *output; - enum pipe pipe; + for_each_pipe_with_valid_output(&data->display, data->pipe, data->output) { + igt_dynamic_f("pipe-%s", kmstest_pipe_name(data->pipe)) + test(data); - for_each_pipe(&data->display, pipe) { - for_each_valid_output_on_pipe(&data->display, pipe, output) { - igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), output->name) { - data->output = output; - data->pipe = pipe; - test(data); - } - - if (!data->extended) - break; - } + if (!data->extended) + break; } } -- 2.39.1