public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] lib/igt_fb: better format printing
@ 2019-03-11 16:47 Daniel Vetter
  2019-03-11 17:03 ` Ville Syrjälä
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: Daniel Vetter @ 2019-03-11 16:47 UTC (permalink / raw)
  To: IGT development; +Cc: Daniel Vetter

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.

Signed-off-by: Daniel Vetter <daniel.vetter@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 f49becbd52f0..bafb0d085af7 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -1115,8 +1115,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, tiling=0x%"PRIx64", size=%"PRIu64")\n",
-		  __func__, width, height, format, tiling, bo_size);
+	igt_debug("%s(width=%d, height=%d, format=" IGT_FORMAT_FMT
+		  ", tiling=0x%"PRIx64", size=%"PRIu64")\n",
+		  __func__, width, height, IGT_FORMAT_ARGS(format), tiling,
+		  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 e1d885e2c5bc..d53b68e56318 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 91de46948a3a..68db63ca893a 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

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2019-03-13 16:27 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-03-11 16:47 [igt-dev] [PATCH i-g-t] lib/igt_fb: better format printing Daniel Vetter
2019-03-11 17:03 ` Ville Syrjälä
2019-03-12  9:50 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
2019-03-12 13:51 ` [igt-dev] [PATCH i-g-t] " Daniel Vetter
2019-03-12 13:53 ` Daniel Vetter
2019-03-13 16:27   ` Daniel Vetter
2019-03-13 11:22 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_fb: better format printing (rev3) Patchwork
2019-03-13 14:32 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox