CHROMIUM: MALI: Fix printf code for 'seqno'
As of CL:1704173, seqno is 64-bit. We need to adjust our printf
strings to account for this. Otherwise, we get warnings like:
.../mali_kbase_jd_debugfs.c:
In function 'kbase_jd_debugfs_fence_info':
.../mali_kbase_jd_debugfs.c:86:7:
error: format '%u' expects argument of type 'unsigned int',
but argument 4 has type 'u64' [-Werror=format=]
Technically seqno moved to 64-bit in kernel 5.1, but we've backported
it to our 4.19 kernel so we can't exactly add a special-version check.
BUG=chromium:941638
TEST=Build and boot
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1709132
Reviewed-by: Guenter Roeck <groeck@chromium.org>
(cherry picked from commit c561f7c4b869e10e0ce02aa03c52eb2707adf702)
Signed-off-by: Nicolas Boichat <drinkcat@chromium.org>
Change-Id: I89e2cbe2e7d6bec892206a5c647729e48d2645d4
diff --git a/drivers/gpu/arm/midgard/mali_kbase_fence.c b/drivers/gpu/arm/midgard/mali_kbase_fence.c
index 96a6ab9..b2f7628 100644
--- a/drivers/gpu/arm/midgard/mali_kbase_fence.c
+++ b/drivers/gpu/arm/midgard/mali_kbase_fence.c
@@ -67,11 +67,7 @@
kbase_fence_fence_value_str(struct dma_fence *fence, char *str, int size)
#endif
{
-#if (KERNEL_VERSION(5, 1, 0) > LINUX_VERSION_CODE)
- snprintf(str, size, "%u", fence->seqno);
-#else
snprintf(str, size, "%llu", fence->seqno);
-#endif
}
#if (LINUX_VERSION_CODE < KERNEL_VERSION(4, 10, 0))
diff --git a/drivers/gpu/arm/midgard/mali_kbase_jd_debugfs.c b/drivers/gpu/arm/midgard/mali_kbase_jd_debugfs.c
index 2fc21fd..c2488a2 100644
--- a/drivers/gpu/arm/midgard/mali_kbase_jd_debugfs.c
+++ b/drivers/gpu/arm/midgard/mali_kbase_jd_debugfs.c
@@ -78,7 +78,7 @@
#if (LINUX_VERSION_CODE < KERNEL_VERSION(4, 8, 0))
"Sd(%u#%u: %s) ",
#else
- "Sd(%llu#%u: %s) ",
+ "Sd(%llu#%llu: %s) ",
#endif
fence->context,
fence->seqno,
@@ -98,7 +98,7 @@
#if (LINUX_VERSION_CODE < KERNEL_VERSION(4, 8, 0))
"Wd(%u#%u: %s) ",
#else
- "Wd(%llu#%u: %s) ",
+ "Wd(%llu#%llu: %s) ",
#endif
fence->context,
fence->seqno,
diff --git a/drivers/gpu/arm/midgard/mali_kbase_sync_file.c b/drivers/gpu/arm/midgard/mali_kbase_sync_file.c
index 0679c48..e0044cc 100644
--- a/drivers/gpu/arm/midgard/mali_kbase_sync_file.c
+++ b/drivers/gpu/arm/midgard/mali_kbase_sync_file.c
@@ -308,9 +308,6 @@
#if (KERNEL_VERSION(4, 8, 0) > LINUX_VERSION_CODE)
scnprintf(info->name, sizeof(info->name), "%u#%u",
fence->context, fence->seqno);
-#elif (KERNEL_VERSION(5, 1, 0) > LINUX_VERSION_CODE)
- scnprintf(info->name, sizeof(info->name), "%llu#%u",
- fence->context, fence->seqno);
#else
scnprintf(info->name, sizeof(info->name), "%llu#%llu",
fence->context, fence->seqno);