2021-12-14 13:55:03

by Xiang wangx

Subject: [PATCH] drm/radeon: Fix syntax errors in comments

Delete the redundant word 'we'.

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

diff --git a/drivers/gpu/drm/radeon/radeon_vce.c b/drivers/gpu/drm/radeon/radeon_vce.c
index 511a942e851d..ca4a36464340 100644
--- a/drivers/gpu/drm/radeon/radeon_vce.c
+++ b/drivers/gpu/drm/radeon/radeon_vce.c
@@ -513,7 +513,7 @@ int radeon_vce_cs_reloc(struct radeon_cs_parser *p, int lo, int hi,
* @allocated: allocated a new handle?
*
* Validates the handle and return the found session index or -EINVAL
- * we we don't have another free session index.
+ * we don't have another free session index.
*/
static int radeon_vce_validate_handle(struct radeon_cs_parser *p,
uint32_t handle, bool *allocated)
--
2.20.1

2021-12-14 14:54:09

by Christian König

Subject: Re: [PATCH] drm/radeon: Fix syntax errors in comments

Am 14.12.21 um 14:52 schrieb Xiang wangx:
> Delete the redundant word 'we'.
>
> Signed-off-by: Xiang wangx <[email protected]>

Well not a syntax error in the sense of a coding error, but valid fix
anyway :)

Reviewed-by: Christian König <[email protected]>

> ---
> drivers/gpu/drm/radeon/radeon_vce.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_vce.c b/drivers/gpu/drm/radeon/radeon_vce.c
> index 511a942e851d..ca4a36464340 100644
> --- a/drivers/gpu/drm/radeon/radeon_vce.c
> +++ b/drivers/gpu/drm/radeon/radeon_vce.c
> @@ -513,7 +513,7 @@ int radeon_vce_cs_reloc(struct radeon_cs_parser *p, int lo, int hi,
> * @allocated: allocated a new handle?
> *
> * Validates the handle and return the found session index or -EINVAL
> - * we we don't have another free session index.
> + * we don't have another free session index.
> */
> static int radeon_vce_validate_handle(struct radeon_cs_parser *p,
> uint32_t handle, bool *allocated)

2021-12-14 19:23:37

by Alex Deucher

Subject: Re: [PATCH] drm/radeon: Fix syntax errors in comments

Applied. Thanks!

Alex

On Tue, Dec 14, 2021 at 9:54 AM Christian König
<[email protected]> wrote:
>
> Am 14.12.21 um 14:52 schrieb Xiang wangx:
> > Delete the redundant word 'we'.
> >
> > Signed-off-by: Xiang wangx <[email protected]>
>
> Well not a syntax error in the sense of a coding error, but valid fix
> anyway :)
>
> Reviewed-by: Christian König <[email protected]>
>
> > ---
> > drivers/gpu/drm/radeon/radeon_vce.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/radeon/radeon_vce.c b/drivers/gpu/drm/radeon/radeon_vce.c
> > index 511a942e851d..ca4a36464340 100644
> > --- a/drivers/gpu/drm/radeon/radeon_vce.c
> > +++ b/drivers/gpu/drm/radeon/radeon_vce.c
> > @@ -513,7 +513,7 @@ int radeon_vce_cs_reloc(struct radeon_cs_parser *p, int lo, int hi,
> > * @allocated: allocated a new handle?
> > *
> > * Validates the handle and return the found session index or -EINVAL
> > - * we we don't have another free session index.
> > + * we don't have another free session index.
> > */
> > static int radeon_vce_validate_handle(struct radeon_cs_parser *p,
> > uint32_t handle, bool *allocated)
>