* [patch] bluetooth: storing negative values as unsigned char
@ 2010-03-08 12:15 Dan Carpenter
2010-03-09 0:54 ` Marcel Holtmann
0 siblings, 1 reply; 2+ messages in thread
From: Dan Carpenter @ 2010-03-08 12:15 UTC (permalink / raw)
To: Marcel Holtmann
Cc: Bing Zhao, Rahul Tank, Amitkumar Karwar, linux-bluetooth,
kernel-janitors
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>
diff --git a/drivers/bluetooth/btmrvl_main.c b/drivers/bluetooth/btmrvl_main.c
index 53a43ad..49ec89c 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) {
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [patch] bluetooth: storing negative values as unsigned char
2010-03-08 12:15 [patch] bluetooth: storing negative values as unsigned char Dan Carpenter
@ 2010-03-09 0:54 ` Marcel Holtmann
0 siblings, 0 replies; 2+ messages in thread
From: Marcel Holtmann @ 2010-03-09 0:54 UTC (permalink / raw)
To: Dan Carpenter
Cc: Bing Zhao, Rahul Tank, Amitkumar Karwar, linux-bluetooth,
kernel-janitors
Hi Dan,
> 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.
patch has been applied. Thanks.
Regards
Marcel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-03-09 0:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-08 12:15 [patch] bluetooth: storing negative values as unsigned char Dan Carpenter
2010-03-09 0:54 ` Marcel Holtmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).