Bluetooth: Fix storing negative values as unsigned char
authorDan Carpenter <error27@gmail.com>
Mon, 8 Mar 2010 12:15:59 +0000 (15:15 +0300)
committerMarcel Holtmann <marcel@holtmann.org>
Mon, 10 May 2010 07:34:03 +0000 (09:34 +0200)
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>
drivers/bluetooth/btmrvl_main.c

index 53a43adf2e21b3d342e9b2bb885716cdb3b810f2..49ec89cd354c524bc6a8d4fd1a0c91261b1c48d3 100644 (file)
@@ -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) {