All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] staging: davinci: Fix typo in staging/media/davinci
@ 2013-03-26 15:23 Masanari Iida
  2013-03-27  4:27 ` Prabhakar Lad
  0 siblings, 1 reply; 2+ messages in thread
From: Masanari Iida @ 2013-03-26 15:23 UTC (permalink / raw)
  To: manjunath.hadli, prabhakar.lad, linux-media, devel; +Cc: gregkh, Masanari Iida

Correct spelling typo in staging/media/davinci

Signed-off-by: Masanari Iida <standby24x7@gmail.com>
---
 drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt | 2 +-
 drivers/staging/media/davinci_vpfe/dm365_isif.c        | 6 +++---
 drivers/staging/media/davinci_vpfe/vpfe_video.c        | 8 ++++----
 drivers/staging/media/davinci_vpfe/vpfe_video.h        | 2 +-
 4 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt b/drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt
index 1dbd564..a1e9177 100644
--- a/drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt
+++ b/drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt
@@ -38,7 +38,7 @@ interface to userspace.
 	DAVINCI RESIZER A
 	DAVINCI RESIZER B
 
-Each possible link in the VPFE is modeled by a link in the Media controller
+Each possible link in the VPFE is modelled by a link in the Media controller
 interface. For an example program see [1].
 
 
diff --git a/drivers/staging/media/davinci_vpfe/dm365_isif.c b/drivers/staging/media/davinci_vpfe/dm365_isif.c
index ebeea72..6d4a93c 100644
--- a/drivers/staging/media/davinci_vpfe/dm365_isif.c
+++ b/drivers/staging/media/davinci_vpfe/dm365_isif.c
@@ -685,7 +685,7 @@ static void isif_config_bclamp(struct vpfe_isif_device *isif,
 	val = (bc->bc_mode_color & ISIF_BC_MODE_COLOR_MASK) <<
 		ISIF_BC_MODE_COLOR_SHIFT;
 
-	/* Enable BC and horizontal clamp caculation paramaters */
+	/* Enable BC and horizontal clamp calculation paramaters */
 	val = val | 1 | ((bc->horz.mode & ISIF_HORZ_BC_MODE_MASK) <<
 	      ISIF_HORZ_BC_MODE_SHIFT);
 
@@ -722,7 +722,7 @@ static void isif_config_bclamp(struct vpfe_isif_device *isif,
 		isif_write(isif->isif_cfg.base_addr, val, CLHWIN2);
 	}
 
-	/* vertical clamp caculation paramaters */
+	/* vertical clamp calculation paramaters */
 	/* OB H Valid */
 	val = bc->vert.ob_h_sz_calc & ISIF_VERT_BC_OB_H_SZ_MASK;
 
@@ -1569,7 +1569,7 @@ isif_pad_set_crop(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh,
 		crop->rect.width = format->width;
 		crop->rect.height = format->height;
 	}
-	/* adjust the width to 16 pixel boundry */
+	/* adjust the width to 16 pixel boundary */
 	crop->rect.width = ((crop->rect.width + 15) & ~0xf);
 	vpfe_isif->crop = crop->rect;
 	if (crop->which == V4L2_SUBDEV_FORMAT_ACTIVE) {
diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.c b/drivers/staging/media/davinci_vpfe/vpfe_video.c
index 99ccbeb..c91d356 100644
--- a/drivers/staging/media/davinci_vpfe/vpfe_video.c
+++ b/drivers/staging/media/davinci_vpfe/vpfe_video.c
@@ -357,7 +357,7 @@ static int vpfe_pipeline_disable(struct vpfe_pipeline *pipe)
  *
  * Set the pipeline to the given stream state.
  *
- * Return 0 if successfull, or the return value of the failed video::s_stream
+ * Return 0 if successful, or the return value of the failed video::s_stream
  * operation otherwise.
  */
 static int vpfe_pipeline_set_stream(struct vpfe_pipeline *pipe,
@@ -644,7 +644,7 @@ static int vpfe_g_fmt(struct file *file, void *priv,
  * fills v4l2_fmtdesc structure with output format set on adjacent subdev,
  * only one format is enumearted as subdevs are already configured
  *
- * Return 0 if successfull, error code otherwise
+ * Return 0 if successful, error code otherwise
  */
 static int vpfe_enum_fmt(struct file *file, void  *priv,
 				   struct v4l2_fmtdesc *fmt)
@@ -769,7 +769,7 @@ static int vpfe_try_fmt(struct file *file, void *priv,
  * fills v4l2_input structure with input available on media chain,
  * only one input is enumearted as media chain is setup by this time
  *
- * Return 0 if successfull, -EINVAL is media chain is invalid
+ * Return 0 if successful, -EINVAL is media chain is invalid
  */
 static int vpfe_enum_input(struct file *file, void *priv,
 				 struct v4l2_input *inp)
@@ -779,7 +779,7 @@ static int vpfe_enum_input(struct file *file, void *priv,
 	struct vpfe_device *vpfe_dev = video->vpfe_dev;
 
 	v4l2_dbg(1, debug, &vpfe_dev->v4l2_dev, "vpfe_enum_input\n");
-	/* enumerate from the subdev user has choosen through mc */
+	/* enumerate from the subdev user has chosen through mc */
 	if (inp->index < sdinfo->num_inputs) {
 		memcpy(inp, &sdinfo->inputs[inp->index],
 		       sizeof(struct v4l2_input));
diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.h b/drivers/staging/media/davinci_vpfe/vpfe_video.h
index bf8af01..df0aeec 100644
--- a/drivers/staging/media/davinci_vpfe/vpfe_video.h
+++ b/drivers/staging/media/davinci_vpfe/vpfe_video.h
@@ -138,7 +138,7 @@ struct vpfe_video_device {
 	v4l2_std_id				stdid;
 	/*
 	 * offset where second field starts from the starting of the
-	 * buffer for field seperated YCbCr formats
+	 * buffer for field separated YCbCr formats
 	 */
 	u32					field_off;
 };
-- 
1.8.2.135.g7b592fa


^ permalink raw reply related	[flat|nested] 2+ messages in thread
* Re: [PATCH] staging: davinci: Fix typo in staging/media/davinci
  2013-03-26 15:23 [PATCH] staging: davinci: Fix typo in staging/media/davinci Masanari Iida
@ 2013-03-27  4:27 ` Prabhakar Lad
  0 siblings, 0 replies; 2+ messages in thread
From: Prabhakar Lad @ 2013-03-27  4:27 UTC (permalink / raw)
  To: Masanari Iida; +Cc: dlos, linux-media, devel, Greg Kroah-Hartman

Hi Masanari,

Thanks for the patch!

On Tue, Mar 26, 2013 at 8:53 PM, Masanari Iida <standby24x7@gmail.com> wrote:
> Correct spelling typo in staging/media/davinci
>
> Signed-off-by: Masanari Iida <standby24x7@gmail.com>
> ---
>  drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt | 2 +-
>  drivers/staging/media/davinci_vpfe/dm365_isif.c        | 6 +++---
>  drivers/staging/media/davinci_vpfe/vpfe_video.c        | 8 ++++----
>  drivers/staging/media/davinci_vpfe/vpfe_video.h        | 2 +-
>  4 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt b/drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt
> index 1dbd564..a1e9177 100644
> --- a/drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt
> +++ b/drivers/staging/media/davinci_vpfe/davinci-vpfe-mc.txt
> @@ -38,7 +38,7 @@ interface to userspace.
>         DAVINCI RESIZER A
>         DAVINCI RESIZER B
>
> -Each possible link in the VPFE is modeled by a link in the Media controller
> +Each possible link in the VPFE is modelled by a link in the Media controller

s/modeled/modelled are one and the same. you can keep it as is.
Rest of the patch looks OK. With this change you can add my ACK.

Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>

Regards,
--Prabhakar

>  interface. For an example program see [1].
>
>
> diff --git a/drivers/staging/media/davinci_vpfe/dm365_isif.c b/drivers/staging/media/davinci_vpfe/dm365_isif.c
> index ebeea72..6d4a93c 100644
> --- a/drivers/staging/media/davinci_vpfe/dm365_isif.c
> +++ b/drivers/staging/media/davinci_vpfe/dm365_isif.c
> @@ -685,7 +685,7 @@ static void isif_config_bclamp(struct vpfe_isif_device *isif,
>         val = (bc->bc_mode_color & ISIF_BC_MODE_COLOR_MASK) <<
>                 ISIF_BC_MODE_COLOR_SHIFT;
>
> -       /* Enable BC and horizontal clamp caculation paramaters */
> +       /* Enable BC and horizontal clamp calculation paramaters */
>         val = val | 1 | ((bc->horz.mode & ISIF_HORZ_BC_MODE_MASK) <<
>               ISIF_HORZ_BC_MODE_SHIFT);
>
> @@ -722,7 +722,7 @@ static void isif_config_bclamp(struct vpfe_isif_device *isif,
>                 isif_write(isif->isif_cfg.base_addr, val, CLHWIN2);
>         }
>
> -       /* vertical clamp caculation paramaters */
> +       /* vertical clamp calculation paramaters */
>         /* OB H Valid */
>         val = bc->vert.ob_h_sz_calc & ISIF_VERT_BC_OB_H_SZ_MASK;
>
> @@ -1569,7 +1569,7 @@ isif_pad_set_crop(struct v4l2_subdev *sd, struct v4l2_subdev_fh *fh,
>                 crop->rect.width = format->width;
>                 crop->rect.height = format->height;
>         }
> -       /* adjust the width to 16 pixel boundry */
> +       /* adjust the width to 16 pixel boundary */
>         crop->rect.width = ((crop->rect.width + 15) & ~0xf);
>         vpfe_isif->crop = crop->rect;
>         if (crop->which == V4L2_SUBDEV_FORMAT_ACTIVE) {
> diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.c b/drivers/staging/media/davinci_vpfe/vpfe_video.c
> index 99ccbeb..c91d356 100644
> --- a/drivers/staging/media/davinci_vpfe/vpfe_video.c
> +++ b/drivers/staging/media/davinci_vpfe/vpfe_video.c
> @@ -357,7 +357,7 @@ static int vpfe_pipeline_disable(struct vpfe_pipeline *pipe)
>   *
>   * Set the pipeline to the given stream state.
>   *
> - * Return 0 if successfull, or the return value of the failed video::s_stream
> + * Return 0 if successful, or the return value of the failed video::s_stream
>   * operation otherwise.
>   */
>  static int vpfe_pipeline_set_stream(struct vpfe_pipeline *pipe,
> @@ -644,7 +644,7 @@ static int vpfe_g_fmt(struct file *file, void *priv,
>   * fills v4l2_fmtdesc structure with output format set on adjacent subdev,
>   * only one format is enumearted as subdevs are already configured
>   *
> - * Return 0 if successfull, error code otherwise
> + * Return 0 if successful, error code otherwise
>   */
>  static int vpfe_enum_fmt(struct file *file, void  *priv,
>                                    struct v4l2_fmtdesc *fmt)
> @@ -769,7 +769,7 @@ static int vpfe_try_fmt(struct file *file, void *priv,
>   * fills v4l2_input structure with input available on media chain,
>   * only one input is enumearted as media chain is setup by this time
>   *
> - * Return 0 if successfull, -EINVAL is media chain is invalid
> + * Return 0 if successful, -EINVAL is media chain is invalid
>   */
>  static int vpfe_enum_input(struct file *file, void *priv,
>                                  struct v4l2_input *inp)
> @@ -779,7 +779,7 @@ static int vpfe_enum_input(struct file *file, void *priv,
>         struct vpfe_device *vpfe_dev = video->vpfe_dev;
>
>         v4l2_dbg(1, debug, &vpfe_dev->v4l2_dev, "vpfe_enum_input\n");
> -       /* enumerate from the subdev user has choosen through mc */
> +       /* enumerate from the subdev user has chosen through mc */
>         if (inp->index < sdinfo->num_inputs) {
>                 memcpy(inp, &sdinfo->inputs[inp->index],
>                        sizeof(struct v4l2_input));
> diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.h b/drivers/staging/media/davinci_vpfe/vpfe_video.h
> index bf8af01..df0aeec 100644
> --- a/drivers/staging/media/davinci_vpfe/vpfe_video.h
> +++ b/drivers/staging/media/davinci_vpfe/vpfe_video.h
> @@ -138,7 +138,7 @@ struct vpfe_video_device {
>         v4l2_std_id                             stdid;
>         /*
>          * offset where second field starts from the starting of the
> -        * buffer for field seperated YCbCr formats
> +        * buffer for field separated YCbCr formats
>          */
>         u32                                     field_off;
>  };
> --
> 1.8.2.135.g7b592fa
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-03-27  4:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-26 15:23 [PATCH] staging: davinci: Fix typo in staging/media/davinci Masanari Iida
2013-03-27  4:27 ` Prabhakar Lad

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.