@@ -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;
@@ -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);
@@ -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),
@@ -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");
@@ -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);
}
@@ -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);
}
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@ffwll.ch> --- 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(-)