Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
To: igt-dev@lists.freedesktop.org, mchehab@kernel.org
Subject: [igt-dev] [i-g-t V2 03/81] tests/color: Move color helpers to lib
Date: Wed, 17 May 2023 12:02:05 +0530	[thread overview]
Message-ID: <20230517063323.4146387-4-bhanuprakash.modem@intel.com> (raw)
In-Reply-To: <20230517063323.4146387-1-bhanuprakash.modem@intel.com>

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@intel.com>
---
 tests/kms_color_helper.c => lib/igt_color_helper.c | 2 +-
 tests/kms_color_helper.h => lib/igt_color_helper.h | 0
 lib/meson.build                                    | 1 +
 tests/chamelium/kms_chamelium_color.c              | 2 +-
 tests/kms_color.c                                  | 2 +-
 tests/meson.build                                  | 5 ++---
 6 files changed, 6 insertions(+), 6 deletions(-)
 rename tests/kms_color_helper.c => lib/igt_color_helper.c (99%)
 rename tests/kms_color_helper.h => lib/igt_color_helper.h (100%)

diff --git a/tests/kms_color_helper.c b/lib/igt_color_helper.c
similarity index 99%
rename from tests/kms_color_helper.c
rename to lib/igt_color_helper.c
index b01c38ef6..87fd096c1 100644
--- a/tests/kms_color_helper.c
+++ b/lib/igt_color_helper.c
@@ -22,7 +22,7 @@
  *
  */
 
-#include "kms_color_helper.h"
+#include "igt_color_helper.h"
 
 bool pipe_output_combo_valid(data_t *data, enum pipe pipe)
 {
diff --git a/tests/kms_color_helper.h b/lib/igt_color_helper.h
similarity index 100%
rename from tests/kms_color_helper.h
rename to lib/igt_color_helper.h
diff --git a/lib/meson.build b/lib/meson.build
index 75e52580b..1cc859f17 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -20,6 +20,7 @@ lib_sources = [
 	'igt_chamelium_helper.c',
 	'igt_collection.c',
 	'igt_color_encoding.c',
+	'igt_color_helper.c',
 	'igt_crc.c',
 	'igt_debugfs.c',
 	'igt_device.c',
diff --git a/tests/chamelium/kms_chamelium_color.c b/tests/chamelium/kms_chamelium_color.c
index 26239a6b8..bbb929b64 100644
--- a/tests/chamelium/kms_chamelium_color.c
+++ b/tests/chamelium/kms_chamelium_color.c
@@ -22,7 +22,7 @@
  *
  */
 
-#include "kms_color_helper.h"
+#include "igt_color_helper.h"
 
 IGT_TEST_DESCRIPTION("Test Color Features at Pipe level using Chamelium to verify instead of CRC");
 
diff --git a/tests/kms_color.c b/tests/kms_color.c
index fdc7120c7..cd2d38329 100644
--- a/tests/kms_color.c
+++ b/tests/kms_color.c
@@ -22,7 +22,7 @@
  *
  */
 
-#include "kms_color_helper.h"
+#include "igt_color_helper.h"
 
 IGT_TEST_DESCRIPTION("Test Color Features at Pipe level");
 
diff --git a/tests/meson.build b/tests/meson.build
index 7983bd019..f503f53f7 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -468,8 +468,7 @@ executable('testdisplay', ['testdisplay.c', 'testdisplay_hotplug.c'],
 	   install : true)
 test_list += 'testdisplay'
 
-test_executables += executable('kms_color',
-	   [ 'kms_color.c', 'kms_color_helper.c' ],
+test_executables += executable('kms_color', 'kms_color.c',
 	   dependencies : test_deps,
 	   install_dir : libexecdir,
 	   install_rpath : libexecdir_rpathdir,
@@ -494,7 +493,7 @@ test_list += 'kms_psr2_sf'
 
 if chamelium.found()
        test_executables += executable('kms_chamelium_color',
-                             [ 'chamelium/kms_chamelium_color.c', 'kms_color_helper.c' ],
+                             join_paths('chamelium', 'kms_chamelium_color.c'),
                              dependencies : test_deps + [ chamelium ],
                              install_dir : libexecdir,
                              install_rpath : libexecdir_rpathdir,
-- 
2.40.0



  parent reply	other threads:[~2023-05-17  6:38 UTC|newest]

Thread overview: 94+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-17  6:32 [igt-dev] [i-g-t V2 00/81] Document KMS tests Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 01/81] tests/chamelium: Move chamelium helpers to lib Bhanuprakash Modem
2023-05-17  9:03   ` Kamil Konieczny
2023-05-17 14:03     ` Modem, Bhanuprakash
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 02/81] tests/dsc: Move dsc " Bhanuprakash Modem
2023-05-17  6:32 ` Bhanuprakash Modem [this message]
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 04/81] testplan/kms: Include testdisplay to kms testplan config Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 05/81] testplan/kms: Add support to differentiate b/w drivers Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 06/81] tests/chamelium/kms_chamelium_audio: Document each subtest for testplan Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 07/81] tests/chamelium/kms_chamelium_color: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 08/81] tests/chamelium/kms_chamelium_edid: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 09/81] tests/chamelium/kms_chamelium_frames: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 10/81] tests/chamelium/kms_chamelium_hpd: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 11/81] tests/i915/kms_big_fb: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 12/81] tests/i915/kms_big_joiner: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 13/81] tests/i915/kms_busy: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 14/81] tests/i915/kms_ccs: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 15/81] tests/i915/kms_cdclk: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 16/81] tests/i915/kms_draw_crc: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 17/81] tests/i915/kms_dsc: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 18/81] tests/i915/kms_fbcon_fbt: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 19/81] tests/i915/kms_fence_pin_leak: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 20/81] tests/i915/kms_flip_scaled_crc: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 21/81] tests/i915/kms_flip_tiling: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 22/81] tests/i915/kms_frontbuffer_tracking: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 23/81] tests/i915/kms_legacy_colorkey: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 24/81] tests/i915/kms_mmap_write_crc: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 25/81] tests/i915/kms_pipe_b_c_ivb: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 26/81] tests/i915/kms_psr: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 27/81] tests/i915/kms_psr2_sf: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 28/81] tests/i915/kms_psr2_su: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 29/81] tests/i915/kms_psr_stress_test: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 30/81] tests/i915/kms_pwrite_crc: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 31/81] tests/kms_3d: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 32/81] tests/kms_addfb_basic: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 33/81] tests/kms_async_flips: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 34/81] tests/kms_atomic: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 35/81] tests/kms_atomic_interruptible: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 36/81] tests/kms_atomic_transition: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 37/81] tests/kms_bw: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 38/81] tests/kms_color: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 39/81] tests/kms_concurrent: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 40/81] tests/kms_content_protection: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 41/81] tests/kms_cursor_crc: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 42/81] tests/kms_cursor_edge_walk: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 43/81] tests/kms_cursor_legacy: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 44/81] tests/kms_display_modes: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 45/81] tests/kms_dither: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 46/81] tests/kms_dp_aux_dev: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 47/81] tests/kms_dp_tiled_display: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 48/81] tests/kms_flip: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 49/81] tests/kms_flip_event_leak: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 50/81] tests/kms_force_connector_basic: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 51/81] tests/kms_getfb: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 52/81] tests/kms_hdmi_inject: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 53/81] tests/kms_hdr: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 54/81] tests/kms_invalid_mode: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 55/81] tests/kms_lease: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 56/81] tests/kms_multipipe_modeset: " Bhanuprakash Modem
2023-05-17  6:32 ` [igt-dev] [i-g-t V2 57/81] tests/kms_panel_fitting: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 58/81] tests/kms_pipe_crc_basic: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 59/81] tests/kms_plane: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 60/81] tests/kms_plane_alpha_blend: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 61/81] tests/kms_plane_cursor: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 62/81] tests/kms_plane_lowres: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 63/81] tests/kms_plane_multiple: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 64/81] tests/kms_plane_scaling: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 65/81] tests/kms_prime: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 66/81] tests/kms_prop_blob: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 67/81] tests/kms_properties: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 68/81] tests/kms_rmfb: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 69/81] tests/kms_rotation_crc: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 70/81] tests/kms_scaling_modes: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 71/81] tests/kms_selftest: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 72/81] tests/kms_sequence: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 73/81] tests/kms_setmode: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 74/81] tests/kms_sysfs_edid_timing: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 75/81] tests/kms_tv_load_detect: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 76/81] tests/kms_universal_plane: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 77/81] tests/kms_vblank: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 78/81] tests/kms_vrr: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 79/81] tests/kms_writeback: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 80/81] tests/testdisplay: " Bhanuprakash Modem
2023-05-17  6:33 ` [igt-dev] [i-g-t V2 81/81] testplan/kms: Make documentation is mandatory for all kms subtests Bhanuprakash Modem
2023-05-17 15:51   ` [igt-dev] [i-g-t V4 " Bhanuprakash Modem
2023-08-14  9:57     ` Mauro Carvalho Chehab
2023-05-17  7:20 ` [igt-dev] ✗ GitLab.Pipeline: warning for Document KMS tests (rev8) Patchwork
2023-05-17  7:46 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2023-05-17 10:36 ` [igt-dev] [i-g-t V2 00/81] Document KMS tests Kamil Konieczny
2023-05-17 14:00   ` Modem, Bhanuprakash
2023-05-17 16:47 ` [igt-dev] ✓ Fi.CI.IGT: success for Document KMS tests (rev8) Patchwork
2023-05-17 16:57 ` [igt-dev] ✗ GitLab.Pipeline: warning for Document KMS tests (rev9) Patchwork
2023-05-17 17:17 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2023-05-18  5:51 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230517063323.4146387-4-bhanuprakash.modem@intel.com \
    --to=bhanuprakash.modem@intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    --cc=mchehab@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox