Commit c4605133 authored by Daniel Vetter's avatar Daniel Vetter

lib/igt_fb: better format printing

Steal if from kms_plane.c and put it into igt_fb.h

Also tiny bikeshed to remove the space, so it fits more tidily into
the usual name1=value1, name2=value2 style printing.

v2: Rebase

v3: It better compile :-/

Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> (v1)
Signed-off-by: Daniel Vetter's avatarDaniel Vetter <daniel.vetter@intel.com>
parent b25e06d6
Pipeline #23837 passed with stages
in 10 minutes and 19 seconds
......@@ -1146,8 +1146,10 @@ igt_create_fb_with_bo_size(int fd, int width, int height,
fb->size = bo_size;
igt_debug("%s(width=%d, height=%d, format=0x%x, modifier=0x%"PRIx64", size=%"PRIu64")\n",
__func__, width, height, format, modifier, bo_size);
igt_debug("%s(width=%d, height=%d, format=" IGT_FORMAT_FMT
", modifier=0x%"PRIx64", size=%"PRIu64")\n",
__func__, width, height, IGT_FORMAT_ARGS(format), modifier,
bo_size);
create_bo_for_fb(fb);
igt_assert(fb->gem_handle > 0);
......
......@@ -44,6 +44,10 @@
*/
#define IGT_FORMAT_FLOAT fourcc_code('I', 'G', 'F', 'x')
#define IGT_FORMAT_FMT "%c%c%c%c(0x%08x)"
#define IGT_FORMAT_ARGS(f) ((f) >> 0) & 0xff, ((f) >> 8) & 0xff, \
((f) >> 16) & 0xff, ((f) >> 24) & 0xff, (f)
/**
* igt_fb_t:
* @fb_id: KMS ID of the framebuffer
......
......@@ -401,9 +401,6 @@ static void set_legacy_lut(data_t *data, enum pipe pipe,
free(lut);
}
#define IGT_FORMAT_FMT "%c%c%c%c (0x%08x)"
#define IGT_FORMAT_ARGS(f) ((f) >> 0) & 0xff, ((f) >> 8) & 0xff, \
((f) >> 16) & 0xff, ((f) >> 24) & 0xff, (f)
static void test_format_plane_color(data_t *data, enum pipe pipe,
igt_plane_t *plane,
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment