[media] ov7670: fix g/s_parm
authorHans Verkuil <hans.verkuil@cisco.com>
Tue, 16 Aug 2016 18:57:46 +0000 (15:57 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Mon, 10 Apr 2017 10:52:59 +0000 (07:52 -0300)
Drop unnecesary memset. Drop the unnecessary extendedmode check and
set the V4L2_CAP_TIMEPERFRAME capability.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/i2c/ov7670.c

index 9af8d3b8f848a866f250121c2144fe591884d566..50e4466a2b37acdb26706d845c4c1b70bb108526 100644 (file)
@@ -1046,7 +1046,6 @@ static int ov7670_g_parm(struct v4l2_subdev *sd, struct v4l2_streamparm *parms)
        if (parms->type != V4L2_BUF_TYPE_VIDEO_CAPTURE)
                return -EINVAL;
 
-       memset(cp, 0, sizeof(struct v4l2_captureparm));
        cp->capability = V4L2_CAP_TIMEPERFRAME;
        info->devtype->get_framerate(sd, &cp->timeperframe);
 
@@ -1061,9 +1060,8 @@ static int ov7670_s_parm(struct v4l2_subdev *sd, struct v4l2_streamparm *parms)
 
        if (parms->type != V4L2_BUF_TYPE_VIDEO_CAPTURE)
                return -EINVAL;
-       if (cp->extendedmode != 0)
-               return -EINVAL;
 
+       cp->capability = V4L2_CAP_TIMEPERFRAME;
        return info->devtype->set_framerate(sd, tpf);
 }