This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: atomisp: Drop second V4L2_PIX_FMT_UYVY atomisp_output_fmts[] 
entry
Author:  Hans de Goede <hdego...@redhat.com>
Date:    Sat Mar 16 21:41:29 2024 +0000

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 <hdego...@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/staging/media/atomisp/pci/atomisp_ioctl.c | 6 ------
 1 file changed, 6 deletions(-)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c 
b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
index fa6c9f0ea6eb..6009b7779f13 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
@@ -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,

Reply via email to