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

Subject: media: atomisp: set default mode
Author:  Mauro Carvalho Chehab <[email protected]>
Date:    Fri Oct 29 08:09:39 2021 +0100

Without setting a default mode at open(), applications that
don't call VIDIOC_SET_PARM with a custom atomisp parameters
won't work, as the pipeline won't be set:

        atomisp-isp2 0000:00:03.0: can't create streams
        atomisp-isp2 0000:00:03.0: __get_frame_info 1600x1200 (padded to 0) 
returned -22

So, as an step to allow generic apps to use this driver, put
the device's run_mode in preview after open.

After this patch, using v4l2grab starts to work:

        $ v4l2grab -D -f 'NV12' -x 1600 -y 1200 -d /dev/video2 -u
        $ nvt/raw2pnm -x1600 -y1200 -f NV12 out017.raw out017.pnm
        $ feh out017.pnm

Signed-off-by: Mauro Carvalho Chehab <[email protected]>

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

---

diff --git a/drivers/staging/media/atomisp/pci/atomisp_fops.c 
b/drivers/staging/media/atomisp/pci/atomisp_fops.c
index 72cbdce2142a..7df982c80b1a 100644
--- a/drivers/staging/media/atomisp/pci/atomisp_fops.c
+++ b/drivers/staging/media/atomisp/pci/atomisp_fops.c
@@ -893,6 +893,11 @@ done:
        else
                pipe->users++;
        rt_mutex_unlock(&isp->mutex);
+
+       /* Ensure that a mode is set */
+       if (asd)
+               v4l2_ctrl_s_ctrl(asd->run_mode, ATOMISP_RUN_MODE_PREVIEW);
+
        return 0;
 
 css_error:

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to