media: atomisp: ov2680: Use defines for fps, lines-per-frame and skip-frames
authorHans de Goede <hdegoede@redhat.com>
Sun, 8 Jan 2023 14:06:26 +0000 (15:06 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Wed, 8 Feb 2023 07:24:33 +0000 (08:24 +0100)
commit3406639ee2ad5bf736b9b38091bf108ac600327a
tree2ad777c2cf6c233acbeff1204ae67fd10e9550fd
parent0611888592df86c3f348146dc26d4b00f850e194
media: atomisp: ov2680: Use defines for fps, lines-per-frame and skip-frames

The fps, lines-per-frame and skip-frames values are the same for all
resolutions, use defines for these.

The ov2680_res_preview[] incorrectly sets fps to 60 for some low-res
modes, this is incorrect with the current fixed (resolution independent)
lines-per-frame value.

Note this not drop the now no longer used fps, lines-per-frame and
skip-frames struct ov2680_resolution members. The entire struct is going
away in the next patches so that would just cause unnecessary changes.

Reviewed-by: Andy Shevchenko <andy@kernel.org>
Signed-off-by: Hans de Goede <hdegoede@redhat.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