[igt-dev] [PATCH i-g-t] lib/igt_fb: better format printing

Daniel Vetter daniel.vetter at ffwll.ch
Tue Mar 12 13:51:25 UTC 2019


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

Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter at intel.com>
---
 lib/igt_fb.c      | 6 ++++--
 lib/igt_fb.h      | 4 ++++
 tests/kms_plane.c | 3 ---
 3 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/lib/igt_fb.c b/lib/igt_fb.c
index e696c863ea70..9f9f7aac0935 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -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_FM
+		  "", 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);
diff --git a/lib/igt_fb.h b/lib/igt_fb.h
index 7667579b3d98..ba22bd664510 100644
--- a/lib/igt_fb.h
+++ b/lib/igt_fb.h
@@ -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
diff --git a/tests/kms_plane.c b/tests/kms_plane.c
index d8098bef891a..69069f6b1356 100644
--- a/tests/kms_plane.c
+++ b/tests/kms_plane.c
@@ -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,
-- 
2.20.1



More information about the igt-dev mailing list