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.verkuil@xxxxxxxxx> --- 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);
