Commit 7bf6afbb authored Jul 18, 2012 by Laurent Pinchart Committed by Mauro Carvalho Chehab Jul 30, 2012

[media] tw9910: Don't access the device in the g_mbus_fmt operation

The g_mbus_fmt operation only needs to return the current mbus frame
format and doesn't need to configure the hardware to do so. Fix it to
avoid requiring the chip to be powered on when calling the operation.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

parent f2730756

drivers/media/video/tw9910.c

+3 −5

Original line number	Diff line number	Diff line
		@@ -699,11 +699,9 @@ static int tw9910_g_fmt(struct v4l2_subdev *sd,
		struct tw9910_priv *priv = to_tw9910(client);

		if (!priv->scale) {
		int ret;
		u32 width = 640, height = 480;
		ret = tw9910_set_frame(sd, &width, &height);
		if (ret < 0)
		return ret;
		priv->scale = tw9910_select_norm(priv->norm, 640, 480);
		if (!priv->scale)
		return -EINVAL;
		}

		mf->width = priv->scale->width;