diff options
author | Dan Carpenter <error27@gmail.com> | 2011-01-19 11:28:27 -0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-03-21 20:32:39 -0300 |
commit | b00aff6a36aeafc68cf9044751041ba432ab4495 (patch) | |
tree | cddbb7f71f0b6b8e20bf0baa056e4c95cdbed081 /drivers/media/dvb/frontends | |
parent | ed54c0e33c6ac72c681f00a41b47840f6570262c (diff) | |
download | op-kernel-dev-b00aff6a36aeafc68cf9044751041ba432ab4495.zip op-kernel-dev-b00aff6a36aeafc68cf9044751041ba432ab4495.tar.gz |
[media] dib9000: fix return type in dib9000_mbx_send_attr()
dib9000_mbx_send_attr() returns an int. It doesn't work to save
negative error codes in an unsigned char, so I've made "ret" an int
type.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/dvb/frontends')
-rw-r--r-- | drivers/media/dvb/frontends/dib9000.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/media/dvb/frontends/dib9000.c b/drivers/media/dvb/frontends/dib9000.c index 43fb6e4..9151876 100644 --- a/drivers/media/dvb/frontends/dib9000.c +++ b/drivers/media/dvb/frontends/dib9000.c @@ -486,10 +486,11 @@ static int dib9000_mbx_host_init(struct dib9000_state *state, u8 risc_id) #define MAX_MAILBOX_TRY 100 static int dib9000_mbx_send_attr(struct dib9000_state *state, u8 id, u16 * data, u8 len, u16 attr) { - u8 ret = 0, *d, b[2]; + u8 *d, b[2]; u16 tmp; u16 size; u32 i; + int ret = 0; if (!state->platform.risc.fw_is_running) return -EINVAL; |