| Message ID | 20230621092223.10805-1-emma.t.christy@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Subject: [PATCH] media: rkvdec: removed redundant blank line | expand |
On Wed, Jun 21, 2023 at 12:22:23PM +0300, emma christy wrote: > Adhere to Linux kernel coding style. Removed redundant blank line. > Issue found by checkpatch. > > Signed-off-by: emma christy <emma.t.christy@gmail.com> You might need to write this as "Emma Christy", right? Also, your "Subject" line has an additional [PATCH] and "Subject:" in it somehow. thanks, greg k-h
On Wed, 21 Jun 2023 12:51:11 +0200 Greg KH <gregkh@linuxfoundation.org> wrote: > On Wed, Jun 21, 2023 at 12:22:23PM +0300, emma christy wrote: > > Adhere to Linux kernel coding style. Removed redundant blank line. > > Issue found by checkpatch. > > > > Signed-off-by: emma christy <emma.t.christy@gmail.com> > > You might need to write this as "Emma Christy", right? > > Also, your "Subject" line has an additional [PATCH] and "Subject:" in > it somehow. > > thanks, > > greg k-h I am sorry for the errors, I am still new at this. I did mean to sign as "Emma Christy". Should I resubmit the patch, because of this? The extra "Subject" got added during "git send-email" and I didn't notice. The extra "[PATCH]" was added by me as I did not know it would be added automatically later. I will not make these mistakes again. Thank You, Emma Christy
On Wed, Jun 21, 2023 at 03:18:06PM +0300, emma wrote: > On Wed, 21 Jun 2023 12:51:11 +0200 > Greg KH <gregkh@linuxfoundation.org> wrote: > > > On Wed, Jun 21, 2023 at 12:22:23PM +0300, emma christy wrote: > > > Adhere to Linux kernel coding style. Removed redundant blank line. > > > Issue found by checkpatch. > > > > > > Signed-off-by: emma christy <emma.t.christy@gmail.com> > > > > You might need to write this as "Emma Christy", right? > > > > Also, your "Subject" line has an additional [PATCH] and "Subject:" in > > it somehow. > > > > thanks, > > > > greg k-h > > I am sorry for the errors, I am still new at this. > > I did mean to sign as "Emma Christy". Should I resubmit the patch, > because of this? We can't accept it because of this issue, and the "Subject:" issue, so yes, you will need to send a v2 of this. thanks, greg k-h
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) {
Adhere to Linux kernel coding style. Removed redundant blank line. Issue found by checkpatch. Signed-off-by: emma christy <emma.t.christy@gmail.com> --- drivers/staging/media/rkvdec/rkvdec-vp9.c | 1 - 1 file changed, 1 deletion(-)