Which device is registered depends on the type argument. The following
 types exist:
 
-- ``VFL_TYPE_GRABBER``: ``/dev/videoX`` for video input/output devices
-- ``VFL_TYPE_VBI``: ``/dev/vbiX`` for vertical blank data (i.e. closed captions, teletext)
-- ``VFL_TYPE_RADIO``: ``/dev/radioX`` for radio tuners
-- ``VFL_TYPE_SDR``: ``/dev/swradioX`` for Software Defined Radio tuners
-- ``VFL_TYPE_TOUCH``: ``/dev/v4l-touchX`` for touch sensors
+========================== ====================         ==============================
+:c:type:`vfl_devnode_type` Device name          Usage
+========================== ====================         ==============================
+``VFL_TYPE_GRABBER``       ``/dev/videoX``       for video input/output devices
+``VFL_TYPE_VBI``           ``/dev/vbiX``         for vertical blank data (i.e.
+                                                closed captions, teletext)
+``VFL_TYPE_RADIO``         ``/dev/radioX``       for radio tuners
+``VFL_TYPE_SUBDEV``        ``/dev/v4l-subdevX``  for V4L2 subdevices
+``VFL_TYPE_SDR``           ``/dev/swradioX``     for Software Defined Radio
+                                                (SDR) tuners
+``VFL_TYPE_TOUCH``         ``/dev/v4l-touchX``   for touch sensors
+========================== ====================         ==============================
 
 The last argument gives you a certain amount of control over the device
 device node number used (i.e. the X in ``videoX``). Normally you will pass -1
 
 
        strcpy(cap->driver, "cx88_blackbird");
        sprintf(cap->bus_info, "PCI:%s", pci_name(dev->pci));
-       cx88_querycap(file, core, cap);
-       return 0;
+       return cx88_querycap(file, core, cap);
 }
 
 static int vidioc_enum_fmt_vid_cap(struct file *file, void  *priv,
 
        return 0;
 }
 
-void cx88_querycap(struct file *file, struct cx88_core *core,
-                  struct v4l2_capability *cap)
+int cx88_querycap(struct file *file, struct cx88_core *core,
+                 struct v4l2_capability *cap)
 {
        struct video_device *vdev = video_devdata(file);
 
        case VFL_TYPE_VBI:
                cap->device_caps |= V4L2_CAP_VBI_CAPTURE;
                break;
+       default:
+               return -EINVAL;
        }
        cap->capabilities = cap->device_caps | V4L2_CAP_VIDEO_CAPTURE |
                V4L2_CAP_VBI_CAPTURE | V4L2_CAP_DEVICE_CAPS;
        if (core->board.radio.type == CX88_RADIO)
                cap->capabilities |= V4L2_CAP_RADIO;
+       return 0;
 }
 EXPORT_SYMBOL(cx88_querycap);
 
 
        strcpy(cap->driver, "cx8800");
        sprintf(cap->bus_info, "PCI:%s", pci_name(dev->pci));
-       cx88_querycap(file, core, cap);
-       return 0;
+       return cx88_querycap(file, core, cap);
 }
 
 static int vidioc_enum_fmt_vid_cap(struct file *file, void  *priv,
 
 int cx88_enum_input(struct cx88_core *core, struct v4l2_input *i);
 int cx88_set_freq(struct cx88_core  *core, const struct v4l2_frequency *f);
 int cx88_video_mux(struct cx88_core *core, unsigned int input);
-void cx88_querycap(struct file *file, struct cx88_core *core,
-                  struct v4l2_capability *cap);
+int cx88_querycap(struct file *file, struct cx88_core *core,
+                 struct v4l2_capability *cap);
 
 #endif
 
        case VFL_TYPE_VBI:
                cap->device_caps |= vbi_caps;
                break;
+       default:
+               return -EINVAL;
        }
        cap->capabilities = radio_caps | video_caps | vbi_caps |
                cap->device_caps | V4L2_CAP_DEVICE_CAPS;
 
        case VFL_TYPE_RADIO:
                radio = 1;
                break;
+       default:
+               return -EINVAL;
        }
 
        cx231xx_videodbg("open dev=%s type=%s users=%d\n",
 
        case VFL_TYPE_RADIO:
                cap->device_caps = V4L2_CAP_RADIO;
                break;
+       default:
+               return -EINVAL;
        }
        cap->device_caps |= V4L2_CAP_TUNER | V4L2_CAP_READWRITE;
        return 0;
 
        case VFL_TYPE_RADIO:
                radio = 1;
                break;
+       default:
+               return -EINVAL;
        }
 
        /* If more than one user, mutex should be added */
 
 
 #ifdef CONFIG_VIDEO_FIXED_MINOR_RANGES
 /* Return the bitmap corresponding to vfl_type. */
-static inline unsigned long *devnode_bits(int vfl_type)
+static inline unsigned long *devnode_bits(enum vfl_devnode_type vfl_type)
 {
        /* Any types not assigned to fixed minor ranges must be mapped to
           one single bitmap for the purposes of finding a free node number
 }
 #else
 /* Return the bitmap corresponding to vfl_type. */
-static inline unsigned long *devnode_bits(int vfl_type)
+static inline unsigned long *devnode_bits(enum vfl_devnode_type vfl_type)
 {
        return devnode_nums[vfl_type];
 }
        return 0;
 }
 
-int __video_register_device(struct video_device *vdev, int type, int nr,
-               int warn_if_nr_in_use, struct module *owner)
+int __video_register_device(struct video_device *vdev,
+                           enum vfl_devnode_type type,
+                           int nr, int warn_if_nr_in_use,
+                           struct module *owner)
 {
        int i = 0;
        int ret;
 
 
 #define VIDEO_MAJOR    81
 
-#define VFL_TYPE_GRABBER       0
-#define VFL_TYPE_VBI           1
-#define VFL_TYPE_RADIO         2
-#define VFL_TYPE_SUBDEV                3
-#define VFL_TYPE_SDR           4
-#define VFL_TYPE_TOUCH         5
-#define VFL_TYPE_MAX           6
+/**
+ * enum vfl_devnode_type - type of V4L2 device node
+ *
+ * @VFL_TYPE_GRABBER:  for video input/output devices
+ * @VFL_TYPE_VBI:      for vertical blank data (i.e. closed captions, teletext)
+ * @VFL_TYPE_RADIO:    for radio tuners
+ * @VFL_TYPE_SUBDEV:   for V4L2 subdevices
+ * @VFL_TYPE_SDR:      for Software Defined Radio tuners
+ * @VFL_TYPE_TOUCH:    for touch sensors
+ */
+enum vfl_devnode_type {
+       VFL_TYPE_GRABBER        = 0,
+       VFL_TYPE_VBI            = 1,
+       VFL_TYPE_RADIO          = 2,
+       VFL_TYPE_SUBDEV         = 3,
+       VFL_TYPE_SDR            = 4,
+       VFL_TYPE_TOUCH          = 5,
+};
+#define VFL_TYPE_MAX VFL_TYPE_TOUCH
 
 /* Is this a receiver, transmitter or mem-to-mem? */
 /* Ignored for VFL_TYPE_SUBDEV. */
  * @prio: pointer to &struct v4l2_prio_state with device's Priority state.
  *      If NULL, then v4l2_dev->prio will be used.
  * @name: video device name
- * @vfl_type: V4L device type
+ * @vfl_type: V4L device type, as defined by &enum vfl_devnode_type
  * @vfl_dir: V4L receiver, transmitter or m2m
  * @minor: device node 'minor'. It is set to -1 if the registration failed
  * @num: number of the video device node
 
        /* device info */
        char name[32];
-       int vfl_type;
+       enum vfl_devnode_type vfl_type;
        int vfl_dir;
        int minor;
        u16 num;
  * __video_register_device - register video4linux devices
  *
  * @vdev: struct video_device to register
- * @type: type of device to register
+ * @type: type of device to register, as defined by &enum vfl_devnode_type
  * @nr:   which device node number is desired:
  *     (0 == /dev/video0, 1 == /dev/video1, ..., -1 == first free)
  * @warn_if_nr_in_use: warn if the desired device node number
  *
  * Returns 0 on success.
  *
- * Valid values for @type are:
- *
- *     - %VFL_TYPE_GRABBER - A frame grabber
- *     - %VFL_TYPE_VBI - Vertical blank data (undecoded)
- *     - %VFL_TYPE_RADIO - A radio card
- *     - %VFL_TYPE_SUBDEV - A subdevice
- *     - %VFL_TYPE_SDR - Software Defined Radio
- *     - %VFL_TYPE_TOUCH - A touch sensor
- *
  * .. note::
  *
  *     This function is meant to be used only inside the V4L2 core.
  *     Drivers should use video_register_device() or
  *     video_register_device_no_warn().
  */
-int __must_check __video_register_device(struct video_device *vdev, int type,
-               int nr, int warn_if_nr_in_use, struct module *owner);
+int __must_check __video_register_device(struct video_device *vdev,
+                                        enum vfl_devnode_type type,
+                                        int nr, int warn_if_nr_in_use,
+                                        struct module *owner);
 
 /**
  *  video_register_device - register video4linux devices
  *
  * @vdev: struct video_device to register
- * @type: type of device to register
+ * @type: type of device to register, as defined by &enum vfl_devnode_type
  * @nr:   which device node number is desired:
  *     (0 == /dev/video0, 1 == /dev/video1, ..., -1 == first free)
  *
  *     you video_device_release() should be called on failure.
  */
 static inline int __must_check video_register_device(struct video_device *vdev,
-               int type, int nr)
+                                                    enum vfl_devnode_type type,
+                                                    int nr)
 {
        return __video_register_device(vdev, type, nr, 1, vdev->fops->owner);
 }
  *  video_register_device_no_warn - register video4linux devices
  *
  * @vdev: struct video_device to register
- * @type: type of device to register
+ * @type: type of device to register, as defined by &enum vfl_devnode_type
  * @nr:   which device node number is desired:
  *     (0 == /dev/video0, 1 == /dev/video1, ..., -1 == first free)
  *
  *     is responsible for freeing any data. Usually that means that
  *     you video_device_release() should be called on failure.
  */
-static inline int __must_check video_register_device_no_warn(
-               struct video_device *vdev, int type, int nr)
+static inline int __must_check
+video_register_device_no_warn(struct video_device *vdev,
+                             enum vfl_devnode_type type, int nr)
 {
        return __video_register_device(vdev, type, nr, 0, vdev->fops->owner);
 }
 
        unsigned int flags;
 };
 
+/**
+ * v4l2_fill_pix_format - Ancillary routine that fills a &struct
+ *     v4l2_pix_format fields from a &struct v4l2_mbus_framefmt.
+ *
+ * @pix_fmt:   pointer to &struct v4l2_pix_format to be filled
+ * @mbus_fmt:  pointer to &struct v4l2_mbus_framefmt to be used as model
+ */
 static inline void v4l2_fill_pix_format(struct v4l2_pix_format *pix_fmt,
                                const struct v4l2_mbus_framefmt *mbus_fmt)
 {
        pix_fmt->xfer_func = mbus_fmt->xfer_func;
 }
 
+/**
+ * v4l2_fill_pix_format - Ancillary routine that fills a &struct
+ *     v4l2_mbus_framefmt from a &struct v4l2_pix_format and a
+ *     data format code.
+ *
+ * @mbus_fmt:  pointer to &struct v4l2_mbus_framefmt to be filled
+ * @pix_fmt:   pointer to &struct v4l2_pix_format to be used as model
+ * @code:      data format code (from &enum v4l2_mbus_pixelcode)
+ */
 static inline void v4l2_fill_mbus_format(struct v4l2_mbus_framefmt *mbus_fmt,
                           const struct v4l2_pix_format *pix_fmt,
                           u32 code)
        mbus_fmt->code = code;
 }
 
+/**
+ * v4l2_fill_pix_format - Ancillary routine that fills a &struct
+ *     v4l2_pix_format_mplane fields from a media bus structure.
+ *
+ * @pix_mp_fmt:        pointer to &struct v4l2_pix_format_mplane to be filled
+ * @mbus_fmt:  pointer to &struct v4l2_mbus_framefmt to be used as model
+ */
 static inline void v4l2_fill_pix_format_mplane(
                                struct v4l2_pix_format_mplane *pix_mp_fmt,
                                const struct v4l2_mbus_framefmt *mbus_fmt)
        pix_mp_fmt->xfer_func = mbus_fmt->xfer_func;
 }
 
+/**
+ * v4l2_fill_pix_format - Ancillary routine that fills a &struct
+ *     v4l2_mbus_framefmt from a &struct v4l2_pix_format_mplane.
+ *
+ * @mbus_fmt:  pointer to &struct v4l2_mbus_framefmt to be filled
+ * @pix_mp_fmt:        pointer to &struct v4l2_pix_format_mplane to be used as model
+ */
 static inline void v4l2_fill_mbus_format_mplane(
                                struct v4l2_mbus_framefmt *mbus_fmt,
                                const struct v4l2_pix_format_mplane *pix_mp_fmt)