Commit 99cd47bc authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab

[media] v4l2-ioctl: add priority handling support

Drivers that use v4l2_fh can now use the core framework support of g/s_priority.
Signed-off-by: default avatarHans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent dfddb244
...@@ -224,7 +224,8 @@ static int radio_si4713_s_frequency(struct file *file, void *p, ...@@ -224,7 +224,8 @@ static int radio_si4713_s_frequency(struct file *file, void *p,
s_frequency, vf); s_frequency, vf);
} }
static long radio_si4713_default(struct file *file, void *p, int cmd, void *arg) static long radio_si4713_default(struct file *file, void *p,
bool valid_prio, int cmd, void *arg)
{ {
return v4l2_device_call_until_err(get_v4l2_dev(file), 0, core, return v4l2_device_call_until_err(get_v4l2_dev(file), 0, core,
ioctl, cmd, arg); ioctl, cmd, arg);
......
...@@ -395,7 +395,8 @@ static int sync(struct camera_data *cam, int frame_nr) ...@@ -395,7 +395,8 @@ static int sync(struct camera_data *cam, int frame_nr)
* *
*****************************************************************************/ *****************************************************************************/
static long cpia2_default(struct file *file, void *fh, int cmd, void *arg) static long cpia2_default(struct file *file, void *fh, bool valid_prio,
int cmd, void *arg)
{ {
struct camera_data *cam = video_drvdata(file); struct camera_data *cam = video_drvdata(file);
__u32 gpio_val; __u32 gpio_val;
......
...@@ -1057,7 +1057,8 @@ static int cx18_log_status(struct file *file, void *fh) ...@@ -1057,7 +1057,8 @@ static int cx18_log_status(struct file *file, void *fh)
return 0; return 0;
} }
static long cx18_default(struct file *file, void *fh, int cmd, void *arg) static long cx18_default(struct file *file, void *fh, bool valid_prio,
int cmd, void *arg)
{ {
struct cx18 *cx = ((struct cx18_open_id *)fh)->cx; struct cx18 *cx = ((struct cx18_open_id *)fh)->cx;
......
...@@ -1719,7 +1719,7 @@ static int vpfe_s_crop(struct file *file, void *priv, ...@@ -1719,7 +1719,7 @@ static int vpfe_s_crop(struct file *file, void *priv,
static long vpfe_param_handler(struct file *file, void *priv, static long vpfe_param_handler(struct file *file, void *priv,
int cmd, void *param) bool valid_prio, int cmd, void *param)
{ {
struct vpfe_device *vpfe_dev = video_drvdata(file); struct vpfe_device *vpfe_dev = video_drvdata(file);
int ret = 0; int ret = 0;
......
...@@ -1795,7 +1795,8 @@ static int ivtv_decoder_ioctls(struct file *filp, unsigned int cmd, void *arg) ...@@ -1795,7 +1795,8 @@ static int ivtv_decoder_ioctls(struct file *filp, unsigned int cmd, void *arg)
return 0; return 0;
} }
static long ivtv_default(struct file *file, void *fh, int cmd, void *arg) static long ivtv_default(struct file *file, void *fh, bool valid_prio,
int cmd, void *arg)
{ {
struct ivtv *itv = ((struct ivtv_open_id *)fh)->itv; struct ivtv *itv = ((struct ivtv_open_id *)fh)->itv;
......
...@@ -1547,7 +1547,8 @@ static int vidioc_streamoff(struct file *file, void *fh, enum v4l2_buf_type i) ...@@ -1547,7 +1547,8 @@ static int vidioc_streamoff(struct file *file, void *fh, enum v4l2_buf_type i)
return 0; return 0;
} }
static long vidioc_default(struct file *file, void *fh, int cmd, void *arg) static long vidioc_default(struct file *file, void *fh, bool valid_prio,
int cmd, void *arg)
{ {
switch (cmd) { switch (cmd) {
case MEYEIOC_G_PARAMS: case MEYEIOC_G_PARAMS:
......
...@@ -643,7 +643,8 @@ static int vidioc_s_register(struct file *file, void *fh, struct v4l2_dbg_regist ...@@ -643,7 +643,8 @@ static int vidioc_s_register(struct file *file, void *fh, struct v4l2_dbg_regist
} }
#endif #endif
static long vidioc_default(struct file *file, void *fh, int cmd, void *arg) static long vidioc_default(struct file *file, void *fh, bool valid_prio,
int cmd, void *arg)
{ {
struct saa7146_dev *dev = ((struct saa7146_fh *)fh)->dev; struct saa7146_dev *dev = ((struct saa7146_fh *)fh)->dev;
struct mxb *mxb = (struct mxb *)dev->ext_priv; struct mxb *mxb = (struct mxb *)dev->ext_priv;
......
...@@ -860,7 +860,8 @@ static int pwc_enum_frameintervals(struct file *file, void *fh, ...@@ -860,7 +860,8 @@ static int pwc_enum_frameintervals(struct file *file, void *fh,
return 0; return 0;
} }
static long pwc_default(struct file *file, void *fh, int cmd, void *arg) static long pwc_default(struct file *file, void *fh, bool valid_prio,
int cmd, void *arg)
{ {
struct pwc_device *pdev = video_drvdata(file); struct pwc_device *pdev = video_drvdata(file);
......
...@@ -24,6 +24,7 @@ ...@@ -24,6 +24,7 @@
#include <media/v4l2-ctrls.h> #include <media/v4l2-ctrls.h>
#include <media/v4l2-fh.h> #include <media/v4l2-fh.h>
#include <media/v4l2-event.h> #include <media/v4l2-event.h>
#include <media/v4l2-device.h>
#include <media/v4l2-chip-ident.h> #include <media/v4l2-chip-ident.h>
#define dbgarg(cmd, fmt, arg...) \ #define dbgarg(cmd, fmt, arg...) \
...@@ -538,6 +539,7 @@ static long __video_do_ioctl(struct file *file, ...@@ -538,6 +539,7 @@ static long __video_do_ioctl(struct file *file,
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;
void *fh = file->private_data; void *fh = file->private_data;
struct v4l2_fh *vfh = NULL;
struct v4l2_format f_copy; struct v4l2_format f_copy;
long ret = -EINVAL; long ret = -EINVAL;
...@@ -553,6 +555,43 @@ static long __video_do_ioctl(struct file *file, ...@@ -553,6 +555,43 @@ static long __video_do_ioctl(struct file *file,
printk(KERN_CONT "\n"); printk(KERN_CONT "\n");
} }
if (test_bit(V4L2_FL_USES_V4L2_FH, &vfd->flags))
vfh = file->private_data;
if (vfh && !ops->vidioc_s_priority) {
switch (cmd) {
case VIDIOC_S_CTRL:
case VIDIOC_S_STD:
case VIDIOC_S_INPUT:
case VIDIOC_S_OUTPUT:
case VIDIOC_S_TUNER:
case VIDIOC_S_FREQUENCY:
case VIDIOC_S_FMT:
case VIDIOC_S_CROP:
case VIDIOC_S_AUDIO:
case VIDIOC_S_AUDOUT:
case VIDIOC_S_EXT_CTRLS:
case VIDIOC_S_FBUF:
case VIDIOC_S_PRIORITY:
case VIDIOC_S_DV_PRESET:
case VIDIOC_S_DV_TIMINGS:
case VIDIOC_S_JPEGCOMP:
case VIDIOC_S_MODULATOR:
case VIDIOC_S_PARM:
case VIDIOC_S_HW_FREQ_SEEK:
case VIDIOC_ENCODER_CMD:
case VIDIOC_OVERLAY:
case VIDIOC_REQBUFS:
case VIDIOC_STREAMON:
case VIDIOC_STREAMOFF:
ret = v4l2_prio_check(vfd->prio, vfh->prio);
if (ret)
goto exit_prio;
ret = -EINVAL;
break;
}
}
switch (cmd) { switch (cmd) {
/* --- capabilities ------------------------------------------ */ /* --- capabilities ------------------------------------------ */
...@@ -579,9 +618,12 @@ static long __video_do_ioctl(struct file *file, ...@@ -579,9 +618,12 @@ static long __video_do_ioctl(struct file *file,
{ {
enum v4l2_priority *p = arg; enum v4l2_priority *p = arg;
if (!ops->vidioc_g_priority) if (ops->vidioc_g_priority) {
break;
ret = ops->vidioc_g_priority(file, fh, p); ret = ops->vidioc_g_priority(file, fh, p);
} else if (vfh) {
*p = v4l2_prio_max(&vfd->v4l2_dev->prio);
ret = 0;
}
if (!ret) if (!ret)
dbgarg(cmd, "priority is %d\n", *p); dbgarg(cmd, "priority is %d\n", *p);
break; break;
...@@ -590,10 +632,13 @@ static long __video_do_ioctl(struct file *file, ...@@ -590,10 +632,13 @@ static long __video_do_ioctl(struct file *file,
{ {
enum v4l2_priority *p = arg; enum v4l2_priority *p = arg;
if (!ops->vidioc_s_priority) if (!ops->vidioc_s_priority && vfh == NULL)
break; break;
dbgarg(cmd, "setting priority to %d\n", *p); dbgarg(cmd, "setting priority to %d\n", *p);
if (ops->vidioc_s_priority)
ret = ops->vidioc_s_priority(file, fh, *p); ret = ops->vidioc_s_priority(file, fh, *p);
else
ret = v4l2_prio_change(&vfd->v4l2_dev->prio, &vfh->prio, *p);
break; break;
} }
...@@ -2138,13 +2183,18 @@ static long __video_do_ioctl(struct file *file, ...@@ -2138,13 +2183,18 @@ static long __video_do_ioctl(struct file *file,
} }
default: default:
{ {
bool valid_prio = true;
if (!ops->vidioc_default) if (!ops->vidioc_default)
break; break;
ret = ops->vidioc_default(file, fh, cmd, arg); if (vfh && !ops->vidioc_s_priority)
valid_prio = v4l2_prio_check(vfd->prio, vfh->prio) >= 0;
ret = ops->vidioc_default(file, fh, valid_prio, cmd, arg);
break; break;
} }
} /* switch */ } /* switch */
exit_prio:
if (vfd->debug & V4L2_DEBUG_IOCTL_ARG) { if (vfd->debug & V4L2_DEBUG_IOCTL_ARG) {
if (ret < 0) { if (ret < 0) {
v4l_print_ioctl(vfd->name, cmd); v4l_print_ioctl(vfd->name, cmd);
......
...@@ -270,7 +270,7 @@ struct v4l2_ioctl_ops { ...@@ -270,7 +270,7 @@ struct v4l2_ioctl_ops {
/* For other private ioctls */ /* For other private ioctls */
long (*vidioc_default) (struct file *file, void *fh, long (*vidioc_default) (struct file *file, void *fh,
int cmd, void *arg); bool valid_prio, int cmd, void *arg);
}; };
......
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