[Intel-gfx] [PATCH i-g-t 1/8] kms_force_connector: use comparison macros to make debug output clearer

[Thomas Wood]

Signed-off-by: Thomas Wood <thomas.w...@intel.com>
---
 tests/kms_force_connector.c | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/tests/kms_force_connector.c b/tests/kms_force_connector.c
index 7485ca8..4ba1e0b 100644
--- a/tests/kms_force_connector.c
+++ b/tests/kms_force_connector.c
@@ -27,8 +27,9 @@
 IGT_TEST_DESCRIPTION("Check the debugfs force connector/edid features work"
                     " correctly.");
 
-#define CHECK_MODE(m, h, w, r) igt_assert(m.hdisplay == h && m.vdisplay == w \
-                                         && m.vrefresh == r)
+#define CHECK_MODE(m, h, w, r) \
+       igt_assert_eq(m.hdisplay, h); igt_assert_eq(m.vdisplay, w); \
+       igt_assert_eq(m.vrefresh, r);
 
 igt_main
 {
@@ -63,8 +64,8 @@ igt_main
                /* force the connector on and check the reported values */
                kmstest_force_connector(drm_fd, vga_connector, 
FORCE_CONNECTOR_ON);
                temp = drmModeGetConnector(drm_fd, vga_connector->connector_id);
-               igt_assert(temp->connection == DRM_MODE_CONNECTED);
-               igt_assert(temp->count_modes > 0);
+               igt_assert_eq(temp->connection, DRM_MODE_CONNECTED);
+               igt_assert_lt(0, temp->count_modes);
                drmModeFreeConnector(temp);
 
                /* attempt to use the display */
@@ -77,15 +78,15 @@ igt_main
                kmstest_force_connector(drm_fd, vga_connector,
                                        FORCE_CONNECTOR_OFF);
                temp = drmModeGetConnector(drm_fd, vga_connector->connector_id);
-               igt_assert(temp->connection == DRM_MODE_DISCONNECTED);
-               igt_assert(temp->count_modes == 0);
+               igt_assert_eq(temp->connection, DRM_MODE_DISCONNECTED);
+               igt_assert_lt(0, temp->count_modes);
                drmModeFreeConnector(temp);
 
                /* check that the previous state is restored */
                kmstest_force_connector(drm_fd, vga_connector,
                                        FORCE_CONNECTOR_UNSPECIFIED);
                temp = drmModeGetConnector(drm_fd, vga_connector->connector_id);
-               igt_assert(temp->connection == vga_connector->connection);
+               igt_assert_eq(temp->connection, vga_connector->connection);
                drmModeFreeConnector(temp);
        }
 
@@ -115,7 +116,7 @@ igt_main
                temp = drmModeGetConnector(drm_fd, vga_connector->connector_id);
                /* the connector should now have the same number of modes that
                 * it started with */
-               igt_assert(temp->count_modes == start_n_modes);
+               igt_assert_eq(temp->count_modes, start_n_modes);
                drmModeFreeConnector(temp);
 
                kmstest_force_connector(drm_fd, vga_connector,
-- 
1.9.1

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