From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM04-BN8-obe.outbound.protection.outlook.com (mail-bn8nam04on2075.outbound.protection.outlook.com [40.107.100.75]) by gabe.freedesktop.org (Postfix) with ESMTPS id 7212510E17B for ; Mon, 24 Apr 2023 14:16:34 +0000 (UTC) From: Hersen Wu To: , , , , Date: Mon, 24 Apr 2023 10:16:26 -0400 Message-ID: <20230424141626.78069-1-hersenxs.wu@amd.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain Subject: [igt-dev] [PATCH] [i-g-t] tests/amdgpu/amd_link_settings: use mode 640x480 for link configuration test List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Hersen Wu Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: with 4k dp connected, amdgpu kernel warning mesaage dp_blank timeout happens intermittently. link configuration <1, 162> could not support 4k mode. dp safe mode 640x480@60hz could be used for all link configuration test. Signed-off-by: Hersen Wu --- tests/amdgpu/amd_link_settings.c | 25 ++++++++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/tests/amdgpu/amd_link_settings.c b/tests/amdgpu/amd_link_settings.c index 14c8c4d7..ec5e6fe5 100644 --- a/tests/amdgpu/amd_link_settings.c +++ b/tests/amdgpu/amd_link_settings.c @@ -159,6 +159,24 @@ static void run_link_training_config(data_t *data, igt_output_t *output) } } +static const drmModeModeInfo dp_safe_mode_640_480 = { + .name = "640x480", + .vrefresh = 60, + .clock = 25200, + + .hdisplay = 640, + .hsync_start = 656, + .hsync_end = 752, + .htotal = 800, + + .vdisplay = 480, + .vsync_start = 490, + .vsync_end = 492, + .vtotal = 525, + + .flags = DRM_MODE_FLAG_NHSYNC | DRM_MODE_FLAG_NVSYNC, +}; + static void test_link_training_configs(data_t *data) { const drmModeModeInfo *orig_mode; @@ -177,7 +195,12 @@ static void test_link_training_configs(data_t *data) /* Init only if display supports link_settings */ test_init(data, output); - orig_mode = igt_output_get_mode(output); + /* run_link_training_config run test from <1, 1.62> + * to highest link configuration. to make sure mode timing + * be fitted into <1, 1.62> and higher configuration, use + * dp safe mode 640x480@60hz + */ + orig_mode = &dp_safe_mode_640_480; igt_assert(orig_mode); igt_output_override_mode(output, orig_mode); -- 2.25.1