Re: [patch] [media] sh_vou: almost forever loop in sh_vou_try_fmt_vid_out()

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

 



On Fri, Aug 23, 2013 at 3:24 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> The "i < " part of the "i < ARRAY_SIZE()" condition was missing.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Reviewed-by:  Lad, Prabhakar <prabhakar.lad@xxxxxx>

Regards,
--Prabhakar Lad

>
> diff --git a/drivers/media/platform/sh_vou.c b/drivers/media/platform/sh_vou.c
> index 7a9c5e9..41f612c 100644
> --- a/drivers/media/platform/sh_vou.c
> +++ b/drivers/media/platform/sh_vou.c
> @@ -776,9 +776,10 @@ static int sh_vou_try_fmt_vid_out(struct file *file, void *priv,
>         v4l_bound_align_image(&pix->width, 0, VOU_MAX_IMAGE_WIDTH, 1,
>                               &pix->height, 0, VOU_MAX_IMAGE_HEIGHT, 1, 0);
>
> -       for (i = 0; ARRAY_SIZE(vou_fmt); i++)
> +       for (i = 0; i < ARRAY_SIZE(vou_fmt); i++) {
>                 if (vou_fmt[i].pfmt == pix->pixelformat)
>                         return 0;
> +       }
>
>         pix->pixelformat = vou_fmt[0].pfmt;
>
--
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