On 04/09/2017 12:56 PM, Mauro Carvalho Chehab wrote:
> The documentation incorrectly mentions MENU and INTEGER_MENU
> at struct v4l2_querymenu table as if they were flags. They're
> not: they're types.
> 
> Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>

Reviewed-by: Hans Verkuil <hans.verk...@cisco.com>

Thanks!

        Hans

> ---
>  Documentation/media/uapi/v4l/vidioc-queryctrl.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/media/uapi/v4l/vidioc-queryctrl.rst 
> b/Documentation/media/uapi/v4l/vidioc-queryctrl.rst
> index 82769de801b1..80842983eb12 100644
> --- a/Documentation/media/uapi/v4l/vidioc-queryctrl.rst
> +++ b/Documentation/media/uapi/v4l/vidioc-queryctrl.rst
> @@ -301,12 +301,12 @@ See also the examples in :ref:`control`.
>        - ``name``\ [32]
>        - Name of the menu item, a NUL-terminated ASCII string. This
>       information is intended for the user. This field is valid for
> -     ``V4L2_CTRL_FLAG_MENU`` type controls.
> +     ``V4L2_CTRL_TYPE_MENU`` type controls.
>      * -
>        - __s64
>        - ``value``
>        - Value of the integer menu item. This field is valid for
> -     ``V4L2_CTRL_FLAG_INTEGER_MENU`` type controls.
> +     ``V4L2_CTRL_TYPE_INTEGER_MENU`` type controls.
>      * - __u32
>        -
>        - ``reserved``
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to