diff options
| author | Hans Verkuil <hans.verkuil@cisco.com> | 2013-03-09 06:18:03 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-03-24 12:10:36 -0300 |
| commit | ffb9749346b4e3bb567d0d5daa88edae9f8ec8ae (patch) | |
| tree | 7d8fa0d99143c9dbac2b054b029fce65a6809183 /drivers/staging/media/go7007/go7007-usb.c | |
| parent | 1589037f8716a605a36ee6dda6f7cdd4d043522b (diff) | |
| download | linux-ffb9749346b4e3bb567d0d5daa88edae9f8ec8ae.tar.bz2 | |
[media] go7007: fix i2c_xfer return codes
The i2c_xfer functions didn't return the proper error codes and (especially
important) on success they returned 0 instead of the number of transferred
messages.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/staging/media/go7007/go7007-usb.c')
| -rw-r--r-- | drivers/staging/media/go7007/go7007-usb.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/staging/media/go7007/go7007-usb.c b/drivers/staging/media/go7007/go7007-usb.c index 9dbf5ecd05a2..914b247e9652 100644 --- a/drivers/staging/media/go7007/go7007-usb.c +++ b/drivers/staging/media/go7007/go7007-usb.c @@ -876,10 +876,10 @@ static int go7007_usb_i2c_master_xfer(struct i2c_adapter *adapter, struct go7007_usb *usb = go->hpi_context; u8 buf[16]; int buf_len, i; - int ret = -1; + int ret = -EIO; if (go->status == STATUS_SHUTDOWN) - return -1; + return -ENODEV; mutex_lock(&usb->i2c_lock); @@ -936,7 +936,7 @@ static int go7007_usb_i2c_master_xfer(struct i2c_adapter *adapter, memcpy(msgs[i].buf, buf + 1, msgs[i].len); } } - ret = 0; + ret = num; i2c_done: mutex_unlock(&usb->i2c_lock); |