V4L/DVB: drivers/media: Use available error codes
authorJulia Lawall <julia@diku.dk>
Mon, 16 Aug 2010 16:26:13 +0000 (13:26 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 21 Oct 2010 03:04:48 +0000 (01:04 -0200)
In each case, 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>
drivers/media/dvb/frontends/drx397xD.c
drivers/media/video/s2255drv.c

index f74cca6dc26b278ba657834de7660b93e02de8f2..a05007c80985c1311080d82a008f099f065eba23 100644 (file)
@@ -232,7 +232,7 @@ static int write_fw(struct drx397xD_state *s, enum blob_ix ix)
 exit_rc:
        read_unlock(&fw[s->chip_rev].lock);
 
-       return 0;
+       return rc;
 }
 
 /* Function is not endian safe, use the RD16 wrapper below */
index 8ec7c9a45a172ec2a62020de156d175e74891f36..8f7434182bc3f9b57b1c1e0e85d5f296ca4d5dfe 100644 (file)
@@ -600,7 +600,7 @@ static int s2255_got_frame(struct s2255_channel *channel, int jpgsize)
        dprintk(2, "%s: [buf/i] [%p/%d]\n", __func__, buf, buf->vb.i);
 unlock:
        spin_unlock_irqrestore(&dev->slock, flags);
-       return 0;
+       return rc;
 }
 
 static const struct s2255_fmt *format_by_fourcc(int fourcc)