And add api doc while at it.
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
{
char buf[64];
- sprintf(buf, "pipe %c %s", pipe_name(pipe_crc->pipe),
+ sprintf(buf, "pipe %s %s", kmstest_pipe_name(pipe_crc->pipe),
pipe_crc_source_name(pipe_crc->source));
errno = 0;
write(pipe_crc->ctl_fd, buf, strlen(buf));
{
char buf[32];
- sprintf(buf, "pipe %c none", pipe_name(pipe));
+ sprintf(buf, "pipe %s none", kmstest_pipe_name(pipe));
write(fd, buf, strlen(buf));
}
pipe_crc->ctl_fd = igt_debugfs_open("i915_display_crc_ctl", O_WRONLY);
igt_assert(pipe_crc->ctl_fd != -1);
- sprintf(buf, "i915_pipe_%c_crc", pipe_name(pipe));
+ sprintf(buf, "i915_pipe_%s_crc", kmstest_pipe_name(pipe));
pipe_crc->crc_fd = igt_debugfs_open(buf, O_RDONLY);
igt_assert(pipe_crc->crc_fd != -1);
{
char buf[32];
- sprintf(buf, "pipe %c none", pipe_name(pipe_crc->pipe));
+ sprintf(buf, "pipe %s none", kmstest_pipe_name(pipe_crc->pipe));
write(pipe_crc->ctl_fd, buf, strlen(buf));
}
* Note that this library's header pulls in the [i-g-t framebuffer](intel-gpu-tools-i-g-t-framebuffer.html)
* library as a dependency.
*/
-const char *kmstest_pipe_str(int pipe)
+
+/**
+ * kmstest_pipe_name:
+ * @pipe: display pipe
+ *
+ * Returns: String represnting @pipe, e.g. "A".
+ */
+const char *kmstest_pipe_name(enum pipe pipe)
{
const char *str[] = { "A", "B", "C" };
c->connector_type_id);
}
- LOG(display, "%s: Selecting pipe %c\n", output->name,
- pipe_name(output->config.pipe));
+ LOG(display, "%s: Selecting pipe %s\n", output->name,
+ kmstest_pipe_name(output->config.pipe));
display->pipes_in_use |= 1 << output->config.pipe;
}
igt_assert_f(a->pending_crtc_idx_mask !=
b->pending_crtc_idx_mask,
- "%s and %s are both trying to use pipe %c\n",
+ "%s and %s are both trying to use pipe %s\n",
igt_output_name(a), igt_output_name(b),
- pipe_name(ffs(a->pending_crtc_idx_mask) - 1));
+ kmstest_pipe_name(ffs(a->pending_crtc_idx_mask) - 1));
}
}
if (plane->fb_changed && fb_id == 0) {
LOG(display,
- "%s: SetPlane pipe %c, plane %d, disabling\n",
+ "%s: SetPlane pipe %s, plane %d, disabling\n",
igt_output_name(output),
- pipe_name(output->config.pipe),
+ kmstest_pipe_name(output->config.pipe),
plane->index);
ret = drmModeSetPlane(display->drm_fd,
CHECK_RETURN(ret, fail_on_error);
} else if (plane->fb_changed || plane->position_changed) {
LOG(display,
- "%s: SetPlane %c.%d, fb %u, position (%d, %d)\n",
+ "%s: SetPlane %s.%d, fb %u, position (%d, %d)\n",
igt_output_name(output),
- pipe_name(output->config.pipe),
+ kmstest_pipe_name(output->config.pipe),
plane->index,
fb_id,
plane->crtc_x, plane->crtc_y);
if (gem_handle) {
LOG(display,
- "%s: SetCursor pipe %c, fb %u %dx%d\n",
+ "%s: SetCursor pipe %s, fb %u %dx%d\n",
igt_output_name(output),
- pipe_name(output->config.pipe),
+ kmstest_pipe_name(output->config.pipe),
gem_handle,
cursor->fb->width, cursor->fb->height);
cursor->fb->height);
} else {
LOG(display,
- "%s: SetCursor pipe %c, disabling\n",
+ "%s: SetCursor pipe %s, disabling\n",
igt_output_name(output),
- pipe_name(output->config.pipe));
+ kmstest_pipe_name(output->config.pipe));
ret = drmModeSetCursor(display->drm_fd, crtc_id,
0, 0, 0);
int y = cursor->crtc_y;
LOG(display,
- "%s: MoveCursor pipe %c, (%d, %d)\n",
+ "%s: MoveCursor pipe %s, (%d, %d)\n",
igt_output_name(output),
- pipe_name(output->config.pipe),
+ kmstest_pipe_name(output->config.pipe),
x, y);
ret = drmModeMoveCursor(display->drm_fd, crtc_id, x, y);
if (fb_id) {
LOG(display,
- "%s: SetCrtc pipe %c, fb %u, panning (%d, %d), "
+ "%s: SetCrtc pipe %s, fb %u, panning (%d, %d), "
"mode %dx%d\n",
igt_output_name(output),
- pipe_name(output->config.pipe),
+ kmstest_pipe_name(output->config.pipe),
fb_id,
primary->pan_x, primary->pan_y,
mode->hdisplay, mode->vdisplay);
mode);
} else {
LOG(display,
- "%s: SetCrtc pipe %c, disabling\n",
+ "%s: SetCrtc pipe %s, disabling\n",
igt_output_name(output),
- pipe_name(output->config.pipe));
+ kmstest_pipe_name(output->config.pipe));
ret = drmModeSetCrtc(display->drm_fd,
crtc_id,
LOG(display, "%s: set_pipe(any)\n", igt_output_name(output));
output->pending_crtc_idx_mask = -1UL;
} else {
- LOG(display, "%s: set_pipe(%c)\n", igt_output_name(output),
- pipe_name(pipe));
+ LOG(display, "%s: set_pipe(%s)\n", igt_output_name(output),
+ kmstest_pipe_name(pipe));
output->pending_crtc_idx_mask = 1 << pipe;
}
}
igt_pipe_t *pipe = plane->pipe;
igt_display_t *display = pipe->display;
- LOG(display, "%c.%d: plane_set_fb(%d)\n", pipe_name(pipe->pipe),
+ LOG(display, "%s.%d: plane_set_fb(%d)\n", kmstest_pipe_name(pipe->pipe),
plane->index, fb ? fb->fb_id : 0);
plane->fb = fb;
igt_pipe_t *pipe = plane->pipe;
igt_display_t *display = pipe->display;
- LOG(display, "%c.%d: plane_set_position(%d,%d)\n",
- pipe_name(pipe->pipe), plane->index, x, y);
+ LOG(display, "%s.%d: plane_set_position(%d,%d)\n",
+ kmstest_pipe_name(pipe->pipe), plane->index, x, y);
plane->crtc_x = x;
plane->crtc_y = y;
igt_pipe_t *pipe = plane->pipe;
igt_display_t *display = pipe->display;
- LOG(display, "%c.%d: plane_set_panning(%d,%d)\n", pipe_name(pipe->pipe),
+ LOG(display, "%s.%d: plane_set_panning(%d,%d)\n",
+ kmstest_pipe_name(pipe->pipe),
plane->index, x, y);
plane->pan_x = x;
igt_pipe_t *pipe = plane->pipe;
igt_display_t *display = pipe->display;
- LOG(display, "%c.%d: plane_set_rotation(%s)\n", pipe_name(pipe->pipe),
+ LOG(display, "%s.%d: plane_set_rotation(%s)\n",
+ kmstest_pipe_name(pipe->pipe),
plane->index, rotation_name(rotation));
plane->rotation = rotation;
PIPE_C,
I915_MAX_PIPES
};
-#define pipe_name(p) ((p) + 'A')
+const char *kmstest_pipe_name(enum pipe pipe);
/* We namespace this enum to not conflict with the Android i915_drm.h */
enum igt_plane {
void kmstest_dump_mode(drmModeModeInfo *mode);
int kmstest_get_pipe_from_crtc_id(int fd, int crtc_id);
-const char *kmstest_pipe_str(int pipe);
const char *kmstest_encoder_type_str(int type);
const char *kmstest_connector_status_str(int type);
const char *kmstest_connector_type_str(int type);
valid_tests++;
- igt_info("Beginning %s on pipe %c, connector %s\n",
- igt_subtest_name(), pipe_name(data->pipe),
+ igt_info("Beginning %s on pipe %s, connector %s\n",
+ igt_subtest_name(),
+ kmstest_pipe_name(data->pipe),
igt_output_name(output));
testfunc(data);
- igt_info("\n%s on pipe %c, connector %s: PASSED\n\n",
- igt_subtest_name(), pipe_name(data->pipe),
+ igt_info("\n%s on pipe %s, connector %s: PASSED\n\n",
+ igt_subtest_name(),
+ kmstest_pipe_name(data->pipe),
igt_output_name(output));
/* cleanup what prepare_crtc() has done */
if (!prepare_crtc(data))
continue;
- igt_info("Beginning %s on pipe %c, connector %s\n",
- igt_subtest_name(), pipe_name(data->pipe),
+ igt_info("Beginning %s on pipe %s, connector %s\n",
+ igt_subtest_name(),
+ kmstest_pipe_name(data->pipe),
igt_output_name(data->output));
if (!prepare_test(data, mode)) {
- igt_info("%s on pipe %c, connector %s: SKIPPED\n",
- igt_subtest_name(), pipe_name(data->pipe),
+ igt_info("%s on pipe %s, connector %s: SKIPPED\n",
+ igt_subtest_name(),
+ kmstest_pipe_name(data->pipe),
igt_output_name(data->output));
continue;
}
test_crc(data, mode);
- igt_info("%s on pipe %c, connector %s: PASSED\n",
- igt_subtest_name(), pipe_name(data->pipe),
+ igt_info("%s on pipe %s, connector %s: PASSED\n",
+ igt_subtest_name(),
+ kmstest_pipe_name(data->pipe),
igt_output_name(data->output));
finish_crtc(data, mode);
for_each_connected_output(display, output) {
- igt_info("%s: Testing connector %s using pipe %c\n",
+ igt_info("%s: Testing connector %s using pipe %s\n",
igt_subtest_name(), igt_output_name(output),
- pipe_name(pipe));
+ kmstest_pipe_name(pipe));
valid_connectors += test_read_crc_for_output(data, pipe, output, flags);
}
drmModeModeInfo *mode;
igt_crc_t crc;
- igt_info("Testing connector %s using pipe %c plane %d\n",
- igt_output_name(output), pipe_name(pipe), plane);
+ igt_info("Testing connector %s using pipe %s plane %d\n",
+ igt_output_name(output), kmstest_pipe_name(pipe), plane);
test_init(data, pipe);
drmModeModeInfo *mode;
igt_crc_t crc;
- igt_info("Testing connector %s using pipe %c plane %d\n", igt_output_name(output), pipe_name(pipe), plane);
+ igt_info("Testing connector %s using pipe %s plane %d\n",
+ igt_output_name(output), kmstest_pipe_name(pipe), plane);
test_init(data, pipe);
static void
run_tests_for_pipe_plane(data_t *data, enum pipe pipe, enum igt_plane plane)
{
- igt_subtest_f("plane-position-covered-pipe-%c-plane-%d",
- pipe_name(pipe), plane)
+ igt_subtest_f("plane-position-covered-pipe-%s-plane-%d",
+ kmstest_pipe_name(pipe), plane)
test_plane_position(data, pipe, plane,
TEST_POSITION_FULLY_COVERED);
- igt_subtest_f("plane-position-hole-pipe-%c-plane-%d",
- pipe_name(pipe), plane)
+ igt_subtest_f("plane-position-hole-pipe-%s-plane-%d",
+ kmstest_pipe_name(pipe), plane)
test_plane_position(data, pipe, plane, 0);
- igt_subtest_f("plane-panning-top-left-pipe-%c-plane-%d",
- pipe_name(pipe), plane)
+ igt_subtest_f("plane-panning-top-left-pipe-%s-plane-%d",
+ kmstest_pipe_name(pipe), plane)
test_plane_panning(data, pipe, plane, TEST_PANNING_TOP_LEFT);
- igt_subtest_f("plane-panning-bottom-right-pipe-%c-plane-%d",
- pipe_name(pipe), plane)
+ igt_subtest_f("plane-panning-bottom-right-pipe-%s-plane-%d",
+ kmstest_pipe_name(pipe), plane)
test_plane_panning(data, pipe, plane,
TEST_PANNING_BOTTOM_RIGHT);
int ret;
#if 0
- fprintf(stdout, "Using pipe %c, %dx%d\n", pipe_name(config->pipe),
+ fprintf(stdout, "Using pipe %s, %dx%d\n", kmstest_pipe_name(config->pipe),
mode->hdisplay, mode->vdisplay);
#endif
mode->name, mode->vrefresh, igt_format_str(format));
igt_assert(ret > 0);
ret = asprintf(&cconf_str, "pipe %s, encoder %s, connector %s",
- kmstest_pipe_str(cconf->pipe),
+ kmstest_pipe_name(cconf->pipe),
kmstest_encoder_type_str(cconf->encoder->encoder_type),
kmstest_connector_type_str(cconf->connector->connector_type));
igt_assert(ret > 0);
pos = snprintf(buf, buf_size,
"CRTC[%d] [Pipe %s] Mode: %s@%dHz Connectors: ",
- crtc->crtc_id, kmstest_pipe_str(crtc->pipe_id),
+ crtc->crtc_id, kmstest_pipe_name(crtc->pipe_id),
crtc->mode.name, crtc->mode.vrefresh);
if (pos > buf_size)
return;
igt_assert(data->display.has_universal_planes);
igt_skip_on(pipe >= display->n_pipes);
- igt_info("Testing connector %s using pipe %c\n", igt_output_name(output), pipe_name(pipe));
+ igt_info("Testing connector %s using pipe %s\n", igt_output_name(output),
+ kmstest_pipe_name(pipe));
functional_test_init(&test, output, pipe);
{
igt_output_t *output;
- igt_subtest_f("universal-plane-pipe-%c-functional", pipe_name(pipe))
+ igt_subtest_f("universal-plane-pipe-%s-functional",
+ kmstest_pipe_name(pipe))
for_each_connected_output(&data->display, output)
functional_test_pipe(data, pipe, output);
- igt_subtest_f("universal-plane-pipe-%c-sanity", pipe_name(pipe))
+ igt_subtest_f("universal-plane-pipe-%s-sanity",
+ kmstest_pipe_name(pipe))
for_each_connected_output(&data->display, output)
sanity_test_pipe(data, pipe, output);
- igt_subtest_f("disable-primary-vs-flip-pipe-%c", pipe_name(pipe))
+ igt_subtest_f("disable-primary-vs-flip-pipe-%s",
+ kmstest_pipe_name(pipe))
for_each_connected_output(&data->display, output)
pageflip_test_pipe(data, pipe, output);
}