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