Hi Andrei,
> Mark request status as done for Read Local Version HCI command.
> ---
> net/bluetooth/hci_event.c | 5 ++++-
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/net/bluetooth/hci_event.c b/net/bluetooth/hci_event.c
> index a646310..e064aa1 100644
> --- a/net/bluetooth/hci_event.c
> +++ b/net/bluetooth/hci_event.c
> @@ -598,7 +598,7 @@ static void hci_cc_read_local_version(struct hci_dev *hdev, struct sk_buff *skb)
> BT_DBG("%s status 0x%x", hdev->name, rp->status);
>
> if (rp->status)
> - return;
> + goto done;
>
> hdev->hci_ver = rp->hci_ver;
> hdev->hci_rev = __le16_to_cpu(rp->hci_rev);
> @@ -612,6 +612,9 @@ static void hci_cc_read_local_version(struct hci_dev *hdev, struct sk_buff *skb)
>
> if (test_bit(HCI_INIT, &hdev->flags))
> hci_setup(hdev);
> +
> +done:
> + hci_req_complete(hdev, HCI_OP_READ_LOCAL_VERSION, rp->status);
> }
this is all getting out of control. We call hci_req_complete for random
functions now. And that is because our init handling sucks.
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
