]> code.delx.au - pulseaudio/blobdiff - src/modules/bluetooth/module-bluetooth-policy.c
bluetooth: Fix lines going over column 128
[pulseaudio] / src / modules / bluetooth / module-bluetooth-policy.c
index 87a5716f1cff11b8219d28f57ba06dcb5ad63d06..0031ccf6335956ca4ba37a68c1b01930302fb2a5 100644 (file)
@@ -38,7 +38,7 @@
 PA_MODULE_AUTHOR("Frédéric Dalleau");
 PA_MODULE_DESCRIPTION("When a bluetooth sink or source is added, load module-loopback");
 PA_MODULE_VERSION(PACKAGE_VERSION);
-PA_MODULE_LOAD_ONCE(TRUE);
+PA_MODULE_LOAD_ONCE(true);
 PA_MODULE_USAGE(
         "a2dp_source=<Handle a2dp_source card profile (sink role)?> "
         "hfgw=<Handle hfgw card profile (headset role)?>");
@@ -54,7 +54,7 @@ struct userdata {
     bool enable_hfgw;
     pa_hook_slot *source_put_slot;
     pa_hook_slot *sink_put_slot;
-    pa_hook_slot *port_available_changed_slot;
+    pa_hook_slot *profile_available_changed_slot;
 };
 
 /* When a source is created, loopback it to default sink */
@@ -89,7 +89,8 @@ static pa_hook_result_t source_put_hook_callback(pa_core *c, pa_source *source,
     }
 
     /* Load module-loopback */
-    args = pa_sprintf_malloc("source=\"%s\" source_dont_move=\"true\" sink_input_properties=\"media.role=%s\"", source->name, role);
+    args = pa_sprintf_malloc("source=\"%s\" source_dont_move=\"true\" sink_input_properties=\"media.role=%s\"", source->name,
+                             role);
     (void) pa_module_load(c, "module-loopback", args);
     pa_xfree(args);
 
@@ -126,93 +127,92 @@ static pa_hook_result_t sink_put_hook_callback(pa_core *c, pa_sink *sink, void *
     }
 
     /* Load module-loopback */
-    args = pa_sprintf_malloc("sink=\"%s\" sink_dont_move=\"true\" source_output_properties=\"media.role=%s\"", sink->name, role);
+    args = pa_sprintf_malloc("sink=\"%s\" sink_dont_move=\"true\" source_output_properties=\"media.role=%s\"", sink->name,
+                             role);
     (void) pa_module_load(c, "module-loopback", args);
     pa_xfree(args);
 
     return PA_HOOK_OK;
 }
 
-static pa_device_port* find_best_port(pa_hashmap *ports) {
-    void *state;
-    pa_device_port *port;
-    pa_device_port *result = NULL;
-
-    PA_HASHMAP_FOREACH(port, ports, state) {
-        if (port->available != PA_PORT_AVAILABLE_YES)
-            continue;
-
-        if (result == NULL || port->priority > result->priority)
-            result = port;
-    }
-
-    return result;
-}
-
-static void set_port_profile(pa_card *card, pa_device_port *port) {
+static pa_card_profile *find_best_profile(pa_card *card) {
     void *state;
     pa_card_profile *profile;
+    pa_card_profile *result = card->active_profile;
+    pa_card_profile *off;
 
-    PA_HASHMAP_FOREACH(profile, port->profiles, state) {
-        if (card->active_profile == profile)
-            return;
-
-        pa_log_debug("Setting card '%s' to profile '%s'", card->name, profile->name);
+    pa_assert_se(off = pa_hashmap_get(card->profiles, "off"));
 
-        if (pa_card_set_profile(card, profile->name, FALSE) != 0)
-            pa_log_warn("Could not set profile '%s'", profile->name);
+    PA_HASHMAP_FOREACH(profile, card->profiles, state) {
+        if (profile->available == PA_AVAILABLE_NO || profile == off)
+            continue;
 
-        return;
+        if (result == NULL ||
+            (profile->available == PA_AVAILABLE_YES && result->available == PA_AVAILABLE_UNKNOWN) ||
+            (profile->available == result->available && profile->priority > result->priority))
+            result = profile;
     }
+
+    return result ? result : off;
 }
 
-static pa_hook_result_t port_available_hook_callback(pa_core *c, pa_device_port *port, void *userdata) {
+static pa_hook_result_t profile_available_hook_callback(pa_core *c, pa_card_profile *profile, void *userdata) {
     pa_card *card;
     const char *s;
-    uint32_t state;
-    pa_bool_t is_active_profile;
-    pa_device_port *port2;
+    bool is_active_profile;
+    pa_card_profile *selected_profile;
 
-    PA_IDXSET_FOREACH(card, c->cards, state)
-        if (port == pa_hashmap_get(card->ports, port->name))
-            break;
-
-    if (!card) {
-        pa_log_warn("Did not find port %s in array of cards", port->name);
-        return PA_HOOK_OK;
-    }
+    pa_assert(c);
+    pa_assert(profile);
+    pa_assert_se((card = profile->card));
 
     /* Only consider bluetooth cards */
     s = pa_proplist_gets(card->proplist, PA_PROP_DEVICE_BUS);
     if (!s || !pa_streq(s, "bluetooth"))
         return PA_HOOK_OK;
 
-    is_active_profile = card->active_profile == pa_hashmap_get(port->profiles, card->active_profile->name);
-
-    if (is_active_profile && port->available == PA_PORT_AVAILABLE_YES)
+    /* Do not automatically switch profiles for headsets, just in case */
+    if (pa_streq(profile->name, "hsp") || pa_streq(profile->name, "a2dp"))
         return PA_HOOK_OK;
 
-    if (!is_active_profile && port->available != PA_PORT_AVAILABLE_YES)
-        return PA_HOOK_OK;
+    is_active_profile = card->active_profile == profile;
 
-    if ((port2 = find_best_port(card->ports)) == NULL)
-        return PA_HOOK_OK;
+    if (profile->available == PA_AVAILABLE_YES) {
+        if (is_active_profile)
+            return PA_HOOK_OK;
+
+        if (card->active_profile->available == PA_AVAILABLE_YES && card->active_profile->priority >= profile->priority)
+            return PA_HOOK_OK;
+
+        selected_profile = profile;
+    } else {
+        if (!is_active_profile)
+            return PA_HOOK_OK;
+
+        pa_assert_se((selected_profile = find_best_profile(card)));
+
+        if (selected_profile == card->active_profile)
+            return PA_HOOK_OK;
+    }
+
+    pa_log_debug("Setting card '%s' to profile '%s'", card->name, selected_profile->name);
 
-    set_port_profile(card, port2);
+    if (pa_card_set_profile(card, selected_profile, false) != 0)
+        pa_log_warn("Could not set profile '%s'", selected_profile->name);
 
     return PA_HOOK_OK;
 }
 
-static void handle_all_ports(pa_core *core) {
+static void handle_all_profiles(pa_core *core) {
     pa_card *card;
     uint32_t state;
 
     PA_IDXSET_FOREACH(card, core->cards, state) {
-        pa_device_port *port;
+        pa_card_profile *profile;
         void *state2;
 
-        PA_HASHMAP_FOREACH(port, card->ports, state2)
-            port_available_hook_callback(core, port, NULL);
+        PA_HASHMAP_FOREACH(profile, card->profiles, state2)
+            profile_available_hook_callback(core, profile, NULL);
     }
 }
 
@@ -229,26 +229,28 @@ int pa__init(pa_module *m) {
 
     m->userdata = u = pa_xnew0(struct userdata, 1);
 
-    u->enable_a2dp_source = TRUE;
+    u->enable_a2dp_source = true;
     if (pa_modargs_get_value_boolean(ma, "a2dp_source", &u->enable_a2dp_source) < 0) {
         pa_log("Failed to parse a2dp_source argument.");
         goto fail;
     }
 
-    u->enable_hfgw = TRUE;
+    u->enable_hfgw = true;
     if (pa_modargs_get_value_boolean(ma, "hfgw", &u->enable_hfgw) < 0) {
         pa_log("Failed to parse hfgw argument.");
         goto fail;
     }
 
-    u->source_put_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SOURCE_PUT], PA_HOOK_NORMAL, (pa_hook_cb_t) source_put_hook_callback, u);
+    u->source_put_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SOURCE_PUT], PA_HOOK_NORMAL,
+                                         (pa_hook_cb_t) source_put_hook_callback, u);
 
-    u->sink_put_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SINK_PUT], PA_HOOK_NORMAL, (pa_hook_cb_t) sink_put_hook_callback, u);
+    u->sink_put_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_SINK_PUT], PA_HOOK_NORMAL,
+                                       (pa_hook_cb_t) sink_put_hook_callback, u);
 
-    u->port_available_changed_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_PORT_AVAILABLE_CHANGED],
-                                       PA_HOOK_NORMAL, (pa_hook_cb_t) port_available_hook_callback, u);
+    u->profile_available_changed_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_CARD_PROFILE_AVAILABLE_CHANGED],
+                                                        PA_HOOK_NORMAL, (pa_hook_cb_t) profile_available_hook_callback, u);
 
-    handle_all_ports(m->core);
+    handle_all_profiles(m->core);
 
     pa_modargs_free(ma);
     return 0;
@@ -272,8 +274,8 @@ void pa__done(pa_module *m) {
     if (u->sink_put_slot)
         pa_hook_slot_free(u->sink_put_slot);
 
-    if (u->port_available_changed_slot)
-        pa_hook_slot_free(u->port_available_changed_slot);
+    if (u->profile_available_changed_slot)
+        pa_hook_slot_free(u->profile_available_changed_slot);
 
     pa_xfree(u);
 }