media: rename VFL_TYPE_GRABBER to _VIDEO

We currently have the following devnode types:

enum vfl_devnode_type {
        VFL_TYPE_GRABBER        = 0,
        VFL_TYPE_VBI,
        VFL_TYPE_RADIO,
        VFL_TYPE_SUBDEV,
        VFL_TYPE_SDR,
        VFL_TYPE_TOUCH,
        VFL_TYPE_MAX /* Shall be the last one */
};

They all make sense, except for the first: GRABBER really refers to /dev/videoX
devices, which can be capture, output or m2m, so 'grabber' doesn't even refer to
their function anymore.

Let's call a spade a spade and rename this to VFL_TYPE_VIDEO.

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
This commit is contained in:
Hans Verkuil 2020-02-03 12:41:09 +01:00 committed by Mauro Carvalho Chehab
parent cc62c74749
commit 238e4a5baa
6 changed files with 16 additions and 14 deletions

View File

@ -185,7 +185,7 @@ This will create the character device for you.
.. code-block:: c .. code-block:: c
err = video_register_device(vdev, VFL_TYPE_GRABBER, -1); err = video_register_device(vdev, VFL_TYPE_VIDEO, -1);
if (err) { if (err) {
video_device_release(vdev); /* or kfree(my_vdev); */ video_device_release(vdev); /* or kfree(my_vdev); */
return err; return err;
@ -201,7 +201,7 @@ types exist:
========================== ==================== ============================== ========================== ==================== ==============================
:c:type:`vfl_devnode_type` Device name Usage :c:type:`vfl_devnode_type` Device name Usage
========================== ==================== ============================== ========================== ==================== ==============================
``VFL_TYPE_GRABBER`` ``/dev/videoX`` for video input/output devices ``VFL_TYPE_VIDEO`` ``/dev/videoX`` for video input/output devices
``VFL_TYPE_VBI`` ``/dev/vbiX`` for vertical blank data (i.e. ``VFL_TYPE_VBI`` ``/dev/vbiX`` for vertical blank data (i.e.
closed captions, teletext) closed captions, teletext)
``VFL_TYPE_RADIO`` ``/dev/radioX`` for radio tuners ``VFL_TYPE_RADIO`` ``/dev/radioX`` for radio tuners

View File

@ -649,7 +649,7 @@ video_device注册
接下来你需要注册视频设备:这会为你创建一个字符设备。 接下来你需要注册视频设备:这会为你创建一个字符设备。
err = video_register_device(vdev, VFL_TYPE_GRABBER, -1); err = video_register_device(vdev, VFL_TYPE_VIDEO, -1);
if (err) { if (err) {
video_device_release(vdev); /* or kfree(my_vdev); */ video_device_release(vdev); /* or kfree(my_vdev); */
return err; return err;
@ -660,7 +660,7 @@ video_device注册
注册哪种设备是根据类型type参数。存在以下类型 注册哪种设备是根据类型type参数。存在以下类型
VFL_TYPE_GRABBER: 用于视频输入/输出设备的 videoX VFL_TYPE_VIDEO: 用于视频输入/输出设备的 videoX
VFL_TYPE_VBI: 用于垂直消隐数据的 vbiX (例如,隐藏式字幕,图文电视) VFL_TYPE_VBI: 用于垂直消隐数据的 vbiX (例如,隐藏式字幕,图文电视)
VFL_TYPE_RADIO: 用于广播调谐器的 radioX VFL_TYPE_RADIO: 用于广播调谐器的 radioX

View File

@ -542,13 +542,13 @@ static void determine_valid_ioctls(struct video_device *vdev)
V4L2_CAP_META_OUTPUT; V4L2_CAP_META_OUTPUT;
DECLARE_BITMAP(valid_ioctls, BASE_VIDIOC_PRIVATE); DECLARE_BITMAP(valid_ioctls, BASE_VIDIOC_PRIVATE);
const struct v4l2_ioctl_ops *ops = vdev->ioctl_ops; const struct v4l2_ioctl_ops *ops = vdev->ioctl_ops;
bool is_vid = vdev->vfl_type == VFL_TYPE_GRABBER && bool is_vid = vdev->vfl_type == VFL_TYPE_VIDEO &&
(vdev->device_caps & vid_caps); (vdev->device_caps & vid_caps);
bool is_vbi = vdev->vfl_type == VFL_TYPE_VBI; bool is_vbi = vdev->vfl_type == VFL_TYPE_VBI;
bool is_radio = vdev->vfl_type == VFL_TYPE_RADIO; bool is_radio = vdev->vfl_type == VFL_TYPE_RADIO;
bool is_sdr = vdev->vfl_type == VFL_TYPE_SDR; bool is_sdr = vdev->vfl_type == VFL_TYPE_SDR;
bool is_tch = vdev->vfl_type == VFL_TYPE_TOUCH; bool is_tch = vdev->vfl_type == VFL_TYPE_TOUCH;
bool is_meta = vdev->vfl_type == VFL_TYPE_GRABBER && bool is_meta = vdev->vfl_type == VFL_TYPE_VIDEO &&
(vdev->device_caps & meta_caps); (vdev->device_caps & meta_caps);
bool is_rx = vdev->vfl_dir != VFL_DIR_TX; bool is_rx = vdev->vfl_dir != VFL_DIR_TX;
bool is_tx = vdev->vfl_dir != VFL_DIR_RX; bool is_tx = vdev->vfl_dir != VFL_DIR_RX;
@ -783,7 +783,7 @@ static int video_register_media_controller(struct video_device *vdev)
vdev->entity.function = MEDIA_ENT_F_UNKNOWN; vdev->entity.function = MEDIA_ENT_F_UNKNOWN;
switch (vdev->vfl_type) { switch (vdev->vfl_type) {
case VFL_TYPE_GRABBER: case VFL_TYPE_VIDEO:
intf_type = MEDIA_INTF_T_V4L_VIDEO; intf_type = MEDIA_INTF_T_V4L_VIDEO;
vdev->entity.function = MEDIA_ENT_F_IO_V4L; vdev->entity.function = MEDIA_ENT_F_IO_V4L;
break; break;
@ -891,7 +891,7 @@ int __video_register_device(struct video_device *vdev,
/* Part 1: check device type */ /* Part 1: check device type */
switch (type) { switch (type) {
case VFL_TYPE_GRABBER: case VFL_TYPE_VIDEO:
name_base = "video"; name_base = "video";
break; break;
case VFL_TYPE_VBI: case VFL_TYPE_VBI:
@ -935,7 +935,7 @@ int __video_register_device(struct video_device *vdev,
* of 128-191 and just pick the first free minor there * of 128-191 and just pick the first free minor there
* (new style). */ * (new style). */
switch (type) { switch (type) {
case VFL_TYPE_GRABBER: case VFL_TYPE_VIDEO:
minor_offset = 0; minor_offset = 0;
minor_cnt = 64; minor_cnt = 64;
break; break;

View File

@ -941,12 +941,12 @@ static int check_fmt(struct file *file, enum v4l2_buf_type type)
V4L2_CAP_META_OUTPUT; V4L2_CAP_META_OUTPUT;
struct video_device *vfd = video_devdata(file); struct video_device *vfd = video_devdata(file);
const struct v4l2_ioctl_ops *ops = vfd->ioctl_ops; const struct v4l2_ioctl_ops *ops = vfd->ioctl_ops;
bool is_vid = vfd->vfl_type == VFL_TYPE_GRABBER && bool is_vid = vfd->vfl_type == VFL_TYPE_VIDEO &&
(vfd->device_caps & vid_caps); (vfd->device_caps & vid_caps);
bool is_vbi = vfd->vfl_type == VFL_TYPE_VBI; bool is_vbi = vfd->vfl_type == VFL_TYPE_VBI;
bool is_sdr = vfd->vfl_type == VFL_TYPE_SDR; bool is_sdr = vfd->vfl_type == VFL_TYPE_SDR;
bool is_tch = vfd->vfl_type == VFL_TYPE_TOUCH; bool is_tch = vfd->vfl_type == VFL_TYPE_TOUCH;
bool is_meta = vfd->vfl_type == VFL_TYPE_GRABBER && bool is_meta = vfd->vfl_type == VFL_TYPE_VIDEO &&
(vfd->device_caps & meta_caps); (vfd->device_caps & meta_caps);
bool is_rx = vfd->vfl_dir != VFL_DIR_TX; bool is_rx = vfd->vfl_dir != VFL_DIR_TX;
bool is_tx = vfd->vfl_dir != VFL_DIR_RX; bool is_tx = vfd->vfl_dir != VFL_DIR_RX;

View File

@ -24,7 +24,8 @@
/** /**
* enum vfl_devnode_type - type of V4L2 device node * enum vfl_devnode_type - type of V4L2 device node
* *
* @VFL_TYPE_GRABBER: for video input/output devices * @VFL_TYPE_VIDEO: for video input/output devices
* @VFL_TYPE_GRABBER: deprecated, same as VFL_TYPE_VIDEO
* @VFL_TYPE_VBI: for vertical blank data (i.e. closed captions, teletext) * @VFL_TYPE_VBI: for vertical blank data (i.e. closed captions, teletext)
* @VFL_TYPE_RADIO: for radio tuners * @VFL_TYPE_RADIO: for radio tuners
* @VFL_TYPE_SUBDEV: for V4L2 subdevices * @VFL_TYPE_SUBDEV: for V4L2 subdevices
@ -33,7 +34,8 @@
* @VFL_TYPE_MAX: number of VFL types, must always be last in the enum * @VFL_TYPE_MAX: number of VFL types, must always be last in the enum
*/ */
enum vfl_devnode_type { enum vfl_devnode_type {
VFL_TYPE_GRABBER = 0, VFL_TYPE_VIDEO,
VFL_TYPE_GRABBER = VFL_TYPE_VIDEO,
VFL_TYPE_VBI, VFL_TYPE_VBI,
VFL_TYPE_RADIO, VFL_TYPE_RADIO,
VFL_TYPE_SUBDEV, VFL_TYPE_SUBDEV,

View File

@ -879,7 +879,7 @@ static int skeleton_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
vdev->tvnorms = SKEL_TVNORMS; vdev->tvnorms = SKEL_TVNORMS;
video_set_drvdata(vdev, skel); video_set_drvdata(vdev, skel);
ret = video_register_device(vdev, VFL_TYPE_GRABBER, -1); ret = video_register_device(vdev, VFL_TYPE_VIDEO, -1);
if (ret) if (ret)
goto free_hdl; goto free_hdl;