Re: [PATCH 2/2] Bluetooth: Implement deferred sco socket setup

Hi,

On 11/20/2012 07:01 PM, Gustavo Padovan wrote:

* Frédéric Dalleau [off-list ref] [2012-11-20 18:55:39 +0100]:

quoted

Hi Gustavo,

I'm taking your remarks into account asap

On 11/20/2012 06:37 PM, Gustavo Padovan wrote:

quoted

Where do you set sk_state to BT_CONNECTED. Doesn't it need to be here?

The state is set to BT_CONNECTED in hci_sync_conn_complete_evt. ie when
the connection is effectively created.

Are we both talking about the sk_state var? I failed to see how it is set to
BT_CONNECTED.

Yes, hci_sync_conn_complete_evt > hci_proto_connect_cfm > sco_connect_cfm >

if there is no error, sco_conn_ready.
Since the socket is already created we get in first if branch.

Frédéric