Bluetooth: Remove pointless inline function
authorMarcel Holtmann <marcel@holtmann.org>
Sat, 5 Oct 2013 18:47:46 +0000 (11:47 -0700)
committerJohan Hedberg <johan.hedberg@intel.com>
Sun, 6 Oct 2013 08:25:07 +0000 (10:25 +0200)
The inline function for BR/EDR controller AMP discover response
info is rather useless. Just include the code into the function
that builds the whole response.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
net/bluetooth/a2mp.c

index 6dfef731825f0b8858d9d02d7d6746487f65a239..a1aae5c45b7eed761ab55a48cf38757af515ab61 100644 (file)
@@ -75,20 +75,15 @@ u8 __next_ident(struct amp_mgr *mgr)
        return mgr->ident;
 }
 
-static inline void __a2mp_cl_bredr(struct a2mp_cl *cl)
-{
-       cl->id = AMP_ID_BREDR;
-       cl->type = AMP_TYPE_BREDR;
-       cl->status = AMP_STATUS_BLUETOOTH_ONLY;
-}
-
 /* hci_dev_list shall be locked */
 static void __a2mp_add_cl(struct amp_mgr *mgr, struct a2mp_cl *cl, u8 num_ctrl)
 {
        int i = 0;
        struct hci_dev *hdev;
 
-       __a2mp_cl_bredr(cl);
+       cl[0].id = AMP_ID_BREDR;
+       cl[0].type = AMP_TYPE_BREDR;
+       cl[0].status = AMP_STATUS_BLUETOOTH_ONLY;
 
        list_for_each_entry(hdev, &hci_dev_list, list) {
                /* Iterate through AMP controllers */