diff options
author | Julia Lawall <julia@diku.dk> | 2010-08-16 12:27:47 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-10-20 23:04:48 -0400 |
commit | 3053814c1cbedb9d2802240f414588009706c74d (patch) | |
tree | db390eb99757060daa707fe876b74f0af1971c3a /drivers | |
parent | f27709792912d30a4f1f5ae98fd23aab2d4483b4 (diff) |
V4L/DVB: drivers/media/video: Use available error codes
Error codes are stored in rc, but the return value is always 0. Return rc
instead.
The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@r@
local idexpression x;
constant C;
@@
if (...) { ...
x = -C
... when != x
(
return <+...x...+>;
|
return NULL;
|
return;
|
* return ...;
)
}
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/media/video/zr364xx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/zr364xx.c b/drivers/media/video/zr364xx.c index a82b5bd18d26..616c61fb3493 100644 --- a/drivers/media/video/zr364xx.c +++ b/drivers/media/video/zr364xx.c | |||
@@ -572,7 +572,7 @@ static int zr364xx_got_frame(struct zr364xx_camera *cam, int jpgsize) | |||
572 | DBG("wakeup [buf/i] [%p/%d]\n", buf, buf->vb.i); | 572 | DBG("wakeup [buf/i] [%p/%d]\n", buf, buf->vb.i); |
573 | unlock: | 573 | unlock: |
574 | spin_unlock_irqrestore(&cam->slock, flags); | 574 | spin_unlock_irqrestore(&cam->slock, flags); |
575 | return 0; | 575 | return rc; |
576 | } | 576 | } |
577 | 577 | ||
578 | /* this function moves the usb stream read pipe data | 578 | /* this function moves the usb stream read pipe data |