2022-07-18 14:14:12

by Jason Wang

[permalink] [raw]
Subject: [PATCH] dma-buf: Fix comment typo

The double `have' is duplicated in line 696, remove one.

Signed-off-by: Jason Wang <[email protected]>
---
drivers/gpu/drm/radeon/radeon_gem.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/radeon/radeon_gem.c b/drivers/gpu/drm/radeon/radeon_gem.c
index 84843b3b3aef..261fcbae88d7 100644
--- a/drivers/gpu/drm/radeon/radeon_gem.c
+++ b/drivers/gpu/drm/radeon/radeon_gem.c
@@ -693,7 +693,7 @@ int radeon_gem_va_ioctl(struct drm_device *dev, void *data,
}

/* !! DONT REMOVE !!
- * We don't support vm_id yet, to be sure we don't have have broken
+ * We don't support vm_id yet, to be sure we don't have broken
* userspace, reject anyone trying to use non 0 value thus moving
* forward we can use those fields without breaking existant userspace
*/
--
2.35.1


2022-07-18 14:52:54

by Christian König

[permalink] [raw]
Subject: Re: [PATCH] dma-buf: Fix comment typo

Am 15.07.22 um 07:20 schrieb Jason Wang:
> The double `have' is duplicated in line 696, remove one.

The subject line is rather confusing since this isn't related to DMA-buf
at all.

Please change that to "drm/radeon:", apart from that the patch looks
good to me.

Christian.

>
> Signed-off-by: Jason Wang <[email protected]>
> ---
> drivers/gpu/drm/radeon/radeon_gem.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_gem.c b/drivers/gpu/drm/radeon/radeon_gem.c
> index 84843b3b3aef..261fcbae88d7 100644
> --- a/drivers/gpu/drm/radeon/radeon_gem.c
> +++ b/drivers/gpu/drm/radeon/radeon_gem.c
> @@ -693,7 +693,7 @@ int radeon_gem_va_ioctl(struct drm_device *dev, void *data,
> }
>
> /* !! DONT REMOVE !!
> - * We don't support vm_id yet, to be sure we don't have have broken
> + * We don't support vm_id yet, to be sure we don't have broken
> * userspace, reject anyone trying to use non 0 value thus moving
> * forward we can use those fields without breaking existant userspace
> */