mirror of
https://github.com/torvalds/linux.git
synced 2026-01-25 15:03:52 +08:00
media: i2c: imx335: Fix frame size enumeration
In commitcfa49ff055("media: i2c: imx335: Support 2592x1940 10-bit mode") the IMX335 driver was extended to support multiple output bitdepth modes. This incorrectly extended the frame size enumeration to check against the supported mbus_codes array instead of the supported mode/frame array. This has the unwanted side effect of reporting the currently supported frame size 2592x1944 three times. Fix the check accordingly to report a frame size for each supported size, which is presently only a single entry. Fixes:cfa49ff055("media: i2c: imx335: Support 2592x1940 10-bit mode") Cc: stable@vger.kernel.org Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
This commit is contained in:
committed by
Hans Verkuil
parent
00cd2d3a1b
commit
b240df2913
@@ -660,7 +660,8 @@ static int imx335_enum_frame_size(struct v4l2_subdev *sd,
|
||||
struct imx335 *imx335 = to_imx335(sd);
|
||||
u32 code;
|
||||
|
||||
if (fsize->index > ARRAY_SIZE(imx335_mbus_codes))
|
||||
/* Only a single supported_mode available. */
|
||||
if (fsize->index > 0)
|
||||
return -EINVAL;
|
||||
|
||||
code = imx335_get_format_code(imx335, fsize->code);
|
||||
|
||||
Reference in New Issue
Block a user