[PATCHv3 3/5] gspca: fix g/s_parm handling

[Hans Verkuil]

From: Hans Verkuil <hans.verk...@cisco.com>

Fix v4l2-compliance error: s_parm never set V4L2_CAP_TIMEPERFRAME.
Also various g/s_parm-related cleanups.

Signed-off-by: Hans Verkuil <hans.verk...@cisco.com>
Reviewed-by: Hans de Goede <hdego...@redhat.com>
---
 drivers/media/usb/gspca/gspca.c | 29 ++++++++++++++++-------------
 drivers/media/usb/gspca/ov534.c |  1 -
 drivers/media/usb/gspca/topro.c |  1 -
 3 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/drivers/media/usb/gspca/gspca.c b/drivers/media/usb/gspca/gspca.c
index ff229d3aae0f..a72799666417 100644
--- a/drivers/media/usb/gspca/gspca.c
+++ b/drivers/media/usb/gspca/gspca.c
@@ -1256,14 +1256,15 @@ static int vidioc_g_parm(struct file *filp, void *priv,
 {
        struct gspca_dev *gspca_dev = video_drvdata(filp);
 
-       parm->parm.capture.readbuffers = 2;
+       parm->parm.capture.readbuffers = gspca_dev->queue.min_buffers_needed;
 
-       if (gspca_dev->sd_desc->get_streamparm) {
-               gspca_dev->usb_err = 0;
-               gspca_dev->sd_desc->get_streamparm(gspca_dev, parm);
-               return gspca_dev->usb_err;
-       }
-       return 0;
+       if (!gspca_dev->sd_desc->get_streamparm)
+               return 0;
+
+       parm->parm.capture.capability = V4L2_CAP_TIMEPERFRAME;
+       gspca_dev->usb_err = 0;
+       gspca_dev->sd_desc->get_streamparm(gspca_dev, parm);
+       return gspca_dev->usb_err;
 }
 
 static int vidioc_s_parm(struct file *filp, void *priv,
@@ -1271,15 +1272,17 @@ static int vidioc_s_parm(struct file *filp, void *priv,
 {
        struct gspca_dev *gspca_dev = video_drvdata(filp);
 
-       parm->parm.capture.readbuffers = 2;
+       parm->parm.capture.readbuffers = gspca_dev->queue.min_buffers_needed;
 
-       if (gspca_dev->sd_desc->set_streamparm) {
-               gspca_dev->usb_err = 0;
-               gspca_dev->sd_desc->set_streamparm(gspca_dev, parm);
-               return gspca_dev->usb_err;
+       if (!gspca_dev->sd_desc->set_streamparm) {
+               parm->parm.capture.capability = 0;
+               return 0;
        }
 
-       return 0;
+       parm->parm.capture.capability = V4L2_CAP_TIMEPERFRAME;
+       gspca_dev->usb_err = 0;
+       gspca_dev->sd_desc->set_streamparm(gspca_dev, parm);
+       return gspca_dev->usb_err;
 }
 
 static int gspca_queue_setup(struct vb2_queue *vq,
diff --git a/drivers/media/usb/gspca/ov534.c b/drivers/media/usb/gspca/ov534.c
index f293921a1f2b..d06dc0755b9a 100644
--- a/drivers/media/usb/gspca/ov534.c
+++ b/drivers/media/usb/gspca/ov534.c
@@ -1476,7 +1476,6 @@ static void sd_get_streamparm(struct gspca_dev *gspca_dev,
        struct v4l2_fract *tpf = &cp->timeperframe;
        struct sd *sd = (struct sd *) gspca_dev;
 
-       cp->capability |= V4L2_CAP_TIMEPERFRAME;
        tpf->numerator = 1;
        tpf->denominator = sd->frame_rate;
 }
diff --git a/drivers/media/usb/gspca/topro.c b/drivers/media/usb/gspca/topro.c
index 82e2be14cad8..6f3ec0366a2f 100644
--- a/drivers/media/usb/gspca/topro.c
+++ b/drivers/media/usb/gspca/topro.c
@@ -4780,7 +4780,6 @@ static void sd_get_streamparm(struct gspca_dev *gspca_dev,
        struct v4l2_fract *tpf = &cp->timeperframe;
        int fr, i;
 
-       cp->capability |= V4L2_CAP_TIMEPERFRAME;
        tpf->numerator = 1;
        i = get_fr_idx(gspca_dev);
        if (i & 0x80) {
-- 
2.17.0