From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by gabe.freedesktop.org (Postfix) with ESMTPS id 359B310E2CE for ; Fri, 6 Jan 2023 07:16:59 +0000 (UTC) From: Mohammed Thasleem To: igt-dev@lists.freedesktop.org Date: Thu, 30 Jun 2022 11:27:34 +0530 Message-Id: <20220630055734.7624-1-mohammed.thasleem@intel.com> In-Reply-To: <20221220135734.6354-1-mohammed.thasleem@intel.com> References: <20221220135734.6354-1-mohammed.thasleem@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t] tests/i915/i915_pm_dc : Check DC5 state with externel active panel List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: This test validates negative scenario of DC5 state by keeping all connectors's DPMS property set to ON. v2: Added function to check externel panel and skip test if not found. v3: Moved check_external_panel inside igt_require_f and removed igt_skip_on_f. Signed-off-by: Mohammed Thasleem --- tests/i915/i915_pm_dc.c | 45 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 45 insertions(+) diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index ba49d014..e29f9eb8 100644 --- a/tests/i915/i915_pm_dc.c +++ b/tests/i915/i915_pm_dc.c @@ -246,6 +246,13 @@ static void check_dc_counter(data_t *data, int dc_flag, uint32_t prev_dc_count) data->debugfs_dump = igt_sysfs_get(data->debugfs_fd, PWR_DOMAIN_INFO)); } +static void check_dc_counter_negative(data_t *data, int dc_flag, uint32_t prev_dc_count) +{ + igt_assert_f(!dc_state_wait_entry(data->debugfs_fd, dc_flag, prev_dc_count), + "%s state is achieved\n%s:\n%s\n", dc_state_name(dc_flag), PWR_DOMAIN_INFO, + data->debugfs_dump = igt_sysfs_get(data->debugfs_fd, PWR_DOMAIN_INFO)); +} + static void setup_videoplayback(data_t *data) { color_t red_green_blue[] = { @@ -411,6 +418,18 @@ static void test_dc_state_dpms(data_t *data, int dc_flag) cleanup_dc_dpms(data); } +static void test_dc_state_dpms_negative(data_t *data, int dc_flag) +{ + uint32_t dc_counter; + + require_dc_counter(data->debugfs_fd, dc_flag); + setup_dc_dpms(data); + dc_counter = read_dc_counter(data->debugfs_fd, dc_flag); + dpms_on(data); + check_dc_counter_negative(data, dc_flag, dc_counter); + cleanup_dc_dpms(data); +} + static bool support_dc6(int debugfs_fd) { char buf[4096]; @@ -485,6 +504,23 @@ static void test_dc9_dpms(data_t *data) setup_dc9_dpms(data, dc_target); } +static int check_external_panel(igt_display_t *display) +{ + igt_output_t *output = NULL; + int external_panel = 0; + + for_each_connected_output(display, output) { + drmModeConnectorPtr c = output->config.connector; + + if (c->connector_type != DRM_MODE_CONNECTOR_eDP && + c->connection == DRM_MODE_CONNECTED) { + external_panel++; + } + } + + return external_panel; +} + static void kms_poll_state_restore(int sig) { int sysfs_fd; @@ -552,6 +588,15 @@ igt_main test_dc_state_dpms(&data, CHECK_DC5); } + igt_describe("This test validates negative scenario of DC5 display " + "engine entry to DC5 state while all connectors's DPMS " + "property set to ON"); + igt_subtest("dc5-dpms-negative") { + igt_require_f(check_external_panel(&data.display), + "External panel not detected, skip execution\n"); + test_dc_state_dpms_negative(&data, CHECK_DC5); + } + igt_describe("This test validates display engine entry to DC5 state " "while all connectors's DPMS property set to OFF"); igt_subtest("dc6-dpms") { -- 2.25.1