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

Subject: media: imx-jpeg: Drop initial source change event if capture has been 
setup
Author:  Ming Qian <ming.q...@nxp.com>
Date:    Thu Oct 19 16:36:36 2023 +0800

In section 4.5.1.5. Initialization, the step 4 may be skipped and
continue with the Capture Setup sequence, so if the capture has been
setup, there is no need to trigger the initial source change event, just
start decoding, and follow the dynamic resolution change flow if the
configured values do not match those parsed by the decoder.

And it won't fail the gstreamer pipeline.

Fixes: b833b178498d ("media: imx-jpeg: notify source chagne event when the 
first picture parsed")
Signed-off-by: Ming Qian <ming.q...@nxp.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c | 3 +++
 1 file changed, 3 insertions(+)

---

diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c 
b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
index de07d2716fc3..1d8913813037 100644
--- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
+++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
@@ -1634,6 +1634,9 @@ static int mxc_jpeg_start_streaming(struct vb2_queue *q, 
unsigned int count)
        dev_dbg(ctx->mxc_jpeg->dev, "Start streaming ctx=%p", ctx);
        q_data->sequence = 0;
 
+       if (V4L2_TYPE_IS_CAPTURE(q->type))
+               ctx->need_initial_source_change_evt = false;
+
        ret = pm_runtime_resume_and_get(ctx->mxc_jpeg->dev);
        if (ret < 0) {
                dev_err(ctx->mxc_jpeg->dev, "Failed to power up jpeg\n");

Reply via email to