[igt-dev] [i-g-t 02/79] tests/dsc: Move dsc helpers to lib

Bhanuprakash Modem bhanuprakash.modem at intel.com
Mon May 15 08:16:03 UTC 2023


The correct place for helpers is lib as the tests directory is
supposed to have test files, also having the helpers in tests
dir will break the compilation of testplan. Hence move all
helpers to library.

Expectation is to have an executable binary for each .c file
the "tests" dir.

Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
---
 tests/i915/kms_dsc_helper.c => lib/igt_dsc_helper.c | 2 +-
 tests/i915/kms_dsc_helper.h => lib/igt_dsc_helper.h | 0
 lib/meson.build                                     | 1 +
 tests/i915/kms_dsc.c                                | 2 +-
 tests/i915/kms_psr2_sf.c                            | 2 +-
 tests/meson.build                                   | 4 ++--
 6 files changed, 6 insertions(+), 5 deletions(-)
 rename tests/i915/kms_dsc_helper.c => lib/igt_dsc_helper.c (99%)
 rename tests/i915/kms_dsc_helper.h => lib/igt_dsc_helper.h (100%)

diff --git a/tests/i915/kms_dsc_helper.c b/lib/igt_dsc_helper.c
similarity index 99%
rename from tests/i915/kms_dsc_helper.c
rename to lib/igt_dsc_helper.c
index 02d1a4848..600a90746 100644
--- a/tests/i915/kms_dsc_helper.c
+++ b/lib/igt_dsc_helper.c
@@ -3,7 +3,7 @@
  * Copyright © 2023 Intel Corporation
  */
 
-#include "kms_dsc_helper.h"
+#include "igt_dsc_helper.h"
 
 static bool force_dsc_en_orig;
 static int force_dsc_restore_fd = -1;
diff --git a/tests/i915/kms_dsc_helper.h b/lib/igt_dsc_helper.h
similarity index 100%
rename from tests/i915/kms_dsc_helper.h
rename to lib/igt_dsc_helper.h
diff --git a/lib/meson.build b/lib/meson.build
index 847488f0e..3f42b40e0 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -100,6 +100,7 @@ lib_sources = [
 	'veboxcopy_gen12.c',
 	'igt_msm.c',
 	'igt_dsc.c',
+	'igt_dsc_helper.c',
 	'xe/xe_compute.c',
 	'xe/xe_compute_square_kernels.c',
 	'xe/xe_ioctl.c',
diff --git a/tests/i915/kms_dsc.c b/tests/i915/kms_dsc.c
index 3ce28f848..4af48b196 100644
--- a/tests/i915/kms_dsc.c
+++ b/tests/i915/kms_dsc.c
@@ -30,7 +30,7 @@
  *
  */
 
-#include "kms_dsc_helper.h"
+#include "igt_dsc_helper.h"
 
 IGT_TEST_DESCRIPTION("Test to validate display stream compression");
 
diff --git a/tests/i915/kms_psr2_sf.c b/tests/i915/kms_psr2_sf.c
index 4cf986440..7c21970a5 100644
--- a/tests/i915/kms_psr2_sf.c
+++ b/tests/i915/kms_psr2_sf.c
@@ -25,7 +25,7 @@
 #include "igt.h"
 #include "igt_sysfs.h"
 #include "igt_psr.h"
-#include "kms_dsc_helper.h"
+#include "igt_dsc_helper.h"
 #include <errno.h>
 #include <stdbool.h>
 #include <stdio.h>
diff --git a/tests/meson.build b/tests/meson.build
index 8fac23ec4..e077dfb4b 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -474,7 +474,7 @@ test_executables += executable('kms_color',
 test_list += 'kms_color'
 
 test_executables += executable('kms_dsc',
-	   [ join_paths('i915', 'kms_dsc.c'), join_paths ('i915', 'kms_dsc_helper.c')],
+	   join_paths('i915', 'kms_dsc.c'),
 	   dependencies : test_deps,
 	   install_dir : libexecdir,
 	   install_rpath : libexecdir_rpathdir,
@@ -482,7 +482,7 @@ test_executables += executable('kms_dsc',
 test_list += 'kms_dsc'
 
 test_executables += executable('kms_psr2_sf',
-	   [ join_paths('i915', 'kms_psr2_sf.c'), join_paths ('i915', 'kms_dsc_helper.c')],
+	   join_paths('i915', 'kms_psr2_sf.c'),
 	   dependencies : test_deps,
 	   install_dir : libexecdir,
 	   install_rpath : libexecdir_rpathdir,
-- 
2.40.0



More information about the igt-dev mailing list