Bluetooth: hci_get_cmd_complete() can be static
authorFengguang Wu <fengguang.wu@intel.com>
Sat, 20 Apr 2013 13:24:31 +0000 (16:24 +0300)
committerGustavo Padovan <gustavo.padovan@collabora.co.uk>
Tue, 23 Apr 2013 23:30:48 +0000 (20:30 -0300)
There are new sparse warnings show up in

tree:   git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next master
head:   a0b644b0385fa58ca578f6dce4473e8a8e6f6c38
commit: 75e84b7c522c6e07964cd1f5bf28535768a1e9fa Bluetooth: Add __hci_cmd_sync() helper function
date:   13 days ago

>> net/bluetooth/hci_core.c:82:16: sparse: symbol 'hci_get_cmd_complete' was not declared. Should it be static?

Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Acked-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
net/bluetooth/hci_core.c

index b04ac355f1186d2976e6ace5e54ccf7d244e37cf..33843c5c49398f0e2a373de8919cde67a840451a 100644 (file)
@@ -79,7 +79,8 @@ static void hci_req_cancel(struct hci_dev *hdev, int err)
        }
 }
 
-struct sk_buff *hci_get_cmd_complete(struct hci_dev *hdev, u16 opcode, u8 event)
+static struct sk_buff *hci_get_cmd_complete(struct hci_dev *hdev, u16 opcode,
+                                           u8 event)
 {
        struct hci_ev_cmd_complete *ev;
        struct hci_event_hdr *hdr;