On Wed Oct 8 12:55:18 2025 +0300, Laurent Pinchart wrote:
> Commit cbd9463da1b1 ("media: v4l2-mem2mem: Avoid calling .device_run in
> v4l2_m2m_job_finish") deferred calls to .device_run() to a work queue to
> avoid recursive calls when a job is finished right away from
> .device_run(). It failed to update the v4l2_m2m_job_finish()
> documentation that still states the function must not be called from
> .device_run(). Fix it.
>
> Fixes: cbd9463da1b1 ("media: v4l2-mem2mem: Avoid calling .device_run in
> v4l2_m2m_job_finish")
> Cc: [email protected]
> Signed-off-by: Laurent Pinchart <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
include/media/v4l2-mem2mem.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
---
diff --git a/include/media/v4l2-mem2mem.h b/include/media/v4l2-mem2mem.h
index 09c6164577cc..500f81f399df 100644
--- a/include/media/v4l2-mem2mem.h
+++ b/include/media/v4l2-mem2mem.h
@@ -192,8 +192,7 @@ void v4l2_m2m_try_schedule(struct v4l2_m2m_ctx *m2m_ctx);
* other instances to take control of the device.
*
* This function has to be called only after &v4l2_m2m_ops->device_run
- * callback has been called on the driver. To prevent recursion, it should
- * not be called directly from the &v4l2_m2m_ops->device_run callback though.
+ * callback has been called on the driver.
*/
void v4l2_m2m_job_finish(struct v4l2_m2m_dev *m2m_dev,
struct v4l2_m2m_ctx *m2m_ctx);
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
