Hello Peter,
on 14.06.2012 19:58 Peter Senna Tschudin said the following:
> In function fops_open variable type was set but not used.
thanks for your patch, but I think it does not work.
> Tested by compilation only.
>
> Signed-off-by: Peter Senna Tschudin <peter.senna@xxxxxxxxx>
> ---
> drivers/media/common/saa7146_fops.c | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/drivers/media/common/saa7146_fops.c b/drivers/media/common/saa7146_fops.c
> index 7d42c11..0cdbd74 100644
> --- a/drivers/media/common/saa7146_fops.c
> +++ b/drivers/media/common/saa7146_fops.c
> @@ -198,7 +198,6 @@ static int fops_open(struct file *file)
> struct saa7146_dev *dev = video_drvdata(file);
> struct saa7146_fh *fh = NULL;
> int result = 0;
> - enum v4l2_buf_type type;
>
> DEB_EE("file:%p, dev:%s\n", file, video_device_node_name(vdev));
>
> @@ -207,10 +206,6 @@ static int fops_open(struct file *file)
>
> DEB_D("using: %p\n", dev);
>
> - type = vdev->vfl_type == VFL_TYPE_GRABBER
> - ? V4L2_BUF_TYPE_VIDEO_CAPTURE
> - : V4L2_BUF_TYPE_VBI_CAPTURE;
> -
> /* check if an extension is registered */
> if( NULL == dev->ext ) {
> DEB_S("no extension registered for this device\n");
A few lines below "fh" is allocated and "fh->type" is set to "type".
Simply removing "type" will result in a compilation error IMO, so I
wonder if your compile-test really worked.
Can you have a look again?
CU
Michael.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
