media: atomisp: Change atomisp_enum_framesizes() too small cut off from 2/3th to...

Change atomisp_enum_framesizes() cut off for too small resolutions
from 2/3th to 5/8th this results in more resolutions being available
with some sensors.

E.g. this allows using 800x600 with a 1280x960 sensor.

Link: https://lore.kernel.org/r/20230619105212.303653-4-hdegoede@redhat.com
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Andy Shevchenko <andy@kernel.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>

diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
index d2174156573a536560f63b1dd3a7cd8e6ace0e52..047b9fb075d046b9f257885b51ee94f3f9c6a549 100644 (file)
--- a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
@@ -729,11 +729,11 @@ static int atomisp_enum_framesizes_crop_inner(struct atomisp_device *isp,
                        continue;
 
                /*
-                * Skip sizes where width and height are less then 2/3th of the
+                * Skip sizes where width and height are less then 5/8th of the
                 * sensor size to avoid sizes with a too small field of view.
                 */
-               if (frame_sizes[i].width < (active->width * 2 / 3) &&
-                   frame_sizes[i].height < (active->height * 2 / 3))
+               if (frame_sizes[i].width < (active->width * 5 / 8) &&
+                   frame_sizes[i].height < (active->height * 5 / 8))
                        continue;
 
                if (*valid_sizes == fsize->index) {