Bluetooth: Simplify a the connection type handling

Now that we have separate ways of doing connections for each link type,
we can do better than an "if" statement to handle each link type.

Signed-off-by: Vinicius Costa Gomes <vinicius.gomes@openbossa.org>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>

diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c
index de7df88a396bfeabcc7daef08fc2902797b86937..2e7b7765e0eaae5ff633599de456032a3d37a1f4 100644 (file)
--- a/net/bluetooth/hci_conn.c
+++ b/net/bluetooth/hci_conn.c
@@ -568,13 +568,16 @@ struct hci_conn *hci_connect(struct hci_dev *hdev, int type, bdaddr_t *dst,
 {
        BT_DBG("%s dst %s", hdev->name, batostr(dst));
 
-       if (type == LE_LINK)
+       switch (type) {
+       case LE_LINK:
                return hci_connect_le(hdev, dst, dst_type, sec_level, auth_type);
-
-       if (type == ACL_LINK)
+       case ACL_LINK:
                return hci_connect_acl(hdev, dst, sec_level, auth_type);
+       case SCO_LINK:
+               return hci_connect_sco(hdev, dst, sec_level, auth_type);
+       }
 
-       return hci_connect_sco(hdev, dst, sec_level, auth_type);
+       return ERR_PTR(-EINVAL);
 }
 
 /* Check link security requirement */