Flag V4L2_FH_USE_PRIO is set if we can safely store the priority level
in file->private_data. Support this flag for the open and release file
operations.
Signed-off-by: Hans Verkuil <hverkuil@xxxxxxxxx>
---
drivers/media/video/v4l2-dev.c | 23 +++++++++++++++++++++--
include/media/v4l2-dev.h | 6 ++++++
2 files changed, 27 insertions(+), 2 deletions(-)
diff --git a/drivers/media/video/v4l2-dev.c b/drivers/media/video/v4l2-dev.c
index 0ca7ec9..fd536ff 100644
--- a/drivers/media/video/v4l2-dev.c
+++ b/drivers/media/video/v4l2-dev.c
@@ -283,6 +283,15 @@ static int v4l2_open(struct inode *inode, struct file *filp)
/* and increase the device refcount */
video_get(vdev);
mutex_unlock(&videodev_lock);
+
+ if (test_bit(V4L2_FL_USES_V4L2_PRIO, &vdev->flags)) {
+ enum v4l2_priority prio =
+ (enum v4l2_priority)filp->private_data;
+
+ v4l2_prio_open(&vdev->v4l2_dev->prio, &prio);
+ filp->private_data = (void *)prio;
+ }
+
if (vdev->fops->open)
ret = vdev->fops->open(filp);
@@ -296,15 +305,20 @@ static int v4l2_open(struct inode *inode, struct file *filp)
static int v4l2_release(struct inode *inode, struct file *filp)
{
struct video_device *vdev = video_devdata(filp);
- int ret = 0;
+ if (test_bit(V4L2_FL_USES_V4L2_PRIO, &vdev->flags)) {
+ enum v4l2_priority prio =
+ (enum v4l2_priority)filp->private_data;
+
+ v4l2_prio_close(&vdev->v4l2_dev->prio, prio);
+ }
if (vdev->fops->release)
vdev->fops->release(filp);
/* decrease the refcount unconditionally since the release()
return value is ignored. */
video_put(vdev);
- return ret;
+ return 0;
}
static const struct file_operations v4l2_unlocked_fops = {
@@ -515,6 +529,11 @@ static int __video_register_device(struct video_device *vdev, int type, int nr,
vdev->index = get_index(vdev);
mutex_unlock(&videodev_lock);
+ /* Determine if we can store v4l2_priority in file->private_data */
+ if (vdev->fops->open == NULL && vdev->fops->release == NULL &&
+ vdev->v4l2_dev != NULL)
+ set_bit(V4L2_FL_USES_V4L2_PRIO, &vdev->flags);
+
/* Part 3: Initialize the character device */
vdev->cdev = cdev_alloc();
if (vdev->cdev == NULL) {
diff --git a/include/media/v4l2-dev.h b/include/media/v4l2-dev.h
index bebe44b..a116bc5 100644
--- a/include/media/v4l2-dev.h
+++ b/include/media/v4l2-dev.h
@@ -32,7 +32,13 @@ struct v4l2_device;
Drivers can clear this flag if they want to block all future
device access. It is cleared by video_unregister_device. */
#define V4L2_FL_REGISTERED (0)
+/* Flag to mark that the driver stores a pointer to struct v4l2_fh in
+ file->private_data. Is automatically detected. */
#define V4L2_FL_USES_V4L2_FH (1)
+/* Flag to mark that the driver stores the local v4l2_priority in
+ file->private_data. Will be used if open and release are both NULL.
+ Drivers can also set it manually. */
+#define V4L2_FL_USES_V4L2_PRIO (2)
struct v4l2_file_operations {
struct module *owner;
--
1.6.4.2
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
