drm/amd/display: Fix wrong format specifier in amdgpu_dm.c
authorHayden Goodfellow <Hayden.Goodfellow@amd.com>
Mon, 13 Sep 2021 01:32:09 +0000 (21:32 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 29 Jul 2022 15:25:33 +0000 (17:25 +0200)
commit 655c167edc8c260b6df08bdcfaca8afde0efbeb6 upstream.

[Why]
Currently, the 32bit kernel build fails due to an incorrect string
format specifier. ARRAY_SIZE() returns size_t type as it uses sizeof().
However, we specify it in a string as %ld. This causes a compiler error
and causes the 32bit build to fail.

[How]
Change the %ld to %zu as size_t (which sizeof() returns) is an unsigned
integer data type. We use 'z' to ensure it also works with 64bit build.

Reviewed-by: Harry Wentland <Harry.Wentland@amd.com>
Acked-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: Hayden Goodfellow <Hayden.Goodfellow@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

index adf0517..e3dfea3 100644 (file)
@@ -758,7 +758,7 @@ static void dm_dmub_outbox1_low_irq(void *interrupt_params)
                        do {
                                dc_stat_get_dmub_notification(adev->dm.dc, &notify);
                                if (notify.type >= ARRAY_SIZE(dm->dmub_thread_offload)) {
-                                       DRM_ERROR("DM: notify type %d larger than the array size %ld !", notify.type,
+                                       DRM_ERROR("DM: notify type %d larger than the array size %zu!", notify.type,
                                        ARRAY_SIZE(dm->dmub_thread_offload));
                                        continue;
                                }