Hi Andre,
> This patch changes hci_connect_le() so it uses the connection
> parameters specified for the certain device. If no parameters
> were configured, we use the default values.
>
> Signed-off-by: Andre Guedes <andre.guedes@xxxxxxxxxxxxx>
> ---
> net/bluetooth/hci_conn.c | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c
> index 195b78f..075d070 100644
> --- a/net/bluetooth/hci_conn.c
> +++ b/net/bluetooth/hci_conn.c
> @@ -608,6 +608,7 @@ static struct hci_conn *hci_connect_le(struct hci_dev *hdev, bdaddr_t *dst,
> {
> struct hci_conn *conn;
> int err;
> + struct hci_conn_params *params;
Put it above err variable.
>
> if (test_bit(HCI_ADVERTISING, &hdev->flags))
> return ERR_PTR(-ENOTSUPP);
> @@ -652,8 +653,16 @@ static struct hci_conn *hci_connect_le(struct hci_dev *hdev, bdaddr_t *dst,
> conn->sec_level = BT_SECURITY_LOW;
> conn->pending_sec_level = sec_level;
> conn->auth_type = auth_type;
> - conn->conn_interval_min = hdev->le_conn_min_interval;
> - conn->conn_interval_max = hdev->le_conn_max_interval;
> +
> + params = hci_find_conn_params(hdev, &conn->dst, conn->dst_type);
Here is it now clear why the naming is horrible.
If you want to match with a put, then this needs to be hci_conn_params_get().
> + if (params) {
> + conn->conn_interval_min = params->conn_interval_min;
> + conn->conn_interval_max = params->conn_interval_max;
> + hci_conn_params_put(params);
> + } else {
> + conn->conn_interval_min = hdev->le_conn_min_interval;
> + conn->conn_interval_max = hdev->le_conn_max_interval;
> + }
>
> err = hci_create_le_conn(conn);
> if (err)
Regards
Marcel
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
