media: rkvdec: removed redundant blank line

[PATCH v2] media: rkvdec: removed redundant blank line

Posted by Emma Christy 2 years, 7 months ago

Adhere to Linux kernel coding style. Removed redundant blank line.
Issue found by checkpatch.

Signed-off-by: Emma Christy <emma.t.christy@gmail.com>
---
Changes in v2:
  - Fixed Singed-off-by.
  - Fixed Subject line.

 drivers/staging/media/rkvdec/rkvdec-vp9.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/staging/media/rkvdec/rkvdec-vp9.c b/drivers/staging/media/rkvdec/rkvdec-vp9.c
index cfae99b40ccb..0e7e16f20eeb 100644
--- a/drivers/staging/media/rkvdec/rkvdec-vp9.c
+++ b/drivers/staging/media/rkvdec/rkvdec-vp9.c
@@ -227,7 +227,6 @@ static void init_intra_only_probs(struct rkvdec_ctx *ctx,
 				}
 			}
 		}
-
 	}
 
 	for (i = 0; i < sizeof(v4l2_vp9_kf_uv_mode_prob); ++i) {
-- 
2.40.1

Re: [PATCH v2] media: rkvdec: removed redundant blank line

Posted by Mauro Carvalho Chehab 2 years, 7 months ago

Em Wed, 21 Jun 2023 18:42:47 +0300
Emma Christy <emma.t.christy@gmail.com> escreveu:

> Adhere to Linux kernel coding style. Removed redundant blank line.
> Issue found by checkpatch.
> 
> Signed-off-by: Emma Christy <emma.t.christy@gmail.com>

Reviewed-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> ---
> Changes in v2:
>   - Fixed Singed-off-by.
>   - Fixed Subject line.
> 
>  drivers/staging/media/rkvdec/rkvdec-vp9.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/staging/media/rkvdec/rkvdec-vp9.c b/drivers/staging/media/rkvdec/rkvdec-vp9.c
> index cfae99b40ccb..0e7e16f20eeb 100644
> --- a/drivers/staging/media/rkvdec/rkvdec-vp9.c
> +++ b/drivers/staging/media/rkvdec/rkvdec-vp9.c
> @@ -227,7 +227,6 @@ static void init_intra_only_probs(struct rkvdec_ctx *ctx,
>  				}
>  			}
>  		}
> -
>  	}
>  
>  	for (i = 0; i < sizeof(v4l2_vp9_kf_uv_mode_prob); ++i) {

Re: [PATCH v2] media: rkvdec: removed redundant blank line

Posted by Nicolas Dufresne 2 years, 7 months ago

Thanks for taking the time to fix this.

Le mercredi 21 juin 2023 à 18:42 +0300, Emma Christy a écrit :
> Adhere to Linux kernel coding style. Removed redundant blank line.
> Issue found by checkpatch.
> 
> Signed-off-by: Emma Christy <emma.t.christy@gmail.com>

Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>

> ---
> Changes in v2:
>   - Fixed Singed-off-by.
>   - Fixed Subject line.
> 
>  drivers/staging/media/rkvdec/rkvdec-vp9.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/staging/media/rkvdec/rkvdec-vp9.c b/drivers/staging/media/rkvdec/rkvdec-vp9.c
> index cfae99b40ccb..0e7e16f20eeb 100644
> --- a/drivers/staging/media/rkvdec/rkvdec-vp9.c
> +++ b/drivers/staging/media/rkvdec/rkvdec-vp9.c
> @@ -227,7 +227,6 @@ static void init_intra_only_probs(struct rkvdec_ctx *ctx,
>  				}
>  			}
>  		}
> -
>  	}
>  
>  	for (i = 0; i < sizeof(v4l2_vp9_kf_uv_mode_prob); ++i) {