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
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
