Hi Johan,
On Thu, Apr 5, 2012 at 4:34 PM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote:
> Hi Hemant,
>
> On Thu, Apr 05, 2012, Hemant Gupta wrote:
>> This patch adds management connect failed event when LE Create Connection
>> Command fails to inform BlueZ that LE Connection failed to get established.
>
> s/BlueZ/user space/
>
> Also change the summary line to:
>
> Bluetooth: mgmt: Fix missing connect failed event for LE
>
Thanks, please have a look at my new patch.
>> --- a/net/bluetooth/hci_event.c
>> +++ b/net/bluetooth/hci_event.c
>> @@ -1649,6 +1649,8 @@ static void hci_cs_le_create_conn(struct hci_dev *hdev, __u8 status)
>> if (status) {
>> if (conn && conn->state == BT_CONNECT) {
>> conn->state = BT_CLOSED;
>> + mgmt_connect_failed(hdev, &cp->peer_addr, conn->type,
>> + conn->dst_type, status);
>> hci_proto_connect_cfm(conn, status);
>> hci_conn_del(conn);
>> }
>
> Please use our new alignment style for split lines, i.e. align with the
> opening parenthesis.
>
Done, sent a new patch for review.
> Johan
--
Best Regards
Hemant Gupta
ST-Ericsson India
--
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
