diff options
| author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2008-11-27 01:03:18 +0000 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2008-11-27 01:03:18 +0000 |
| commit | be469f334286914f2d78ec4ea51b8083b4542e1b (patch) | |
| tree | 067a18e6d4ebe3e670e0bbe2d55f9ee3079d732f /linux/drivers | |
| parent | 811d39fde60ebeaa0bb102736149c5ddecb94535 (diff) | |
| download | mediapointer-dvb-s2-be469f334286914f2d78ec4ea51b8083b4542e1b.tar.gz mediapointer-dvb-s2-be469f334286914f2d78ec4ea51b8083b4542e1b.tar.bz2 | |
cx23885: unsigned cx23417_mailbox cannot be negative
From: roel kluin <roel.kluin@gmail.com>
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 'linux/drivers')
| -rw-r--r-- | linux/drivers/media/video/cx23885/cx23885-417.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/linux/drivers/media/video/cx23885/cx23885-417.c b/linux/drivers/media/video/cx23885/cx23885-417.c index 1aa8a9ca3..52f6824ff 100644 --- a/linux/drivers/media/video/cx23885/cx23885-417.c +++ b/linux/drivers/media/video/cx23885/cx23885-417.c @@ -1027,12 +1027,13 @@ static int cx23885_initialize_codec(struct cx23885_dev *dev) printk(KERN_ERR "%s() f/w load failed\n", __func__); return retval; } - dev->cx23417_mailbox = cx23885_find_mailbox(dev); - if (dev->cx23417_mailbox < 0) { + retval = cx23885_find_mailbox(dev); + if (retval < 0) { printk(KERN_ERR "%s() mailbox < 0, error\n", __func__); return -1; } + dev->cx23417_mailbox = retval; retval = cx23885_api_cmd(dev, CX2341X_ENC_PING_FW, 0, 0); if (retval < 0) { printk(KERN_ERR |