From: Rob Clark <robdclark@gmail.com>
To: igt-dev@lists.freedesktop.org
Cc: Rob Clark <robdclark@chromium.org>,
Kamil Konieczny <kamil.konieczny@linux.intel.com>
Subject: [PATCH 1/2] tests/kms_plane_scaling: Pass display to get_num_scalers()
Date: Mon, 19 Aug 2024 13:25:45 -0700 [thread overview]
Message-ID: <20240819202546.111243-1-robdclark@gmail.com> (raw)
From: Rob Clark <robdclark@chromium.org>
Prep for next patch. The next commit will need the display to figure
out the number of scalers. This commit has no functional change, just
plumbing the extra parameter everywhere.
Signed-off-by: Rob Clark <robdclark@chromium.org>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
---
lib/igt_kms.c | 5 +++--
lib/igt_kms.h | 2 +-
tests/kms_plane_scaling.c | 24 ++++++++++++------------
3 files changed, 16 insertions(+), 15 deletions(-)
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index e030b35a6051..99cdeeb76033 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -6650,17 +6650,18 @@ int igt_get_dp_mst_connector_id(igt_output_t *output)
/**
* get_num_scalers:
- * @drm_fd: drm file descriptor
+ * @display: the display
* @pipe: display pipe
*
* Returns: Number of scalers supported per pipe.
*/
-int get_num_scalers(int drm_fd, enum pipe pipe)
+int get_num_scalers(igt_display_t *display, enum pipe pipe)
{
char buf[8120];
char *start_loc1, *start_loc2;
int dir, res;
int num_scalers = 0;
+ int drm_fd = display->drm_fd;
char dest[20] = ":pipe ";
strcat(dest, kmstest_pipe_name(pipe));
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index e8582a45b53e..25ba509164e3 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -1223,6 +1223,6 @@ bool igt_parse_mode_string(const char *mode_string, drmModeModeInfo *mode);
bool intel_pipe_output_combo_valid(igt_display_t *display);
bool igt_check_output_is_dp_mst(igt_output_t *output);
int igt_get_dp_mst_connector_id(igt_output_t *output);
-int get_num_scalers(int drm_fd, enum pipe pipe);
+int get_num_scalers(igt_display_t *display, enum pipe pipe);
#endif /* __IGT_KMS_H__ */
diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c
index 90bfb004773b..42eb67409899 100644
--- a/tests/kms_plane_scaling.c
+++ b/tests/kms_plane_scaling.c
@@ -1080,8 +1080,8 @@ static void test_scaler_with_multi_pipe_plane(data_t *d)
igt_output_set_pipe(output1, pipe1);
igt_output_set_pipe(output2, pipe2);
- igt_require(get_num_scalers(d->drm_fd, pipe1) >= 2);
- igt_require(get_num_scalers(d->drm_fd, pipe2) >= 2);
+ igt_require(get_num_scalers(display, pipe1) >= 2);
+ igt_require(get_num_scalers(display, pipe2) >= 2);
plane[0] = igt_output_get_plane(output1, 0);
igt_require(plane[0]);
@@ -1169,7 +1169,7 @@ static void invalid_parameter_tests(data_t *d)
igt_output_set_pipe(output, pipe);
plane = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
- igt_require(get_num_scalers(d->drm_fd, pipe) >= 1);
+ igt_require(get_num_scalers(&d->display, pipe) >= 1);
igt_create_fb(d->drm_fd, 256, 256,
DRM_FORMAT_XRGB8888,
@@ -1346,7 +1346,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_valid_output_on_pipe(&data.display, pipe, output) {
if (!pipe_output_combo_valid(&data.display, pipe, output))
continue;
- if (get_num_scalers(data.drm_fd, pipe) < 1)
+ if (get_num_scalers(&data.display, pipe) < 1)
continue;
igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) {
@@ -1370,7 +1370,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_valid_output_on_pipe(&data.display, pipe, output) {
if (!pipe_output_combo_valid(&data.display, pipe, output))
continue;
- if (get_num_scalers(data.drm_fd, pipe) < 1)
+ if (get_num_scalers(&data.display, pipe) < 1)
continue;
igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) {
@@ -1394,7 +1394,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_valid_output_on_pipe(&data.display, pipe, output) {
if (!pipe_output_combo_valid(&data.display, pipe, output))
continue;
- if (get_num_scalers(data.drm_fd, pipe) < 1)
+ if (get_num_scalers(&data.display, pipe) < 1)
continue;
igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) {
@@ -1417,7 +1417,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_valid_output_on_pipe(&data.display, pipe, output) {
if (!pipe_output_combo_valid(&data.display, pipe, output))
continue;
- if (get_num_scalers(data.drm_fd, pipe) < 1)
+ if (get_num_scalers(&data.display, pipe) < 1)
continue;
igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) {
@@ -1437,7 +1437,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_valid_output_on_pipe(&data.display, pipe, output) {
if (!pipe_output_combo_valid(&data.display, pipe, output))
continue;
- if (get_num_scalers(data.drm_fd, pipe) < 1)
+ if (get_num_scalers(&data.display, pipe) < 1)
continue;
igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) {
@@ -1457,7 +1457,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_valid_output_on_pipe(&data.display, pipe, output) {
if (!pipe_output_combo_valid(&data.display, pipe, output))
continue;
- if (get_num_scalers(data.drm_fd, pipe) < 1)
+ if (get_num_scalers(&data.display, pipe) < 1)
continue;
igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) {
@@ -1477,7 +1477,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_valid_output_on_pipe(&data.display, pipe, output) {
if (!pipe_output_combo_valid(&data.display, pipe, output))
continue;
- if (get_num_scalers(data.drm_fd, pipe) < 2)
+ if (get_num_scalers(&data.display, pipe) < 2)
continue;
igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) {
@@ -1502,7 +1502,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_valid_output_on_pipe(&data.display, pipe, output) {
drmModeModeInfo *mode = NULL;
- if (get_num_scalers(data.drm_fd, pipe) < 1)
+ if (get_num_scalers(&data.display, pipe) < 1)
continue;
/*
* Need to find mode with lowest vrefresh else
@@ -1527,7 +1527,7 @@ igt_main_args("", long_opts, help_str, opt_handler, &data)
for_each_pipe_with_valid_output(&data.display, pipe, output) {
if (!pipe_output_combo_valid(&data.display, pipe, output))
continue;
- if (get_num_scalers(data.drm_fd, pipe) < 1)
+ if (get_num_scalers(&data.display, pipe) < 1)
continue;
igt_dynamic_f("pipe-%s-%s-invalid-num-scalers",
--
2.46.0
next reply other threads:[~2024-08-19 20:25 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-19 20:25 Rob Clark [this message]
2024-08-19 20:25 ` [PATCH 2/2] tests/kms_plane_scaling: Improve msm support Rob Clark
2024-08-20 10:02 ` Kamil Konieczny
2024-08-19 22:28 ` ✓ CI.xeBAT: success for series starting with [1/2] tests/kms_plane_scaling: Pass display to get_num_scalers() Patchwork
2024-08-19 22:46 ` ✓ Fi.CI.BAT: " Patchwork
2024-08-20 5:58 ` ✗ CI.xeFULL: failure " Patchwork
2024-08-20 11:26 ` ✗ Fi.CI.IGT: " Patchwork
-- strict thread matches above, loose matches on Subject: below --
2024-08-14 18:53 [PATCH 1/2] " Rob Clark
2024-08-19 18:20 ` Kamil Konieczny
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=20240819202546.111243-1-robdclark@gmail.com \
--to=robdclark@gmail.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=kamil.konieczny@linux.intel.com \
--cc=robdclark@chromium.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