Doubling the default VTS value in 2 lane mode to keep the default fps at 30 fps while doubling the pixelrate make sense. But there is no reason to also double the minimum VTS value. If userspace wants to use the extra bandwidth to get more fps (at the expense of the max exposure time) then this should be allowed. Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> --- drivers/media/i2c/ov02c10.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/ov02c10.c b/drivers/media/i2c/ov02c10.c index a33f9d4033e6..ad5ab6c8a803 100644 --- a/drivers/media/i2c/ov02c10.c +++ b/drivers/media/i2c/ov02c10.c @@ -376,7 +376,7 @@ static const struct ov02c10_mode supported_modes[] = { .height = 1092, .hts = 2280, .vts_def = 2328, - .vts_min = 2328, + .vts_min = 1164, .mipi_lanes = 2, .reg_sequence = sensor_1928x1092_30fps_setting, .sequence_length = ARRAY_SIZE(sensor_1928x1092_30fps_setting), -- 2.48.1