Skip to contentSkip to conversation

staging: media: reformat line to 80 chars or less

PATCH Superseded Parthiban Nallathambi 2 participants 4 messages Feb 12, 2018 – Feb 15, 2018

Subsystems (4)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 staging: media: reformat line to 80 chars or less

This is a cleanup patch to fix line length issue found by checkpatch.pl script.

In this patch, line 144 have been wrapped.

---
 drivers/staging/media/imx/imx-media-capture.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/staging/media/imx/imx-media-capture.c +2 -1
index 576bdc7e9c42..0ccabe04b0e1 100644
@@ -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;
 
Signature · 6 lines
2.14.3

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
maintainer
#1 lore raw

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

Quoted 15 lines · 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.

---
  drivers/staging/media/imx/imx-media-capture.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)
drivers/staging/media/imx/imx-media-capture.c +0 -0
index 576bdc7e9c42..0ccabe04b0e1 100644

drivers/staging/media/imx/imx-media-capture.c static int capture_enum_frameintervals(struct file *file, void *fh, · line 148

Quoted 9 lines
drivers/staging/media/imx/imx-media-capture.c +2 -1
@@ -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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

#2 lore raw

PATCH v2 1/1 media: imx: capture: reformat line to 80 chars or less

This is a cleanup patch to fix line length issue found by checkpatch.pl script.

In this patch, line 144 have been wrapped.

---
Changes in v2:
- Changed commit message

 drivers/staging/media/imx/imx-media-capture.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/staging/media/imx/imx-media-capture.c +2 -1
index 576bdc7e9c42..0ccabe04b0e1 100644
@@ -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;
 
Signature · 1 line
2.14.3
maintainer Acked-by on v2 1/1
#3 lore raw

Re: [PATCH v2] media: imx: capture: reformat line to 80 chars or less

Quoted 18 lines · 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.

---
Changes in v2:
- Changed commit message

  drivers/staging/media/imx/imx-media-capture.c | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)
drivers/staging/media/imx/imx-media-capture.c +0 -0
index 576bdc7e9c42..0ccabe04b0e1 100644

drivers/staging/media/imx/imx-media-capture.c static int capture_enum_frameintervals(struct file *file, void *fh, · line 148

Quoted 9 lines
drivers/staging/media/imx/imx-media-capture.c +2 -1
@@ -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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel