Em Fri, 23 Mar 2018 11:07:42 -0300 Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> escreveu: > Em Fri, 23 Mar 2018 15:48:41 +0200 > Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> escreveu: > > > An older version of the driver patches were merged accidentally which > > resulted in missing the array_size parameter that tells the length of the > > array that contains the different supported sizes. > > > > Bring it back to v4l2_find_nearest size and make the corresponding change > > for the drivers using it as well. > > > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > --- > > Hi Mauro, > > > > Here's the patch I mentioned. It restores the intended state of the > > v4l2_find_nearest_size() API as it was reviewed and acked (by Hans). > > > > This time the exact patch is tested for vivid. > > > > drivers/media/i2c/ov13858.c | 5 +++-- > > drivers/media/i2c/ov5670.c | 5 +++-- > > drivers/media/platform/vivid/vivid-vid-cap.c | 5 +++-- > > include/media/v4l2-common.h | 5 +++-- > > 4 files changed, 12 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/media/i2c/ov13858.c b/drivers/media/i2c/ov13858.c > > index 30ee9f71bf0d..420af1e32d4e 100644 > > --- a/drivers/media/i2c/ov13858.c > > +++ b/drivers/media/i2c/ov13858.c > > @@ -1375,8 +1375,9 @@ ov13858_set_pad_format(struct v4l2_subdev *sd, > > if (fmt->format.code != MEDIA_BUS_FMT_SGRBG10_1X10) > > fmt->format.code = MEDIA_BUS_FMT_SGRBG10_1X10; > > > > - mode = v4l2_find_nearest_size(supported_modes, width, height, > > - fmt->format.width, fmt->format.height); > > + mode = v4l2_find_nearest_size( > > + supported_modes, ARRAY_SIZE(supported_modes), width, height, > > + fmt->format.width, fmt->format.height); > > > Nitpick... I find ugly and arder to mentally parse things like the above, "arder" -> "harder" My keyboard sometimes is losing keystrokes. It seems it is approaching the time to replace it again. Thanks, Mauro
