This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/media_tree.git tree:

Subject: [media] saa7146: fix compiler warning
Author:  Hans Verkuil <[email protected]>
Date:    Thu Aug 25 09:54:30 2011 -0300

Signed-off-by: Hans Verkuil <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/common/saa7146_video.c |   12 ++++--------
 1 files changed, 4 insertions(+), 8 deletions(-)

---

http://git.linuxtv.org/media_tree.git?a=commitdiff;h=7b4668efc4a8f75d12337cf8013307a7ec437878

diff --git a/drivers/media/common/saa7146_video.c 
b/drivers/media/common/saa7146_video.c
index fcdf4a0..384b358 100644
--- a/drivers/media/common/saa7146_video.c
+++ b/drivers/media/common/saa7146_video.c
@@ -1356,18 +1356,14 @@ static void video_close(struct saa7146_dev *dev, struct 
file *file)
        struct saa7146_fh *fh = file->private_data;
        struct saa7146_vv *vv = dev->vv_data;
        struct videobuf_queue *q = &fh->video_q;
-       int err;
 
-       if (IS_CAPTURE_ACTIVE(fh) != 0) {
-               err = video_end(fh, file);
-       } else if (IS_OVERLAY_ACTIVE(fh) != 0) {
-               err = saa7146_stop_preview(fh);
-       }
+       if (IS_CAPTURE_ACTIVE(fh) != 0)
+               video_end(fh, file);
+       else if (IS_OVERLAY_ACTIVE(fh) != 0)
+               saa7146_stop_preview(fh);
 
        videobuf_stop(q);
-
        /* hmm, why is this function declared void? */
-       /* return err */
 }
 
 

_______________________________________________
linuxtv-commits mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to