* [PATCH] staging: media: reformat line to 80 chars or less
@ 2018-02-12 12:05 Parthiban Nallathambi
2018-02-14 23:01 ` Steve Longerbeam
0 siblings, 1 reply; 4+ messages in thread
From: Parthiban Nallathambi @ 2018-02-12 12:05 UTC (permalink / raw)
To: slongerbeam
Cc: devel, gregkh, linux-kernel, p.zabel, Parthiban Nallathambi,
mchehab, linux-media
This is a cleanup patch to fix line length issue found
by checkpatch.pl script.
In this patch, line 144 have been wrapped.
Signed-off-by: Parthiban Nallathambi <pn@denx.de>
---
drivers/staging/media/imx/imx-media-capture.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/media/imx/imx-media-capture.c b/drivers/staging/media/imx/imx-media-capture.c
index 576bdc7e9c42..0ccabe04b0e1 100644
--- a/drivers/staging/media/imx/imx-media-capture.c
+++ b/drivers/staging/media/imx/imx-media-capture.c
@@ -141,7 +141,8 @@ static int capture_enum_frameintervals(struct file *file, void *fh,
fie.code = cc->codes[0];
- ret = v4l2_subdev_call(priv->src_sd, pad, enum_frame_interval, NULL, &fie);
+ ret = v4l2_subdev_call(priv->src_sd, pad, enum_frame_interval,
+ NULL, &fie);
if (ret)
return ret;
--
2.14.3
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] staging: media: reformat line to 80 chars or less
2018-02-12 12:05 [PATCH] staging: media: reformat line to 80 chars or less Parthiban Nallathambi
@ 2018-02-14 23:01 ` Steve Longerbeam
2018-02-15 9:25 ` [PATCH v2] media: imx: capture: " Parthiban Nallathambi
0 siblings, 1 reply; 4+ messages in thread
From: Steve Longerbeam @ 2018-02-14 23:01 UTC (permalink / raw)
To: Parthiban Nallathambi
Cc: devel, gregkh, linux-kernel, p.zabel, mchehab, linux-media
Hi Parthiban, please rename the commit title to
"media: imx: capture: reformat line to 80 chars or less"
Otherwise it is fine with me.
Steve
On 02/12/2018 04:05 AM, Parthiban Nallathambi wrote:
> This is a cleanup patch to fix line length issue found
> by checkpatch.pl script.
>
> In this patch, line 144 have been wrapped.
>
> Signed-off-by: Parthiban Nallathambi <pn@denx.de>
> ---
> drivers/staging/media/imx/imx-media-capture.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/media/imx/imx-media-capture.c b/drivers/staging/media/imx/imx-media-capture.c
> index 576bdc7e9c42..0ccabe04b0e1 100644
> --- a/drivers/staging/media/imx/imx-media-capture.c
> +++ b/drivers/staging/media/imx/imx-media-capture.c
> @@ -141,7 +141,8 @@ static int capture_enum_frameintervals(struct file *file, void *fh,
>
> fie.code = cc->codes[0];
>
> - ret = v4l2_subdev_call(priv->src_sd, pad, enum_frame_interval, NULL, &fie);
> + ret = v4l2_subdev_call(priv->src_sd, pad, enum_frame_interval,
> + NULL, &fie);
> if (ret)
> return ret;
>
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2] media: imx: capture: reformat line to 80 chars or less
2018-02-14 23:01 ` Steve Longerbeam
@ 2018-02-15 9:25 ` Parthiban Nallathambi
2018-02-15 17:37 ` Steve Longerbeam
0 siblings, 1 reply; 4+ messages in thread
From: Parthiban Nallathambi @ 2018-02-15 9:25 UTC (permalink / raw)
To: slongerbeam
Cc: p.zabel, mchehab, gregkh, linux-media, devel, linux-kernel,
Parthiban Nallathambi
This is a cleanup patch to fix line length issue found
by checkpatch.pl script.
In this patch, line 144 have been wrapped.
Signed-off-by: Parthiban Nallathambi <pn@denx.de>
---
Changes in v2:
- Changed commit message
drivers/staging/media/imx/imx-media-capture.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/media/imx/imx-media-capture.c b/drivers/staging/media/imx/imx-media-capture.c
index 576bdc7e9c42..0ccabe04b0e1 100644
--- a/drivers/staging/media/imx/imx-media-capture.c
+++ b/drivers/staging/media/imx/imx-media-capture.c
@@ -141,7 +141,8 @@ static int capture_enum_frameintervals(struct file *file, void *fh,
fie.code = cc->codes[0];
- ret = v4l2_subdev_call(priv->src_sd, pad, enum_frame_interval, NULL, &fie);
+ ret = v4l2_subdev_call(priv->src_sd, pad, enum_frame_interval,
+ NULL, &fie);
if (ret)
return ret;
--
2.14.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] media: imx: capture: reformat line to 80 chars or less
2018-02-15 9:25 ` [PATCH v2] media: imx: capture: " Parthiban Nallathambi
@ 2018-02-15 17:37 ` Steve Longerbeam
0 siblings, 0 replies; 4+ messages in thread
From: Steve Longerbeam @ 2018-02-15 17:37 UTC (permalink / raw)
To: Parthiban Nallathambi
Cc: devel, gregkh, linux-kernel, p.zabel, mchehab, linux-media
Acked-by: Steve Longerbeam <steve_longerbeam@mentor.com>
On 02/15/2018 01:25 AM, Parthiban Nallathambi wrote:
> This is a cleanup patch to fix line length issue found
> by checkpatch.pl script.
>
> In this patch, line 144 have been wrapped.
>
> Signed-off-by: Parthiban Nallathambi <pn@denx.de>
> ---
> Changes in v2:
> - Changed commit message
>
> drivers/staging/media/imx/imx-media-capture.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/media/imx/imx-media-capture.c b/drivers/staging/media/imx/imx-media-capture.c
> index 576bdc7e9c42..0ccabe04b0e1 100644
> --- a/drivers/staging/media/imx/imx-media-capture.c
> +++ b/drivers/staging/media/imx/imx-media-capture.c
> @@ -141,7 +141,8 @@ static int capture_enum_frameintervals(struct file *file, void *fh,
>
> fie.code = cc->codes[0];
>
> - ret = v4l2_subdev_call(priv->src_sd, pad, enum_frame_interval, NULL, &fie);
> + ret = v4l2_subdev_call(priv->src_sd, pad, enum_frame_interval,
> + NULL, &fie);
> if (ret)
> return ret;
>
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-02-15 17:37 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-12 12:05 [PATCH] staging: media: reformat line to 80 chars or less Parthiban Nallathambi
2018-02-14 23:01 ` Steve Longerbeam
2018-02-15 9:25 ` [PATCH v2] media: imx: capture: " Parthiban Nallathambi
2018-02-15 17:37 ` Steve Longerbeam
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).