diff options
| author | Sakari Ailus <sakari.ailus@linux.intel.com> | 2025-06-19 12:23:48 +0300 |
|---|---|---|
| committer | Hans Verkuil <hverkuil@xs4all.nl> | 2025-06-30 08:47:41 +0200 |
| commit | bd227d0fc25c46c7fda43fe62cf7b04e9d0f0baa (patch) | |
| tree | 313310b69daf8344d627142c7945f9b51961717b /Documentation | |
| parent | 04f541cef2dba78b17c83ad8b5a0742012399530 (diff) | |
| download | net-bd227d0fc25c46c7fda43fe62cf7b04e9d0f0baa.tar.gz | |
media: Documentation: Document new v4l2_ctrl_handler_free() behaviour
v4l2_ctrl_handler_free() no longer resets the handler's error code.
Document it.
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Diffstat (limited to 'Documentation')
| -rw-r--r-- | Documentation/driver-api/media/v4l2-controls.rst | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/Documentation/driver-api/media/v4l2-controls.rst b/Documentation/driver-api/media/v4l2-controls.rst index b2e91804829bde..fc04907589ab04 100644 --- a/Documentation/driver-api/media/v4l2-controls.rst +++ b/Documentation/driver-api/media/v4l2-controls.rst @@ -110,6 +110,7 @@ For sub-device drivers: v4l2_ctrl_handler_free(&foo->ctrl_handler); +:c:func:`v4l2_ctrl_handler_free` does not touch the handler's ``error`` field. 2) Add controls: @@ -191,12 +192,8 @@ These functions are typically called right after the V4L2_CID_TEST_PATTERN, ARRAY_SIZE(test_pattern) - 1, 0, 0, test_pattern); ... - if (foo->ctrl_handler.error) { - int err = foo->ctrl_handler.error; - - v4l2_ctrl_handler_free(&foo->ctrl_handler); - return err; - } + if (foo->ctrl_handler.error) + return v4l2_ctrl_handler_free(&foo->ctrl_handler); The :c:func:`v4l2_ctrl_new_std` function returns the v4l2_ctrl pointer to the new control, but if you do not need to access the pointer outside the |