From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932125AbcGOGVO (ORCPT ); Fri, 15 Jul 2016 02:21:14 -0400 Received: from mailout1.w1.samsung.com ([210.118.77.11]:62265 "EHLO mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750764AbcGOGVM (ORCPT ); Fri, 15 Jul 2016 02:21:12 -0400 X-AuditID: cbfec7f4-f796c6d000001486-6e-578880d51a6d Message-id: <578880D4.6000105@samsung.com> Date: Fri, 15 Jul 2016 08:21:08 +0200 From: Jacek Anaszewski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130804 Thunderbird/17.0.8 MIME-version: 1.0 To: Shuah Khan Cc: andrzej.p@samsung.com, mchehab@kernel.org, javier@osg.samsung.com, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] media: s5p-jpeg add missing blank lines after declarations References: <1468526516-8892-1-git-send-email-shuahkh@osg.samsung.com> In-reply-to: <1468526516-8892-1-git-send-email-shuahkh@osg.samsung.com> Content-type: text/plain; charset=ISO-8859-1; format=flowed Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrFLMWRmVeSWpSXmKPExsVy+t/xa7pXGzrCDT5cVreY9bKdxeLN2zVM FpseX2O1uLxrDptFz4atrBbLNv1hspj65QOLA7vHplWdbB6bl9R7bOm/y+7Rt2UVo8fnTXIB rFFcNimpOZllqUX6dglcGadaNrMUTJWqOLF3J2sD43WRLkZODgkBE4k9kz+wQdhiEhfurQey uTiEBJYySlzr6GKBcJ4xSjTfOQlWxSugJfF7y33mLkYODhYBVYlzi6NBwmwChhI/X7xmArFF BSIk/pzexwpRLijxY/I9FhBbREBD4smsSWBjmAVWMUp0T1IHsYUF/CWWNO8DqxEScJM4MW8Z WC+ngLvEu8a5rBD11hIrJ21jhLDlJTavecs8gVFgFpIVs5CUzUJStoCReRWjaGppckFxUnqu oV5xYm5xaV66XnJ+7iZGSIh/2cG4+JjVIUYBDkYlHl6Bo+3hQqyJZcWVuYcYJTiYlUR4deo6 woV4UxIrq1KL8uOLSnNSiw8xSnOwKInzzt31PkRIID2xJDU7NbUgtQgmy8TBKdXAmO6qMTdh RfWpyffmvpdRO7X5wW2FJ2IzJj11sDri1nSTcdmHiZL7rNl/3Nt4xaHlmm8nX2hZdPX9U8/u bFpm9q5a557u6gvJ1803sSyNZ7X6qC1mE3nwU6FYzqyMuUyhvOL3RF0zahNzWw/lCqQ73n2w Om+i4vp5664ZmM22Y7C6fWNa0DfdXUosxRmJhlrMRcWJAFsxUZJtAgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Shuah, Thanks for the patch. On 07/14/2016 10:01 PM, Shuah Khan wrote: > Missing blank lines after declarations are making it hard to read the > code. Fix them and also fix other checkpatch warnings at the same time. > > Signed-off-by: Shuah Khan > --- > drivers/media/platform/s5p-jpeg/jpeg-core.c | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git a/drivers/media/platform/s5p-jpeg/jpeg-core.c b/drivers/media/platform/s5p-jpeg/jpeg-core.c > index 17bc94092..fe5554f 100644 > --- a/drivers/media/platform/s5p-jpeg/jpeg-core.c > +++ b/drivers/media/platform/s5p-jpeg/jpeg-core.c > @@ -537,6 +537,7 @@ static const u32 fourcc_to_dwngrd_schema_id[] = { > static int s5p_jpeg_get_dwngrd_sch_id_by_fourcc(u32 fourcc) > { > int i; > + > for (i = 0; i < ARRAY_SIZE(fourcc_to_dwngrd_schema_id); ++i) { > if (fourcc_to_dwngrd_schema_id[i] == fourcc) > return i; > @@ -1273,7 +1274,8 @@ static int enum_fmt(struct s5p_jpeg_fmt *sjpeg_formats, int n, > if (num == f->index) > break; > /* Correct type but haven't reached our index yet, > - * just increment per-type index */ > + * just increment per-type index > + */ > ++num; > } > } > @@ -1349,6 +1351,7 @@ static int s5p_jpeg_g_fmt(struct file *file, void *priv, struct v4l2_format *f) > pix->bytesperline = 0; > if (q_data->fmt->fourcc != V4L2_PIX_FMT_JPEG) { > u32 bpl = q_data->w; > + > if (q_data->fmt->colplanes == 1) > bpl = (bpl * q_data->fmt->depth) >> 3; > pix->bytesperline = bpl; > @@ -1374,6 +1377,7 @@ static struct s5p_jpeg_fmt *s5p_jpeg_find_format(struct s5p_jpeg_ctx *ctx, > > for (k = 0; k < ARRAY_SIZE(sjpeg_formats); k++) { > struct s5p_jpeg_fmt *fmt = &sjpeg_formats[k]; > + > if (fmt->fourcc == pixelformat && > fmt->flags & fmt_flag && > fmt->flags & ctx->jpeg->variant->fmt_ver_flag) { > @@ -1431,7 +1435,8 @@ static int vidioc_try_fmt(struct v4l2_format *f, struct s5p_jpeg_fmt *fmt, > return -EINVAL; > > /* V4L2 specification suggests the driver corrects the format struct > - * if any of the dimensions is unsupported */ > + * if any of the dimensions is unsupported > + */ > if (q_type == FMT_TYPE_OUTPUT) > jpeg_bound_align_image(ctx, &pix->width, S5P_JPEG_MIN_WIDTH, > S5P_JPEG_MAX_WIDTH, 0, > @@ -2490,6 +2495,7 @@ static void s5p_jpeg_buf_queue(struct vb2_buffer *vb) > if (ctx->mode == S5P_JPEG_DECODE && > vb->vb2_queue->type == V4L2_BUF_TYPE_VIDEO_OUTPUT) { > struct s5p_jpeg_q_data tmp, *q_data; > + > ctx->hdr_parsed = s5p_jpeg_parse_hdr(&tmp, > (unsigned long)vb2_plane_vaddr(vb, 0), > min((unsigned long)ctx->out_q.size, > @@ -3025,7 +3031,8 @@ static int s5p_jpeg_resume(struct device *dev) > > static const struct dev_pm_ops s5p_jpeg_pm_ops = { > SET_SYSTEM_SLEEP_PM_OPS(s5p_jpeg_suspend, s5p_jpeg_resume) > - SET_RUNTIME_PM_OPS(s5p_jpeg_runtime_suspend, s5p_jpeg_runtime_resume, NULL) > + SET_RUNTIME_PM_OPS(s5p_jpeg_runtime_suspend, s5p_jpeg_runtime_resume, > + NULL) > }; > > static struct s5p_jpeg_variant s5p_jpeg_drvdata = { > Acked-by: Jacek Anaszewski -- Best regards, Jacek Anaszewski