On 3/31/20 3:06 PM, Dafna Hirschfeld wrote: > When enumerating the frame sizes, the value sent to > imx219_get_format_code should be fse->code > (the code from the ioctl) and not imx219->fmt.code > which is the code set currently in the driver. > > Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@xxxxxxxxxxxxx> Reviewed-by: Helen Koike <helen.koike@xxxxxxxxxxxxx> > --- > drivers/media/i2c/imx219.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/i2c/imx219.c b/drivers/media/i2c/imx219.c > index b1f30debe449..df2a6ed7c8ac 100644 > --- a/drivers/media/i2c/imx219.c > +++ b/drivers/media/i2c/imx219.c > @@ -781,7 +781,7 @@ static int imx219_enum_frame_size(struct v4l2_subdev *sd, > if (fse->index >= ARRAY_SIZE(supported_modes)) > return -EINVAL; > > - if (fse->code != imx219_get_format_code(imx219, imx219->fmt.code)) > + if (fse->code != imx219_get_format_code(imx219, fse->code)) > return -EINVAL; > > fse->min_width = supported_modes[fse->index].width; >
