[pulseaudio-discuss] [PATCH v1 3/4] bluetooth: Avoid duplicating profile argument twice

Mikel Astiz mikel.astiz.oss at gmail.com
Thu Jul 26 05:32:47 PDT 2012


From: Mikel Astiz <mikel.astiz at bmw-carit.de>

The module module-bluetooth-device should never be given parameter
'profile' twice, even if both HFGW and A2DP are playing.
---
 src/modules/bluetooth/module-bluetooth-discover.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/modules/bluetooth/module-bluetooth-discover.c b/src/modules/bluetooth/module-bluetooth-discover.c
index 6fe1334..93dfbb3 100644
--- a/src/modules/bluetooth/module-bluetooth-discover.c
+++ b/src/modules/bluetooth/module-bluetooth-discover.c
@@ -107,11 +107,10 @@ static pa_hook_result_t load_module_for_device(pa_bluetooth_discovery *y, const
                 args = tmp;
             }
 
-            if (d->audio_source_state >= PA_BT_AUDIO_STATE_PLAYING)
-                args = pa_sprintf_malloc("%s profile=\"a2dp_source\" auto_connect=no", args);
-
             if (d->hfgw_state >= PA_BT_AUDIO_STATE_PLAYING)
                 args = pa_sprintf_malloc("%s profile=\"hfgw\"", args);
+            else if (d->audio_source_state >= PA_BT_AUDIO_STATE_PLAYING)
+                args = pa_sprintf_malloc("%s profile=\"a2dp_source\" auto_connect=no", args);
 
             pa_log_debug("Loading module-bluetooth-device %s", args);
             m = pa_module_load(u->module->core, "module-bluetooth-device", args);
-- 
1.7.7.6



More information about the pulseaudio-discuss mailing list