Hi, thanks for the patch, one comment below:
>Vaibhav Hiremath ([email protected]) wrote: > >Signed-off-by: Vaibhav Hiremath <[email protected]> >--- > drivers/media/video/v4l2-mem2mem.c | 40 ++++++++++++++--------------------- > 1 files changed, 16 insertions(+), 24 deletions(-) [...] >@@ -319,10 +317,9 @@ static void v4l2_m2m_try_schedule(struct v4l2_m2m_ctx >*m2m_ctx) > return; > } > >- if (!(m2m_ctx->job_flags & TRANS_QUEUED)) { >- list_add_tail(&m2m_ctx->queue, &m2m_dev->jobqueue); >- m2m_ctx->job_flags |= TRANS_QUEUED; >- } >+ list_add_tail(&m2m_ctx->queue, &m2m_dev->jobqueue); >+ m2m_ctx->job_flags |= TRANS_QUEUED; >+ > spin_unlock_irqrestore(&m2m_dev->job_spinlock, flags_job); > > v4l2_m2m_try_run(m2m_dev); Nice catch! This wasn't the case before, but as v3 is now holding the job_spinlock for the whole time, the check can be safely removed. [...] Acked-by: Pawel Osciak <[email protected]> Best regards -- Pawel Osciak Linux Platform Group Samsung Poland R&D Center -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
