V4L/DVB (12516): ov772x: successful S_FMT and S_CROP must update user-provided rectangle
authorGuennadi Liakhovetski <g.liakhovetski@gmx.de>
Tue, 25 Aug 2009 14:46:42 +0000 (11:46 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 19 Sep 2009 03:18:44 +0000 (00:18 -0300)
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/ov772x.c

index 3417398e1b50622ce7e8ca2cd0faa19f5e68863f..c0c549fa786f202c23d20e9253b6801ea3d4ace2 100644 (file)
@@ -806,7 +806,7 @@ static const struct ov772x_win_size *ov772x_select_win(u32 width, u32 height)
 }
 
 static int ov772x_set_params(struct i2c_client *client,
-                            u32 width, u32 height, u32 pixfmt)
+                            u32 *width, u32 *height, u32 pixfmt)
 {
        struct ov772x_priv *priv = to_ov772x(client);
        int ret = -EINVAL;
@@ -829,7 +829,7 @@ static int ov772x_set_params(struct i2c_client *client,
        /*
         * select win
         */
-       priv->win = ov772x_select_win(width, height);
+       priv->win = ov772x_select_win(*width, *height);
 
        /*
         * reset hardware
@@ -941,6 +941,9 @@ static int ov772x_set_params(struct i2c_client *client,
                        goto ov772x_set_fmt_error;
        }
 
+       *width = priv->win->width;
+       *height = priv->win->height;
+
        return ret;
 
 ov772x_set_fmt_error:
@@ -961,7 +964,7 @@ static int ov772x_set_crop(struct soc_camera_device *icd,
        if (!priv->fmt)
                return -EINVAL;
 
-       return ov772x_set_params(client, rect->width, rect->height,
+       return ov772x_set_params(client, &rect->width, &rect->height,
                                 priv->fmt->fourcc);
 }
 
@@ -970,7 +973,7 @@ static int ov772x_s_fmt(struct v4l2_subdev *sd, struct v4l2_format *f)
        struct i2c_client *client = sd->priv;
        struct v4l2_pix_format *pix = &f->fmt.pix;
 
-       return ov772x_set_params(client, pix->width, pix->height,
+       return ov772x_set_params(client, &pix->width, &pix->height,
                                 pix->pixelformat);
 }