Hi Georg,
On Sat, Apr 29, 2017 at 2:40 PM, Georg Chini <georg at chini.tk> wrote:
> On 26.04.2017 14:19, Luiz Augusto von Dentz wrote:
>>
>> From: Luiz Augusto von Dentz <luiz.von.dentz at intel.com>
>>
>> This means something went wrong, which in case of ofono backend it is
>> probably due to the profile not connecting immediately but it can be
>> safely in that case.
>
>
> Is there something missing in the commit message? I don't understand
> the last part. Do you mean that in this case it can be safely assumed
> that the profile is connected now?
I mean it can be safely resume in case it does connects later.
>
>> ---
>> src/modules/bluetooth/module-bluez5-device.c | 8 +++++++-
>> 1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/modules/bluetooth/module-bluez5-device.c
>> b/src/modules/bluetooth/module-bluez5-device.c
>> index a96da17..2f0ec97 100644
>> --- a/src/modules/bluetooth/module-bluez5-device.c
>> +++ b/src/modules/bluetooth/module-bluez5-device.c
>> @@ -2060,8 +2060,14 @@ static pa_hook_result_t
>> transport_state_changed_cb(pa_bluetooth_discovery *y, pa
>> if (t == u->transport && t->state <=
>> PA_BLUETOOTH_TRANSPORT_STATE_DISCONNECTED)
>> pa_assert_se(pa_card_set_profile(u->card,
>> pa_hashmap_get(u->card->profiles, "off"), false) >= 0);
>> - if (t->device == u->device)
>> + if (t->device == u->device) {
>> + /* Auto recover from errors causing the profile to be set to off
>> */
>> + if (u->profile == PA_BLUETOOTH_PROFILE_OFF && t->state ==
>> PA_BLUETOOTH_TRANSPORT_STATE_PLAYING) {
>> + pa_log_debug("Switching to profile %s",
>> pa_bluetooth_profile_to_string(t->profile));
>> + pa_assert_se(pa_card_set_profile(u->card,
>> pa_hashmap_get(u->card->profiles,
>> pa_bluetooth_profile_to_string(t->profile)), false) >= 0);
>> + }
>> handle_transport_state_change(u, t);
>> + }
>> return PA_HOOK_OK;
>> }
>
>
>
--
Luiz Augusto von Dentz
