The __v4l2_ctrl_modify_range is the unlocked variant, so the comment about
taking a lock is obviously wrong.

Signed-off-by: Hans Verkuil <hans.verk...@cisco.com>
---
diff --git a/include/media/v4l2-ctrls.h b/include/media/v4l2-ctrls.h
index 5253b5471897..33ce194a7481 100644
--- a/include/media/v4l2-ctrls.h
+++ b/include/media/v4l2-ctrls.h
@@ -760,9 +760,6 @@ void v4l2_ctrl_grab(struct v4l2_ctrl *ctrl, bool grabbed);
  *
  * An error is returned if one of the range arguments is invalid for this
  * control type.
- *
- * This function assumes that the control handler is not locked and will
- * take the lock itself.
  */
 int __v4l2_ctrl_modify_range(struct v4l2_ctrl *ctrl,
                             s64 min, s64 max, u64 step, s64 def);

Reply via email to