Re: [igt-dev] [PATCH i-g-t 2/8] tests/kms_plane: Use IGT_MODIFIER_{FMT, ARGS}
From: Karthik B S <hidden>
Date: 2021-10-18 06:53:45
On 10/14/2021 3:47 AM, Ville Syrjala wrote:
From: Ville Syrjälä <redacted>
Use IGT_MODIFIER_{FMT,ARGS} to print a human readable name for
the modifier.
Signed-off-by: Ville Syrjälä <redacted>Reviewed-by: Karthik B S <redacted>
quoted hunk ↗ jump to hunk
--- tests/kms_plane.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-)diff --git a/tests/kms_plane.c b/tests/kms_plane.c index effc54cecbf7..405d4c2180a6 100644 --- a/tests/kms_plane.c +++ b/tests/kms_plane.c@@ -768,8 +768,8 @@ static bool test_format_plane_rgb(data_t *data, enum pipe pipe, igt_crc_t ref_crc[], struct igt_fb *fb) { - igt_info("Testing format " IGT_FORMAT_FMT " / modifier 0x%" PRIx64 " on %s.%u\n", - IGT_FORMAT_ARGS(format), modifier, + igt_info("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT " on %s.%u\n", + IGT_FORMAT_ARGS(format), IGT_MODIFIER_ARGS(modifier), kmstest_pipe_name(pipe), plane->index); return test_format_plane_colors(data, pipe, plane,@@ -806,8 +806,8 @@ static bool test_format_plane_yuv(data_t *data, enum pipe pipe, igt_color_range_to_str(r))) continue; - igt_info("Testing format " IGT_FORMAT_FMT " / modifier 0x%" PRIx64 " (%s, %s) on %s.%u\n", - IGT_FORMAT_ARGS(format), modifier, + igt_info("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT " (%s, %s) on %s.%u\n", + IGT_FORMAT_ARGS(format), IGT_MODIFIER_ARGS(modifier), igt_color_encoding_to_str(e), igt_color_range_to_str(r), kmstest_pipe_name(pipe), plane->index);@@ -886,8 +886,8 @@ static bool test_format_plane(data_t *data, enum pipe pipe, igt_pipe_crc_start(data->pipe_crc); - igt_info("Testing format " IGT_FORMAT_FMT " / modifier 0x%" PRIx64 " on %s.%u\n", - IGT_FORMAT_ARGS(ref.format), ref.modifier, + igt_info("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT " on %s.%u\n", + IGT_FORMAT_ARGS(ref.format), IGT_MODIFIER_ARGS(ref.modifier), kmstest_pipe_name(pipe), plane->index); if (data->display.is_atomic) {@@ -948,8 +948,8 @@ static bool test_format_plane(data_t *data, enum pipe pipe, }; if (igt_vec_index(&tested_formats, &rf) >= 0) { - igt_info("Skipping format " IGT_FORMAT_FMT " / modifier 0x%" PRIx64 " on %s.%u\n", - IGT_FORMAT_ARGS(f.format), f.modifier, + igt_info("Skipping format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT " on %s.%u\n", + IGT_FORMAT_ARGS(f.format), IGT_MODIFIER_ARGS(f.modifier), kmstest_pipe_name(pipe), plane->index); continue; }