media: atomisp: ov2680: Stop using half pixelclock for binned modes
authorHans de Goede <hdegoede@redhat.com>
Sun, 4 Jun 2023 16:14:03 +0000 (17:14 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 9 Jun 2023 14:43:06 +0000 (15:43 +0100)
commit7d45a432251a30ed0252655177e708229b163291
treea46f4c47539fa24c67988fdf343552abd0a2bfd9
parentef0feca22e866a8544847232fe32d5fdc11f0fbd
media: atomisp: ov2680: Stop using half pixelclock for binned modes

Stop using half pixelclock for binned modes this fixes:
1. The exposure being twice as high for binned mods (due to the half clk)
2. The framerate being 15 fps instead of 30 fps

The original code with fixed per mode register lists did use half pixel
clk, but this should be combined with using half for the VTS value too.

Using half VTS fixes the framerate issue, but this has the undesired
side-effect of change the exposure ctrl range (half the range, double
the amount of exposure per step).

Link: https://lore.kernel.org/r/20230604161406.69369-3-hdegoede@redhat.com
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/staging/media/atomisp/i2c/atomisp-ov2680.c
drivers/staging/media/atomisp/i2c/ov2680.h