diff options
author | roel kluin <roel.kluin@gmail.com> | 2008-11-26 20:03:18 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-01-02 14:12:18 -0500 |
commit | f347535a6065be6f9e65526fa82c088d68040f42 (patch) | |
tree | 3c12a8c85b70e16cea44b29c40179fdc462616a1 /drivers/media/video | |
parent | 9ed55375919bc30c448c6dd5107e8d593f96856f (diff) |
V4L/DVB (10148): cx23885: unsigned cx23417_mailbox cannot be negative
Unsigned cx23417_mailbox cannot be negative
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video')
-rw-r--r-- | drivers/media/video/cx23885/cx23885-417.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/media/video/cx23885/cx23885-417.c b/drivers/media/video/cx23885/cx23885-417.c index d9888136255d..8f1db57bd1dd 100644 --- a/drivers/media/video/cx23885/cx23885-417.c +++ b/drivers/media/video/cx23885/cx23885-417.c | |||
@@ -1027,12 +1027,13 @@ static int cx23885_initialize_codec(struct cx23885_dev *dev) | |||
1027 | printk(KERN_ERR "%s() f/w load failed\n", __func__); | 1027 | printk(KERN_ERR "%s() f/w load failed\n", __func__); |
1028 | return retval; | 1028 | return retval; |
1029 | } | 1029 | } |
1030 | dev->cx23417_mailbox = cx23885_find_mailbox(dev); | 1030 | retval = cx23885_find_mailbox(dev); |
1031 | if (dev->cx23417_mailbox < 0) { | 1031 | if (retval < 0) { |
1032 | printk(KERN_ERR "%s() mailbox < 0, error\n", | 1032 | printk(KERN_ERR "%s() mailbox < 0, error\n", |
1033 | __func__); | 1033 | __func__); |
1034 | return -1; | 1034 | return -1; |
1035 | } | 1035 | } |
1036 | dev->cx23417_mailbox = retval; | ||
1036 | retval = cx23885_api_cmd(dev, CX2341X_ENC_PING_FW, 0, 0); | 1037 | retval = cx23885_api_cmd(dev, CX2341X_ENC_PING_FW, 0, 0); |
1037 | if (retval < 0) { | 1038 | if (retval < 0) { |
1038 | printk(KERN_ERR | 1039 | printk(KERN_ERR |