Mesa (master): panfrost: Fix printf format specifier.

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Sat May 30 02:49:02 UTC 2020


Module: Mesa
Branch: master
Commit: 13735c4f476f997966baa8a1f4c071867d78b401
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=13735c4f476f997966baa8a1f4c071867d78b401

Author: Vinson Lee <vlee at freedesktop.org>
Date:   Thu May 28 16:36:54 2020 -0700

panfrost: Fix printf format specifier.

bifrost_sampler_descriptor.zero1 is of type uint8_t.

Fix warning reported by Coverity.

Invalid type in argument to printf format specifier (PRINTF_ARGS)
invalid_type: Argument s->zero1 to format specifier %lx was expected to
have type unsigned long but has type unsigned char.

Fixes: 6148d1be4bb5 ("panfrost: Fix size of bifrost sampler descriptor")
Signed-off-by: Vinson Lee <vlee at freedesktop.org>
Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5248>

---

 src/panfrost/pandecode/decode.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/panfrost/pandecode/decode.c b/src/panfrost/pandecode/decode.c
index e2bd753142f..8c42a575ba5 100644
--- a/src/panfrost/pandecode/decode.c
+++ b/src/panfrost/pandecode/decode.c
@@ -2456,7 +2456,7 @@ pandecode_samplers(mali_ptr samplers, unsigned sampler_count, int job_no, bool i
 
                         if (s->zero1 || s->zero2 || s->zero3 || s->zero4) {
                                 pandecode_msg("XXX: sampler zero tripped\n");
-                                pandecode_prop("zero = 0x%" PRIx64 ", 0x%" PRIx64 ", 0x%" PRIx64 ", 0x%" PRIx64 "\n", s->zero1, s->zero2, s->zero3, s->zero4);
+                                pandecode_prop("zero = 0x%" PRIx8 ", 0x%" PRIx64 ", 0x%" PRIx64 ", 0x%" PRIx64 "\n", s->zero1, s->zero2, s->zero3, s->zero4);
                         }
 
                         pandecode_indent--;



More information about the mesa-commit mailing list