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

Subject: media: atomisp: Clamp width to max 1920 pixels when in 
ATOMISP_RUN_MODE_PREVIEW
Author:  Hans de Goede <hdego...@redhat.com>
Date:    Mon Jun 19 12:52:08 2023 +0200

The pipeline firmware-binaries used in previed mode have
ia_css_binary_xinfo.output.max_width set to 1920.

This causes ia_css_binary_find() to fail when trying to set a higher
resolution resulting in the dump_stack() call in ia_css_binary_find()
triggering and resulting in the try_fmt() or set_fmt() IOCTL failing.

Fix this by clamping the width to max 1920 when in preview mode.

Link: https://lore.kernel.org/r/20230619105212.303653-3-hdego...@redhat.com

Signed-off-by: Hans de Goede <hdego...@redhat.com>
Reviewed-by: Andy Shevchenko <a...@kernel.org>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/staging/media/atomisp/pci/atomisp_cmd.c | 4 ++++
 1 file changed, 4 insertions(+)

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_cmd.c 
b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
index 0803b296e9ac..a5110636d6f5 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_cmd.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_cmd.c
@@ -3808,6 +3808,10 @@ int atomisp_try_fmt(struct atomisp_device *isp, struct 
v4l2_pix_format *f,
                        return -EINVAL;
        }
 
+       /* The preview pipeline does not support width > 1920 */
+       if (asd->run_mode->val == ATOMISP_RUN_MODE_PREVIEW)
+               f->width = min_t(u32, f->width, 1920);
+
        /*
         * atomisp_set_fmt() will set the sensor resolution to the requested
         * resolution + padding. Add padding here and remove it again after

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to