Commit 270c0024 authored by Luca Ceresoli's avatar Luca Ceresoli Committed by Mauro Carvalho Chehab

media: docs: v4l2-controls: convert code blocks to C

All these code blocks contain C code, enable C formatting for a nicer
reading.
Signed-off-by: default avatarLuca Ceresoli <luca@lucaceresoli.net>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent 6b623dbf
...@@ -54,7 +54,7 @@ Basic usage for V4L2 and sub-device drivers ...@@ -54,7 +54,7 @@ Basic usage for V4L2 and sub-device drivers
For V4L2 drivers: For V4L2 drivers:
.. code-block:: none .. code-block:: c
struct foo_dev { struct foo_dev {
... ...
...@@ -66,7 +66,7 @@ For V4L2 drivers: ...@@ -66,7 +66,7 @@ For V4L2 drivers:
For sub-device drivers: For sub-device drivers:
.. code-block:: none .. code-block:: c
struct foo_dev { struct foo_dev {
... ...
...@@ -78,7 +78,7 @@ For sub-device drivers: ...@@ -78,7 +78,7 @@ For sub-device drivers:
1.2) Initialize the handler: 1.2) Initialize the handler:
.. code-block:: none .. code-block:: c
v4l2_ctrl_handler_init(&foo->ctrl_handler, nr_of_controls); v4l2_ctrl_handler_init(&foo->ctrl_handler, nr_of_controls);
...@@ -90,7 +90,7 @@ information. It is a hint only. ...@@ -90,7 +90,7 @@ information. It is a hint only.
For V4L2 drivers: For V4L2 drivers:
.. code-block:: none .. code-block:: c
foo->v4l2_dev.ctrl_handler = &foo->ctrl_handler; foo->v4l2_dev.ctrl_handler = &foo->ctrl_handler;
...@@ -101,13 +101,13 @@ Those are now no longer needed. ...@@ -101,13 +101,13 @@ Those are now no longer needed.
For sub-device drivers: For sub-device drivers:
.. code-block:: none .. code-block:: c
foo->sd.ctrl_handler = &foo->ctrl_handler; foo->sd.ctrl_handler = &foo->ctrl_handler;
1.4) Clean up the handler at the end: 1.4) Clean up the handler at the end:
.. code-block:: none .. code-block:: c
v4l2_ctrl_handler_free(&foo->ctrl_handler); v4l2_ctrl_handler_free(&foo->ctrl_handler);
...@@ -116,7 +116,7 @@ For sub-device drivers: ...@@ -116,7 +116,7 @@ For sub-device drivers:
You add non-menu controls by calling :c:func:`v4l2_ctrl_new_std`: You add non-menu controls by calling :c:func:`v4l2_ctrl_new_std`:
.. code-block:: none .. code-block:: c
struct v4l2_ctrl *v4l2_ctrl_new_std(struct v4l2_ctrl_handler *hdl, struct v4l2_ctrl *v4l2_ctrl_new_std(struct v4l2_ctrl_handler *hdl,
const struct v4l2_ctrl_ops *ops, const struct v4l2_ctrl_ops *ops,
...@@ -125,7 +125,7 @@ You add non-menu controls by calling :c:func:`v4l2_ctrl_new_std`: ...@@ -125,7 +125,7 @@ You add non-menu controls by calling :c:func:`v4l2_ctrl_new_std`:
Menu and integer menu controls are added by calling Menu and integer menu controls are added by calling
:c:func:`v4l2_ctrl_new_std_menu`: :c:func:`v4l2_ctrl_new_std_menu`:
.. code-block:: none .. code-block:: c
struct v4l2_ctrl *v4l2_ctrl_new_std_menu(struct v4l2_ctrl_handler *hdl, struct v4l2_ctrl *v4l2_ctrl_new_std_menu(struct v4l2_ctrl_handler *hdl,
const struct v4l2_ctrl_ops *ops, const struct v4l2_ctrl_ops *ops,
...@@ -134,7 +134,7 @@ Menu and integer menu controls are added by calling ...@@ -134,7 +134,7 @@ Menu and integer menu controls are added by calling
Menu controls with a driver specific menu are added by calling Menu controls with a driver specific menu are added by calling
:c:func:`v4l2_ctrl_new_std_menu_items`: :c:func:`v4l2_ctrl_new_std_menu_items`:
.. code-block:: none .. code-block:: c
struct v4l2_ctrl *v4l2_ctrl_new_std_menu_items( struct v4l2_ctrl *v4l2_ctrl_new_std_menu_items(
struct v4l2_ctrl_handler *hdl, struct v4l2_ctrl_handler *hdl,
...@@ -144,7 +144,7 @@ Menu controls with a driver specific menu are added by calling ...@@ -144,7 +144,7 @@ Menu controls with a driver specific menu are added by calling
Integer menu controls with a driver specific menu can be added by calling Integer menu controls with a driver specific menu can be added by calling
:c:func:`v4l2_ctrl_new_int_menu`: :c:func:`v4l2_ctrl_new_int_menu`:
.. code-block:: none .. code-block:: c
struct v4l2_ctrl *v4l2_ctrl_new_int_menu(struct v4l2_ctrl_handler *hdl, struct v4l2_ctrl *v4l2_ctrl_new_int_menu(struct v4l2_ctrl_handler *hdl,
const struct v4l2_ctrl_ops *ops, const struct v4l2_ctrl_ops *ops,
...@@ -153,7 +153,7 @@ Integer menu controls with a driver specific menu can be added by calling ...@@ -153,7 +153,7 @@ Integer menu controls with a driver specific menu can be added by calling
These functions are typically called right after the These functions are typically called right after the
:c:func:`v4l2_ctrl_handler_init`: :c:func:`v4l2_ctrl_handler_init`:
.. code-block:: none .. code-block:: c
static const s64 exp_bias_qmenu[] = { static const s64 exp_bias_qmenu[] = {
-2, -1, 0, 1, 2 -2, -1, 0, 1, 2
...@@ -232,7 +232,7 @@ a bit faster that way. ...@@ -232,7 +232,7 @@ a bit faster that way.
3) Optionally force initial control setup: 3) Optionally force initial control setup:
.. code-block:: none .. code-block:: c
v4l2_ctrl_handler_setup(&foo->ctrl_handler); v4l2_ctrl_handler_setup(&foo->ctrl_handler);
...@@ -243,7 +243,7 @@ the hardware are in sync. ...@@ -243,7 +243,7 @@ the hardware are in sync.
4) Finally: implement the :c:type:`v4l2_ctrl_ops` 4) Finally: implement the :c:type:`v4l2_ctrl_ops`
.. code-block:: none .. code-block:: c
static const struct v4l2_ctrl_ops foo_ctrl_ops = { static const struct v4l2_ctrl_ops foo_ctrl_ops = {
.s_ctrl = foo_s_ctrl, .s_ctrl = foo_s_ctrl,
...@@ -251,7 +251,7 @@ the hardware are in sync. ...@@ -251,7 +251,7 @@ the hardware are in sync.
Usually all you need is s_ctrl: Usually all you need is s_ctrl:
.. code-block:: none .. code-block:: c
static int foo_s_ctrl(struct v4l2_ctrl *ctrl) static int foo_s_ctrl(struct v4l2_ctrl *ctrl)
{ {
...@@ -304,7 +304,7 @@ Accessing Control Values ...@@ -304,7 +304,7 @@ Accessing Control Values
The following union is used inside the control framework to access control The following union is used inside the control framework to access control
values: values:
.. code-block:: none .. code-block:: c
union v4l2_ctrl_ptr { union v4l2_ctrl_ptr {
s32 *p_s32; s32 *p_s32;
...@@ -316,7 +316,7 @@ values: ...@@ -316,7 +316,7 @@ values:
The v4l2_ctrl struct contains these fields that can be used to access both The v4l2_ctrl struct contains these fields that can be used to access both
current and new values: current and new values:
.. code-block:: none .. code-block:: c
s32 val; s32 val;
struct { struct {
...@@ -329,7 +329,7 @@ current and new values: ...@@ -329,7 +329,7 @@ current and new values:
If the control has a simple s32 type type, then: If the control has a simple s32 type type, then:
.. code-block:: none .. code-block:: c
&ctrl->val == ctrl->p_new.p_s32 &ctrl->val == ctrl->p_new.p_s32
&ctrl->cur.val == ctrl->p_cur.p_s32 &ctrl->cur.val == ctrl->p_cur.p_s32
...@@ -353,7 +353,7 @@ exception is for controls that return a volatile register such as a signal ...@@ -353,7 +353,7 @@ exception is for controls that return a volatile register such as a signal
strength read-out that changes continuously. In that case you will need to strength read-out that changes continuously. In that case you will need to
implement g_volatile_ctrl like this: implement g_volatile_ctrl like this:
.. code-block:: none .. code-block:: c
static int foo_g_volatile_ctrl(struct v4l2_ctrl *ctrl) static int foo_g_volatile_ctrl(struct v4l2_ctrl *ctrl)
{ {
...@@ -371,7 +371,7 @@ changes. ...@@ -371,7 +371,7 @@ changes.
To mark a control as volatile you have to set V4L2_CTRL_FLAG_VOLATILE: To mark a control as volatile you have to set V4L2_CTRL_FLAG_VOLATILE:
.. code-block:: none .. code-block:: c
ctrl = v4l2_ctrl_new_std(&sd->ctrl_handler, ...); ctrl = v4l2_ctrl_new_std(&sd->ctrl_handler, ...);
if (ctrl) if (ctrl)
...@@ -392,7 +392,7 @@ not to introduce deadlocks. ...@@ -392,7 +392,7 @@ not to introduce deadlocks.
Outside of the control ops you have to go through to helper functions to get Outside of the control ops you have to go through to helper functions to get
or set a single control value safely in your driver: or set a single control value safely in your driver:
.. code-block:: none .. code-block:: c
s32 v4l2_ctrl_g_ctrl(struct v4l2_ctrl *ctrl); s32 v4l2_ctrl_g_ctrl(struct v4l2_ctrl *ctrl);
int v4l2_ctrl_s_ctrl(struct v4l2_ctrl *ctrl, s32 val); int v4l2_ctrl_s_ctrl(struct v4l2_ctrl *ctrl, s32 val);
...@@ -403,7 +403,7 @@ will result in a deadlock since these helpers lock the handler as well. ...@@ -403,7 +403,7 @@ will result in a deadlock since these helpers lock the handler as well.
You can also take the handler lock yourself: You can also take the handler lock yourself:
.. code-block:: none .. code-block:: c
mutex_lock(&state->ctrl_handler.lock); mutex_lock(&state->ctrl_handler.lock);
pr_info("String value is '%s'\n", ctrl1->p_cur.p_char); pr_info("String value is '%s'\n", ctrl1->p_cur.p_char);
...@@ -416,7 +416,7 @@ Menu Controls ...@@ -416,7 +416,7 @@ Menu Controls
The v4l2_ctrl struct contains this union: The v4l2_ctrl struct contains this union:
.. code-block:: none .. code-block:: c
union { union {
u32 step; u32 step;
...@@ -444,7 +444,7 @@ Custom Controls ...@@ -444,7 +444,7 @@ Custom Controls
Driver specific controls can be created using v4l2_ctrl_new_custom(): Driver specific controls can be created using v4l2_ctrl_new_custom():
.. code-block:: none .. code-block:: c
static const struct v4l2_ctrl_config ctrl_filter = { static const struct v4l2_ctrl_config ctrl_filter = {
.ops = &ctrl_custom_ops, .ops = &ctrl_custom_ops,
...@@ -498,7 +498,7 @@ By default all controls are independent from the others. But in more ...@@ -498,7 +498,7 @@ By default all controls are independent from the others. But in more
complex scenarios you can get dependencies from one control to another. complex scenarios you can get dependencies from one control to another.
In that case you need to 'cluster' them: In that case you need to 'cluster' them:
.. code-block:: none .. code-block:: c
struct foo { struct foo {
struct v4l2_ctrl_handler ctrl_handler; struct v4l2_ctrl_handler ctrl_handler;
...@@ -522,7 +522,7 @@ composite control. Similar to how a 'struct' works in C. ...@@ -522,7 +522,7 @@ composite control. Similar to how a 'struct' works in C.
So when s_ctrl is called with V4L2_CID_AUDIO_VOLUME as argument, you should set So when s_ctrl is called with V4L2_CID_AUDIO_VOLUME as argument, you should set
all two controls belonging to the audio_cluster: all two controls belonging to the audio_cluster:
.. code-block:: none .. code-block:: c
static int foo_s_ctrl(struct v4l2_ctrl *ctrl) static int foo_s_ctrl(struct v4l2_ctrl *ctrl)
{ {
...@@ -544,7 +544,7 @@ all two controls belonging to the audio_cluster: ...@@ -544,7 +544,7 @@ all two controls belonging to the audio_cluster:
In the example above the following are equivalent for the VOLUME case: In the example above the following are equivalent for the VOLUME case:
.. code-block:: none .. code-block:: c
ctrl == ctrl->cluster[AUDIO_CL_VOLUME] == state->audio_cluster[AUDIO_CL_VOLUME] ctrl == ctrl->cluster[AUDIO_CL_VOLUME] == state->audio_cluster[AUDIO_CL_VOLUME]
ctrl->cluster[AUDIO_CL_MUTE] == state->audio_cluster[AUDIO_CL_MUTE] ctrl->cluster[AUDIO_CL_MUTE] == state->audio_cluster[AUDIO_CL_MUTE]
...@@ -552,7 +552,7 @@ In the example above the following are equivalent for the VOLUME case: ...@@ -552,7 +552,7 @@ In the example above the following are equivalent for the VOLUME case:
In practice using cluster arrays like this becomes very tiresome. So instead In practice using cluster arrays like this becomes very tiresome. So instead
the following equivalent method is used: the following equivalent method is used:
.. code-block:: none .. code-block:: c
struct { struct {
/* audio cluster */ /* audio cluster */
...@@ -564,7 +564,7 @@ The anonymous struct is used to clearly 'cluster' these two control pointers, ...@@ -564,7 +564,7 @@ The anonymous struct is used to clearly 'cluster' these two control pointers,
but it serves no other purpose. The effect is the same as creating an but it serves no other purpose. The effect is the same as creating an
array with two control pointers. So you can just do: array with two control pointers. So you can just do:
.. code-block:: none .. code-block:: c
state->volume = v4l2_ctrl_new_std(&state->ctrl_handler, ...); state->volume = v4l2_ctrl_new_std(&state->ctrl_handler, ...);
state->mute = v4l2_ctrl_new_std(&state->ctrl_handler, ...); state->mute = v4l2_ctrl_new_std(&state->ctrl_handler, ...);
...@@ -620,7 +620,7 @@ changing that control affects the control flags of the manual controls. ...@@ -620,7 +620,7 @@ changing that control affects the control flags of the manual controls.
In order to simplify this a special variation of v4l2_ctrl_cluster was In order to simplify this a special variation of v4l2_ctrl_cluster was
introduced: introduced:
.. code-block:: none .. code-block:: c
void v4l2_ctrl_auto_cluster(unsigned ncontrols, struct v4l2_ctrl **controls, void v4l2_ctrl_auto_cluster(unsigned ncontrols, struct v4l2_ctrl **controls,
u8 manual_val, bool set_volatile); u8 manual_val, bool set_volatile);
...@@ -675,7 +675,7 @@ of another handler (e.g. for a video device node), then you should first add ...@@ -675,7 +675,7 @@ of another handler (e.g. for a video device node), then you should first add
the controls to the first handler, add the other controls to the second the controls to the first handler, add the other controls to the second
handler and finally add the first handler to the second. For example: handler and finally add the first handler to the second. For example:
.. code-block:: none .. code-block:: c
v4l2_ctrl_new_std(&radio_ctrl_handler, &radio_ops, V4L2_CID_AUDIO_VOLUME, ...); v4l2_ctrl_new_std(&radio_ctrl_handler, &radio_ops, V4L2_CID_AUDIO_VOLUME, ...);
v4l2_ctrl_new_std(&radio_ctrl_handler, &radio_ops, V4L2_CID_AUDIO_MUTE, ...); v4l2_ctrl_new_std(&radio_ctrl_handler, &radio_ops, V4L2_CID_AUDIO_MUTE, ...);
...@@ -689,7 +689,7 @@ all controls. ...@@ -689,7 +689,7 @@ all controls.
Or you can add specific controls to a handler: Or you can add specific controls to a handler:
.. code-block:: none .. code-block:: c
volume = v4l2_ctrl_new_std(&video_ctrl_handler, &ops, V4L2_CID_AUDIO_VOLUME, ...); volume = v4l2_ctrl_new_std(&video_ctrl_handler, &ops, V4L2_CID_AUDIO_VOLUME, ...);
v4l2_ctrl_new_std(&video_ctrl_handler, &ops, V4L2_CID_BRIGHTNESS, ...); v4l2_ctrl_new_std(&video_ctrl_handler, &ops, V4L2_CID_BRIGHTNESS, ...);
...@@ -698,7 +698,7 @@ Or you can add specific controls to a handler: ...@@ -698,7 +698,7 @@ Or you can add specific controls to a handler:
What you should not do is make two identical controls for two handlers. What you should not do is make two identical controls for two handlers.
For example: For example:
.. code-block:: none .. code-block:: c
v4l2_ctrl_new_std(&radio_ctrl_handler, &radio_ops, V4L2_CID_AUDIO_MUTE, ...); v4l2_ctrl_new_std(&radio_ctrl_handler, &radio_ops, V4L2_CID_AUDIO_MUTE, ...);
v4l2_ctrl_new_std(&video_ctrl_handler, &video_ops, V4L2_CID_AUDIO_MUTE, ...); v4l2_ctrl_new_std(&video_ctrl_handler, &video_ops, V4L2_CID_AUDIO_MUTE, ...);
...@@ -719,7 +719,7 @@ not own. For example, if you have to find a volume control from a subdev. ...@@ -719,7 +719,7 @@ not own. For example, if you have to find a volume control from a subdev.
You can do that by calling v4l2_ctrl_find: You can do that by calling v4l2_ctrl_find:
.. code-block:: none .. code-block:: c
struct v4l2_ctrl *volume; struct v4l2_ctrl *volume;
...@@ -728,7 +728,7 @@ You can do that by calling v4l2_ctrl_find: ...@@ -728,7 +728,7 @@ You can do that by calling v4l2_ctrl_find:
Since v4l2_ctrl_find will lock the handler you have to be careful where you Since v4l2_ctrl_find will lock the handler you have to be careful where you
use it. For example, this is not a good idea: use it. For example, this is not a good idea:
.. code-block:: none .. code-block:: c
struct v4l2_ctrl_handler ctrl_handler; struct v4l2_ctrl_handler ctrl_handler;
...@@ -737,7 +737,7 @@ use it. For example, this is not a good idea: ...@@ -737,7 +737,7 @@ use it. For example, this is not a good idea:
...and in video_ops.s_ctrl: ...and in video_ops.s_ctrl:
.. code-block:: none .. code-block:: c
case V4L2_CID_BRIGHTNESS: case V4L2_CID_BRIGHTNESS:
contrast = v4l2_find_ctrl(&ctrl_handler, V4L2_CID_CONTRAST); contrast = v4l2_find_ctrl(&ctrl_handler, V4L2_CID_CONTRAST);
...@@ -759,7 +759,7 @@ not when it is used in consumer-level hardware. In that case you want to keep ...@@ -759,7 +759,7 @@ not when it is used in consumer-level hardware. In that case you want to keep
those low-level controls local to the subdev. You can do this by simply those low-level controls local to the subdev. You can do this by simply
setting the 'is_private' flag of the control to 1: setting the 'is_private' flag of the control to 1:
.. code-block:: none .. code-block:: c
static const struct v4l2_ctrl_config ctrl_private = { static const struct v4l2_ctrl_config ctrl_private = {
.ops = &ctrl_custom_ops, .ops = &ctrl_custom_ops,
...@@ -796,7 +796,7 @@ Sometimes the platform or bridge driver needs to be notified when a control ...@@ -796,7 +796,7 @@ Sometimes the platform or bridge driver needs to be notified when a control
from a sub-device driver changes. You can set a notify callback by calling from a sub-device driver changes. You can set a notify callback by calling
this function: this function:
.. code-block:: none .. code-block:: c
void v4l2_ctrl_notify(struct v4l2_ctrl *ctrl, void v4l2_ctrl_notify(struct v4l2_ctrl *ctrl,
void (*notify)(struct v4l2_ctrl *ctrl, void *priv), void *priv); void (*notify)(struct v4l2_ctrl *ctrl, void *priv), void *priv);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment