Functions which provide feature checks through igt_skip should be of the form <prefix>_require_<feature>. Otoh feature checks which return in a boolean whether the feature is available should be of the form <prefix>_has_<feature>, e.g. gem_has_blt. Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> --- lib/igt_debugfs.c | 4 ++-- lib/igt_debugfs.h | 2 +- tests/kms_cursor_crc.c | 2 +- tests/kms_fbc_crc.c | 2 +- tests/kms_pipe_crc_basic.c | 2 +- tests/kms_plane.c | 2 +- 6 files changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c index fe3ec3a55a7d..1879dcd5cbfa 100644 --- a/lib/igt_debugfs.c +++ b/lib/igt_debugfs.c @@ -320,13 +320,13 @@ static void pipe_crc_exit_handler(int sig) } /** - * igt_pipe_crc_check: + * igt_require_pipe_crc: * * Convenience helper to check whether pipe CRC capturing is supported by the * kernel. Uses igt_skip to automatically skip the test/subtest if this isn't * the case. */ -void igt_pipe_crc_check(void) +void igt_require_pipe_crc(void) { const char *cmd = "pipe A none"; FILE *ctl; diff --git a/lib/igt_debugfs.h b/lib/igt_debugfs.h index 1618d8e4e833..d3d23896b50c 100644 --- a/lib/igt_debugfs.h +++ b/lib/igt_debugfs.h @@ -89,7 +89,7 @@ bool igt_crc_is_null(igt_crc_t *crc); bool igt_crc_equal(igt_crc_t *a, igt_crc_t *b); char *igt_crc_to_string(igt_crc_t *crc); -void igt_pipe_crc_check(void); +void igt_require_pipe_crc(void); igt_pipe_crc_t * igt_pipe_crc_new(enum pipe pipe, enum intel_pipe_crc_source source); void igt_pipe_crc_free(igt_pipe_crc_t *pipe_crc); diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index 61eb1c237d68..704eac2ee359 100644 --- a/tests/kms_cursor_crc.c +++ b/tests/kms_cursor_crc.c @@ -316,7 +316,7 @@ igt_main igt_set_vt_graphics_mode(); - igt_pipe_crc_check(); + igt_require_pipe_crc(); igt_display_init(&data.display, data.drm_fd); data.pipe_crc = calloc(igt_display_get_n_pipes(&data.display), diff --git a/tests/kms_fbc_crc.c b/tests/kms_fbc_crc.c index 405d9bd69780..2f1b90d28e18 100644 --- a/tests/kms_fbc_crc.c +++ b/tests/kms_fbc_crc.c @@ -491,7 +491,7 @@ igt_main data.devid = intel_get_drm_devid(data.drm_fd); - igt_pipe_crc_check(); + igt_require_pipe_crc(); status = igt_debugfs_fopen("i915_fbc_status", "r"); igt_require_f(status, "No i915_fbc_status found\n"); diff --git a/tests/kms_pipe_crc_basic.c b/tests/kms_pipe_crc_basic.c index 6b8ff116474e..75217df4f5e1 100644 --- a/tests/kms_pipe_crc_basic.c +++ b/tests/kms_pipe_crc_basic.c @@ -137,7 +137,7 @@ igt_main igt_set_vt_graphics_mode(); - igt_pipe_crc_check(); + igt_require_pipe_crc(); igt_display_init(&data.display, data.drm_fd); } diff --git a/tests/kms_plane.c b/tests/kms_plane.c index 09def232669e..223f0a772021 100644 --- a/tests/kms_plane.c +++ b/tests/kms_plane.c @@ -229,7 +229,7 @@ igt_main igt_set_vt_graphics_mode(); - igt_pipe_crc_check(); + igt_require_pipe_crc(); igt_display_init(&data.display, data.drm_fd); } -- 1.8.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx