Re: [PATCH] staging: davinci: Fix typo in staging/media/davinci

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Masanari,

Thanks for the patch!

On Tue, Mar 26, 2013 at 8:53 PM, Masanari Iida <standby24x7@xxxxxxxxx> wrote:
> Correct spelling typo in staging/media/davinci
>
> Signed-off-by: Masanari Iida <standby24x7@xxxxxxxxx>
> ---
>  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@xxxxxxxxx>

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@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux