diff options
author | Julia Lawall <julia@diku.dk> | 2010-08-16 18:25:39 +0200 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-10-21 05:04:48 +0200 |
commit | 46a7a3cc9fe11f7a8d58fee6e7d5849298bc0212 (patch) | |
tree | a14a62a117249f940ccb1949d4cb620ec9a98dfb /drivers/media/video/zoran | |
parent | V4L/DVB: drivers/media/video: Adjust confusing if indentation (diff) | |
download | linux-46a7a3cc9fe11f7a8d58fee6e7d5849298bc0212.tar.xz linux-46a7a3cc9fe11f7a8d58fee6e7d5849298bc0212.zip |
V4L/DVB: drivers/media/video/zoran: Use available error codes
Error codes are stored in res, but the return value is always 0. Return
res 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/media/video/zoran')
-rw-r--r-- | drivers/media/video/zoran/zoran_driver.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/zoran/zoran_driver.c b/drivers/media/video/zoran/zoran_driver.c index 6f89d0a096ea..c155ddfb1332 100644 --- a/drivers/media/video/zoran/zoran_driver.c +++ b/drivers/media/video/zoran/zoran_driver.c @@ -3322,7 +3322,7 @@ zoran_mmap (struct file *file, mmap_unlock_and_return: mutex_unlock(&zr->resource_lock); - return 0; + return res; } static const struct v4l2_ioctl_ops zoran_ioctl_ops = { |