drm/radeon: fix signed v unsigned print formats

[PATCH] drm/radeon: fix signed v unsigned print formats

Posted by Ben Dooks 1 month ago

Fix several places where %ld or %d has been used in place of
%lu or %u.

Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
 drivers/gpu/drm/radeon/radeon_gem.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_gem.c b/drivers/gpu/drm/radeon/radeon_gem.c
index f86773f3db20..891ef929cfa8 100644
--- a/drivers/gpu/drm/radeon/radeon_gem.c
+++ b/drivers/gpu/drm/radeon/radeon_gem.c
@@ -110,7 +110,7 @@ int radeon_gem_object_create(struct radeon_device *rdev, unsigned long size,
 	 */
 	max_size = rdev->mc.gtt_size - rdev->gart_pin_size;
 	if (size > max_size) {
-		DRM_DEBUG("Allocation size %ldMb bigger than %ldMb limit\n",
+		DRM_DEBUG("Allocation size %luMb bigger than %luMb limit\n",
 			  size >> 20, max_size >> 20);
 		return -ENOMEM;
 	}
@@ -560,7 +560,7 @@ int radeon_gem_set_tiling_ioctl(struct drm_device *dev, void *data,
 	struct radeon_bo *robj;
 	int r = 0;
 
-	DRM_DEBUG("%d \n", args->handle);
+	DRM_DEBUG("%u \n", args->handle);
 	gobj = drm_gem_object_lookup(filp, args->handle);
 	if (gobj == NULL)
 		return -ENOENT;
@@ -886,7 +886,7 @@ static int radeon_debugfs_gem_info_show(struct seq_file *m, void *unused)
 			placement = " CPU";
 			break;
 		}
-		seq_printf(m, "bo[0x%08x] %8ldkB %8ldMB %s pid %8ld\n",
+		seq_printf(m, "bo[0x%08x] %8lukB %8luMB %s pid %8lu\n",
 			   i, radeon_bo_size(rbo) >> 10, radeon_bo_size(rbo) >> 20,
 			   placement, (unsigned long)rbo->pid);
 		i++;
-- 
2.37.2.352.g3c44437643

Re: [PATCH] drm/radeon: fix signed v unsigned print formats

Posted by Alex Deucher 1 month ago

Applied.  thanks!

Alex

On Fri, Jan 2, 2026 at 7:36 PM Ben Dooks <ben.dooks@codethink.co.uk> wrote:
>
> Fix several places where %ld or %d has been used in place of
> %lu or %u.
>
> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
> ---
>  drivers/gpu/drm/radeon/radeon_gem.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_gem.c b/drivers/gpu/drm/radeon/radeon_gem.c
> index f86773f3db20..891ef929cfa8 100644
> --- a/drivers/gpu/drm/radeon/radeon_gem.c
> +++ b/drivers/gpu/drm/radeon/radeon_gem.c
> @@ -110,7 +110,7 @@ int radeon_gem_object_create(struct radeon_device *rdev, unsigned long size,
>          */
>         max_size = rdev->mc.gtt_size - rdev->gart_pin_size;
>         if (size > max_size) {
> -               DRM_DEBUG("Allocation size %ldMb bigger than %ldMb limit\n",
> +               DRM_DEBUG("Allocation size %luMb bigger than %luMb limit\n",
>                           size >> 20, max_size >> 20);
>                 return -ENOMEM;
>         }
> @@ -560,7 +560,7 @@ int radeon_gem_set_tiling_ioctl(struct drm_device *dev, void *data,
>         struct radeon_bo *robj;
>         int r = 0;
>
> -       DRM_DEBUG("%d \n", args->handle);
> +       DRM_DEBUG("%u \n", args->handle);
>         gobj = drm_gem_object_lookup(filp, args->handle);
>         if (gobj == NULL)
>                 return -ENOENT;
> @@ -886,7 +886,7 @@ static int radeon_debugfs_gem_info_show(struct seq_file *m, void *unused)
>                         placement = " CPU";
>                         break;
>                 }
> -               seq_printf(m, "bo[0x%08x] %8ldkB %8ldMB %s pid %8ld\n",
> +               seq_printf(m, "bo[0x%08x] %8lukB %8luMB %s pid %8lu\n",
>                            i, radeon_bo_size(rbo) >> 10, radeon_bo_size(rbo) >> 20,
>                            placement, (unsigned long)rbo->pid);
>                 i++;
> --
> 2.37.2.352.g3c44437643
>