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 EBA3310E13D for ; Tue, 15 Nov 2022 12:56:13 +0000 (UTC) From: Nidhi Gupta To: igt-dev@lists.freedesktop.org Date: Tue, 15 Nov 2022 18:30:49 +0530 Message-Id: <1668517250-4514-3-git-send-email-nidhi1.gupta@intel.com> In-Reply-To: <1668517250-4514-1-git-send-email-nidhi1.gupta@intel.com> References: <1668517250-4514-1-git-send-email-nidhi1.gupta@intel.com> Subject: [igt-dev] [PATCH 2/3] tests/i915/i915_pm_backlight: Add a common structure for all subtests List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Nidhi Gupta Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Create a common structure for all the subtests which includes the test name, test description and flag indicating whether the test is suspend test or dpms test. And then create the array of the above structure and iterate over it. Signed-off-by: Nidhi Gupta --- tests/i915/i915_pm_backlight.c | 78 ++++++++++++++++++++---------------------- 1 file changed, 38 insertions(+), 40 deletions(-) diff --git a/tests/i915/i915_pm_backlight.c b/tests/i915/i915_pm_backlight.c index a482e2a..e94214a 100644 --- a/tests/i915/i915_pm_backlight.c +++ b/tests/i915/i915_pm_backlight.c @@ -43,6 +43,12 @@ struct context { char path[PATH_MAX]; }; +enum { + TEST_NONE = 0, + TEST_DPMS, + TEST_SUSPEND, +}; + #define TOLERANCE 5 /* percent */ #define BACKLIGHT_PATH "/sys/class/backlight" @@ -160,7 +166,7 @@ static void test_fade(struct context *context) } static void -test_fade_with_dpms(struct context *context, igt_output_t *output) +check_dpms(igt_output_t *output) { igt_require(igt_setup_runtime_pm(output->display->drm_fd)); @@ -173,16 +179,12 @@ test_fade_with_dpms(struct context *context, igt_output_t *output) output->config.connector, DRM_MODE_DPMS_ON); igt_assert(igt_wait_for_pm_status(IGT_RUNTIME_PM_STATUS_ACTIVE)); - - test_fade(context); } static void -test_fade_with_suspend(struct context *context, igt_output_t *output) +check_suspend(igt_output_t *output) { igt_system_suspend_autoresume(SUSPEND_STATE_MEM, SUSPEND_TEST_NONE); - - test_fade(context); } static void test_cleanup(igt_display_t *display, igt_output_t *output) @@ -231,6 +233,18 @@ igt_main char file_path_n[PATH_MAX] = ""; bool dual_edp = false; struct context contexts[NUM_EDP_OUTPUTS]; + struct { + const char *name; + const char *desc; + void (*test_t) (struct context *); + int flags; + } tests[] = { + { "basic-brightness", "test the basic brightness.", test_brightness, TEST_NONE }, + { "bad-brightness", "test the bad brightness.", test_bad_brightness, TEST_NONE }, + { "fade", "test basic fade.", test_fade, TEST_NONE }, + { "fade-with-dpms", "test the fade with DPMS.", test_fade, TEST_DPMS }, + { "fade-with-suspend", "test the fade with suspend.", test_fade, TEST_SUSPEND }, + }; igt_fixture { bool found = false; @@ -289,45 +303,29 @@ igt_main igt_require_f(found, "No valid output found.\n"); } - igt_subtest("basic-brightness") { - for (i = 0; i < (dual_edp ? 2 : 1); i++) { - test_setup(display, &contexts->output[i]); - test_brightness(&contexts[i]); - test_cleanup(&display, output); - } - } + for (i = 0; i < ARRAY_SIZE(tests); i++) { + igt_describe(tests[i].desc); + igt_subtest(tests[i].name) { + for (int j = 0; j < (dual_edp ? 2 : 1); j++) { + test_setup(display, &contexts->output[j]); + + if (backlight_read(&old[j], "brightness", &contexts[j])) + continue; + + if (tests[i].flags == TEST_DPMS) + check_dpms(contexts[j].output); + + if (tests[i].flags == TEST_SUSPEND) + check_suspend(contexts[j].output); + + igt_assert(backlight_read(&contexts[j].max, "max_brightness", &contexts[j]) > -1); + tests[i].test_t(&contexts[j]); + } - igt_subtest("bad-brightness") { - for (i = 0; i < (dual_edp ? 2 : 1); i++) { - test_setup(display, &contexts->output[i]); - test_bad_brightness(&contexts[i]); - test_cleanup(&display, output); - } - } - igt_subtest("fade") { - for (i = 0; i < (dual_edp ? 2 : 1); i++) { - test_setup(display, &contexts->output[i]); - test_fade(&contexts[i]); - test_cleanup(&display, output); - } - } - igt_subtest("fade_with_dpms") { - for (i = 0; i < (dual_edp ? 2 : 1); i++) { - test_setup(display, &contexts->output[i]); - test_fade_with_dpms(&contexts[i], output); - test_cleanup(&display, output); - } - } - igt_subtest("fade_with_suspend") { - for (i = 0; i < (dual_edp ? 2 : 1); i++) { - test_setup(display, &contexts->output[i]); - test_fade_with_suspend(&contexts[i], output); test_cleanup(&display, output); } } - - igt_fixture { /* Restore old brightness */ for (int j = 0; j < (dual_edp ? 2 : 1); j++) -- 1.9.1