[Intel-gfx] [PATCH i-g-t 5/7] kms_flip: Remove magic constant in run_test_on_crtc_set()

[Rodrigo Siqueira]

The function run_test_on_crtc_set() expects a parameter named crtc_count
which slight changes the behavior of the test. If this crtc_count is
‘1’, the test will run in a single CRTC; otherwise, it will run in two
different CRTC. However, this function uses hardcoded literal 1 and 2
for each case. This patch creates two constant with the goal to improve
the readability.

Signed-off-by: Rodrigo Siqueira <rodrigosiqueiram...@gmail.com>
---
 tests/kms_flip.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index 9ef77de9..42ae3ebc 100755
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -76,6 +76,9 @@
 #define EVENT_FLIP             (1 << 0)
 #define EVENT_VBLANK           (1 << 1)
 
+#define RUN_TEST               1
+#define RUN_PAIR               2
+
 #ifndef DRM_CAP_TIMESTAMP_MONOTONIC
 #define DRM_CAP_TIMESTAMP_MONOTONIC 6
 #endif
@@ -1164,7 +1167,7 @@ static void run_test_on_crtc_set(struct test_output *o, 
int *crtc_idxs,
        int i;
 
        switch (crtc_count) {
-       case 1:
+       case RUN_TEST:
                connector_find_preferred_mode(o->_connector[0], crtc_idxs[0], 
o);
                if (!o->mode_valid)
                        return;
@@ -1175,7 +1178,7 @@ static void run_test_on_crtc_set(struct test_output *o, 
int *crtc_idxs,
                         
kmstest_connector_type_str(o->kconnector[0]->connector_type),
                         o->kconnector[0]->connector_type_id);
                break;
-       case 2:
+       case RUN_PAIR:
                connector_find_compatible_mode(crtc_idxs[0], crtc_idxs[1], o);
                if (!o->mode_valid)
                        return;
@@ -1341,7 +1344,7 @@ static int run_test(int duration, int flags)
                        o.depth = 24;
 
                        crtc_idx = n;
-                       run_test_on_crtc_set(&o, &crtc_idx, 1, duration);
+                       run_test_on_crtc_set(&o, &crtc_idx, RUN_TEST, duration);
                }
        }
 
@@ -1410,7 +1413,8 @@ static int run_pair(int duration, int flags)
                                        crtc_idxs[0] = n;
                                        crtc_idxs[1] = m;
 
-                                       run_test_on_crtc_set(&o, crtc_idxs, 2,
+                                       run_test_on_crtc_set(&o, crtc_idxs,
+                                                            RUN_PAIR,
                                                             duration);
                                }
                        }
-- 
2.21.0

signature.asc
Description: PGP signature

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx