Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1266001

Re: [PATCH v2] Staging: media: davinci_vpfe: Fix over 80 characters coding style issue

From Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2] Staging: media: davinci_vpfe: Fix over 80 characters coding style issue
Date 2015-11-09 22:00 +0100
Message-ID <qt1HJ-2YW-11@gated-at.bofh.it> (permalink)
References <pW37c-1PR-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hello Junsu,

Thank you for the patch.

On Monday 10 August 2015 16:40:59 Junsu Shin wrote:
> This is a patch to the dm365_ipipe.c that fixes over 80 characters warning
> detected.

It's a bit ironic to submit a patch fixing a 80 characters limit issue and 
having a commit message that is larger than 72 characters per line :-)

Anyway, I've wrapped the commit message to 72 columns and applied the patch to 
my tree.

> Signed-off-by: Junsu Shin <jjunes0@gmail.com>
> ---
>  drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/media/davinci_vpfe/dm365_ipipe.c
> b/drivers/staging/media/davinci_vpfe/dm365_ipipe.c index 1bbb90c..a474adf
> 100644
> --- a/drivers/staging/media/davinci_vpfe/dm365_ipipe.c
> +++ b/drivers/staging/media/davinci_vpfe/dm365_ipipe.c
> @@ -1536,8 +1536,9 @@ ipipe_get_format(struct v4l2_subdev *sd, struct
> v4l2_subdev_pad_config *cfg, * @fse: pointer to v4l2_subdev_frame_size_enum
> structure.
>   */
>  static int
> -ipipe_enum_frame_size(struct v4l2_subdev *sd, struct v4l2_subdev_pad_config
> *cfg, -			  struct v4l2_subdev_frame_size_enum *fse)
> +ipipe_enum_frame_size(struct v4l2_subdev *sd,
> +		       struct v4l2_subdev_pad_config *cfg,
> +		       struct v4l2_subdev_frame_size_enum *fse)
>  {
>  	struct vpfe_ipipe_device *ipipe = v4l2_get_subdevdata(sd);
>  	struct v4l2_mbus_framefmt format;

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

Re: [PATCH v2] Staging: media: davinci_vpfe: Fix over 80 characters coding style issue Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2015-11-09 22:00 +0100

csiph-web