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 B6C7510E04D for ; Tue, 28 Mar 2023 08:39:14 +0000 (UTC) Received: from linux.intel.com (unknown [10.252.28.3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by linux.intel.com (Postfix) with ESMTPS id 65DD5580C25 for ; Tue, 28 Mar 2023 01:39:13 -0700 (PDT) Received: from maurocar by linux.intel.com with local (Exim 4.96) (envelope-from ) id 1ph4rG-00AOHB-30 for igt-dev@lists.freedesktop.org; Tue, 28 Mar 2023 10:39:10 +0200 From: Mauro Carvalho Chehab To: igt-dev@lists.freedesktop.org Date: Tue, 28 Mar 2023 10:39:08 +0200 Message-Id: <20230328083908.2476581-1-mauro.chehab@linux.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t] xe/xe_huc_copy: use IS_TIGERLAKE macro List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: From: Mauro Carvalho Chehab Instead of hardcoding the PCI IDs at the test, use a macro to check if the platform is compatible with the test. Signed-off-by: Mauro Carvalho Chehab --- tests/xe/xe_huc_copy.c | 38 +------------------------------------- 1 file changed, 1 insertion(+), 37 deletions(-) diff --git a/tests/xe/xe_huc_copy.c b/tests/xe/xe_huc_copy.c index ee3896cef8b9..cd68dbb5ac50 100644 --- a/tests/xe/xe_huc_copy.c +++ b/tests/xe/xe_huc_copy.c @@ -152,42 +152,6 @@ test_huc_copy(int fd) xe_vm_destroy(fd, vm); } -static bool -is_device_supported(int fd) -{ - struct drm_xe_query_config *config; - struct drm_xe_device_query query = { - .extensions = 0, - .query = DRM_XE_DEVICE_QUERY_CONFIG, - .size = 0, - .data = 0, - }; - uint16_t devid; - - igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_XE_DEVICE_QUERY, &query), 0); - - config = malloc(query.size); - igt_assert(config); - - query.data = to_user_pointer(config); - igt_assert_eq(igt_ioctl(fd, DRM_IOCTL_XE_DEVICE_QUERY, &query), 0); - - devid = config->info[XE_QUERY_CONFIG_REV_AND_DEVICE_ID] & 0xffff; - return ( - devid == 0x9A60 || - devid == 0x9A68 || - devid == 0x9A70 || - devid == 0x9A40 || - devid == 0x9A49 || - devid == 0x9A59 || - devid == 0x9A78 || - devid == 0x9AC0 || - devid == 0x9AC9 || - devid == 0x9AD9 || - devid == 0x9AF8 - ); -} - igt_main { int xe; @@ -198,7 +162,7 @@ igt_main } igt_subtest("huc_copy") { - igt_skip_on(!is_device_supported(xe)); + igt_skip_on(!IS_TIGERLAKE(intel_get_drm_devid(xe))); test_huc_copy(xe); } -- 2.39.2