media: atomisp: Drop second V4L2_PIX_FMT_UYVY atomisp_output_fmts[] entry
authorHans de Goede <hdegoede@redhat.com>
Sat, 16 Mar 2024 21:41:29 +0000 (21:41 +0000)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 26 Apr 2024 09:08:53 +0000 (10:08 +0100)
Some userspace apps (e.g. libcamera) do not like it if the enum_fmts
ioctl returns the same fmt twice. Drop the second "do not use"
V4L2_PIX_FMT_UYVY entry for parallel sensors to fix this.

The current atomisp code does not support anything other then raw bayer
sensors anyways, so dropping this is not an issue.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/staging/media/atomisp/pci/atomisp_ioctl.c

index fa6c9f0ea6eb7980f39c392a687a53d39f3aebc8..6009b7779f1388260e2f29b90fbb64e40f851ff4 100644 (file)
@@ -222,12 +222,6 @@ const struct atomisp_format_bridge atomisp_output_fmts[] = {
                .mbus_code = MEDIA_BUS_FMT_UYVY8_1X16,
                .sh_fmt = IA_CSS_FRAME_FORMAT_UYVY,
                .description = "UYVY, interleaved"
-       }, { /* This one is for parallel sensors! DO NOT USE! */
-               .pixelformat = V4L2_PIX_FMT_UYVY,
-               .depth = 16,
-               .mbus_code = MEDIA_BUS_FMT_UYVY8_2X8,
-               .sh_fmt = IA_CSS_FRAME_FORMAT_UYVY,
-               .description = "UYVY, interleaved"
        }, {
                .pixelformat = V4L2_PIX_FMT_SBGGR16,
                .depth = 16,