diff options
author | Dan Carpenter <error27@gmail.com> | 2010-03-08 15:15:59 +0300 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2010-05-10 09:34:03 +0200 |
commit | 31f085fea891d5972e994dfb423231a3d4d60a8a (patch) | |
tree | cfb04aa39eebb895d07e1a4d97f865796594396f /drivers/bluetooth/btmrvl_main.c | |
parent | f48fd9c8cd746fdb055a97249a209c77dca0f710 (diff) | |
download | lwn-31f085fea891d5972e994dfb423231a3d4d60a8a.tar.gz lwn-31f085fea891d5972e994dfb423231a3d4d60a8a.zip |
Bluetooth: Fix storing negative values as unsigned char
This is mostly cleanup. There is only one caller and it just checks for
non-zero return values. Still "ret" should be int because we want to return
-EINVAL on errors.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'drivers/bluetooth/btmrvl_main.c')
-rw-r--r-- | drivers/bluetooth/btmrvl_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/bluetooth/btmrvl_main.c b/drivers/bluetooth/btmrvl_main.c index 53a43adf2e21..49ec89cd354c 100644 --- a/drivers/bluetooth/btmrvl_main.c +++ b/drivers/bluetooth/btmrvl_main.c @@ -66,7 +66,7 @@ int btmrvl_process_event(struct btmrvl_private *priv, struct sk_buff *skb) { struct btmrvl_adapter *adapter = priv->adapter; struct btmrvl_event *event; - u8 ret = 0; + int ret = 0; event = (struct btmrvl_event *) skb->data; if (event->ec != 0xff) { |