diff options
| author | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-04-29 21:00:30 +0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2015-05-01 12:10:38 +0300 |
| commit | 6948524d577dcad963bcdfcbf6e7d934cd18b8f7 (patch) | |
| tree | 0a23feecb67b2acee4027ce8cbc46711976ee158 /drivers/media/rc/redrat3.c | |
| parent | 269d91f53fe34b263c0b7735937f3b4ac5f56580 (diff) | |
| download | linux-6948524d577dcad963bcdfcbf6e7d934cd18b8f7.tar.xz | |
[media] redrat3: change return argument on redrat3_send_cmd() to int
redrat3_send_cmd() can return an error or the read data. However,
it currently returns an u8, as reported by smatch:
drivers/media/rc/redrat3.c:416 redrat3_send_cmd() warn: signedness bug returning '(-12)'
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/rc/redrat3.c')
| -rw-r--r-- | drivers/media/rc/redrat3.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c index c4def66f9aa2..c83292ad1b34 100644 --- a/drivers/media/rc/redrat3.c +++ b/drivers/media/rc/redrat3.c @@ -405,7 +405,7 @@ static void redrat3_process_ir_data(struct redrat3_dev *rr3) } /* Util fn to send rr3 cmds */ -static u8 redrat3_send_cmd(int cmd, struct redrat3_dev *rr3) +static int redrat3_send_cmd(int cmd, struct redrat3_dev *rr3) { struct usb_device *udev; u8 *data; |