Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
 static const struct sr030pc30_format *try_fmt(struct v4l2_subdev *sd,
                                              struct v4l2_mbus_framefmt *mf)
 {
-       int i = ARRAY_SIZE(sr030pc30_formats);
+       int i;
 
        sr030pc30_try_frame_size(mf);
 
-       while (i--)
+       for (i = 0; i < ARRAY_SIZE(sr030pc30_formats); i++) {
                if (mf->code == sr030pc30_formats[i].code)
                        break;
+       }
+       if (i == ARRAY_SIZE(sr030pc30_formats))
+               i = 0;
 
        mf->code = sr030pc30_formats[i].code;