[pulseaudio-discuss] [PATCH 4/4] Revert "card: Support adding ports dynamically"
Mikel Astiz
mikel.astiz.oss at gmail.com
Tue Feb 19 01:33:50 PST 2013
From: Mikel Astiz <mikel.astiz at bmw-carit.de>
This reverts commit a9c3f2fb0fc1a1ee0072aac6425ad7311a2a5888.
It has been recently agreed that ports should somehow have some physical
meaning, leading to the port merge in module-bluetooth-device.
With this assumption in mind, it is very unlikely that a card would
add or remove ports dynamically. Therefore, the core can be simplified
by removing the support for this.
The revert affects the code added to module-card-restore in commit
a1a0ad1af209e58c136ac1c884aa5d14672a7144, which can now be partially
removed.
Conflicts:
src/pulsecore/card.c
src/pulsecore/core.h
---
src/modules/module-card-restore.c | 43 ---------------------------------------
src/pulsecore/card.c | 19 -----------------
src/pulsecore/card.h | 1 -
src/pulsecore/core.h | 1 -
4 files changed, 64 deletions(-)
diff --git a/src/modules/module-card-restore.c b/src/modules/module-card-restore.c
index 3b061b6..2b3235c 100644
--- a/src/modules/module-card-restore.c
+++ b/src/modules/module-card-restore.c
@@ -65,7 +65,6 @@ struct userdata {
pa_hook_slot *card_new_hook_slot;
pa_hook_slot *card_put_hook_slot;
pa_hook_slot *card_profile_hook_slot;
- pa_hook_slot *port_new_hook_slot;
pa_hook_slot *port_offset_hook_slot;
pa_time_event *save_time_event;
pa_database *database;
@@ -127,15 +126,6 @@ static struct port_info *port_info_new(pa_device_port *port) {
return p_info;
}
-static void port_info_update(struct port_info *p_info, pa_device_port *port) {
- pa_assert(p_info);
- pa_assert(port);
-
- pa_xfree(p_info->name);
- p_info->name = pa_xstrdup(port->name);
- p_info->offset = port->latency_offset;
-}
-
static void port_info_free(struct port_info *p_info) {
pa_assert(p_info);
@@ -413,37 +403,6 @@ static pa_hook_result_t card_profile_change_callback(pa_core *c, pa_card *card,
return PA_HOOK_OK;
}
-static pa_hook_result_t card_port_add_callback(pa_core *c, pa_device_port *port, struct userdata *u) {
- struct entry *entry;
- pa_card *card;
-
- pa_assert(port);
- card = port->card;
-
- if ((entry = entry_read(u, card->name))) {
- struct port_info *p_info;
-
- if ((p_info = pa_hashmap_get(entry->ports, port->name)))
- port_info_update(p_info, port);
- else {
- p_info = port_info_new(port);
- pa_assert_se(pa_hashmap_put(entry->ports, p_info->name, p_info) >= 0);
- }
-
- pa_log_info("Storing port info for port %s on card %s.", port->name, card->name);
-
- } else {
- entry = entry_from_card(card);
- show_full_info(card);
- }
-
- if (entry_write(u, card->name, entry))
- trigger_save(u);
-
- entry_free(entry);
- return PA_HOOK_OK;
-}
-
static pa_hook_result_t port_offset_change_callback(pa_core *c, pa_device_port *port, struct userdata *u) {
struct entry *entry;
pa_card *card;
@@ -529,7 +488,6 @@ int pa__init(pa_module*m) {
u->card_new_hook_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_CARD_NEW], PA_HOOK_EARLY, (pa_hook_cb_t) card_new_hook_callback, u);
u->card_put_hook_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_CARD_PUT], PA_HOOK_NORMAL, (pa_hook_cb_t) card_put_hook_callback, u);
u->card_profile_hook_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_CARD_PROFILE_CHANGED], PA_HOOK_NORMAL, (pa_hook_cb_t) card_profile_change_callback, u);
- u->port_new_hook_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_PORT_ADDED], PA_HOOK_NORMAL, (pa_hook_cb_t) card_port_add_callback, u);
u->port_offset_hook_slot = pa_hook_connect(&m->core->hooks[PA_CORE_HOOK_PORT_LATENCY_OFFSET_CHANGED], PA_HOOK_NORMAL, (pa_hook_cb_t) port_offset_change_callback, u);
u->hooks_connected = true;
@@ -569,7 +527,6 @@ void pa__done(pa_module*m) {
pa_hook_slot_free(u->card_new_hook_slot);
pa_hook_slot_free(u->card_put_hook_slot);
pa_hook_slot_free(u->card_profile_hook_slot);
- pa_hook_slot_free(u->port_new_hook_slot);
pa_hook_slot_free(u->port_offset_hook_slot);
}
diff --git a/src/pulsecore/card.c b/src/pulsecore/card.c
index 5ea8a63..9dc7c6c 100644
--- a/src/pulsecore/card.c
+++ b/src/pulsecore/card.c
@@ -113,25 +113,6 @@ void pa_card_add_profile(pa_card *c, pa_card_profile *profile) {
pa_hook_fire(&c->core->hooks[PA_CORE_HOOK_CARD_PROFILE_ADDED], profile);
}
-void pa_card_add_ports(pa_card *c, pa_hashmap *ports) {
- pa_device_port *p;
- void *state;
-
- pa_assert(c);
- pa_assert(ports);
-
- /* take ownership of the ports */
- PA_HASHMAP_FOREACH(p, ports, state) {
- p->card = c;
- pa_assert_se(pa_hashmap_put(c->ports, p->name, p) >= 0);
- }
-
- pa_subscription_post(c->core, PA_SUBSCRIPTION_EVENT_CARD|PA_SUBSCRIPTION_EVENT_CHANGE, c->index);
-
- while ((p = pa_hashmap_steal_first(ports)) != NULL)
- pa_hook_fire(&c->core->hooks[PA_CORE_HOOK_PORT_ADDED], p);
-}
-
void pa_card_new_data_set_profile(pa_card_new_data *data, const char *profile) {
pa_assert(data);
diff --git a/src/pulsecore/card.h b/src/pulsecore/card.h
index 393dbac..32deb7b 100644
--- a/src/pulsecore/card.h
+++ b/src/pulsecore/card.h
@@ -114,7 +114,6 @@ pa_card *pa_card_new(pa_core *c, pa_card_new_data *data);
void pa_card_free(pa_card *c);
void pa_card_add_profile(pa_card *c, pa_card_profile *profile);
-void pa_card_add_ports(pa_card *c, pa_hashmap *ports);
int pa_card_set_profile(pa_card *c, const char *name, pa_bool_t save);
diff --git a/src/pulsecore/core.h b/src/pulsecore/core.h
index 381897a..533ba9e 100644
--- a/src/pulsecore/core.h
+++ b/src/pulsecore/core.h
@@ -116,7 +116,6 @@ typedef enum pa_core_hook {
PA_CORE_HOOK_CARD_PROFILE_ADDED,
PA_CORE_HOOK_CARD_PROFILE_AVAILABLE_CHANGED,
PA_CORE_HOOK_PORT_AVAILABLE_CHANGED,
- PA_CORE_HOOK_PORT_ADDED,
PA_CORE_HOOK_PORT_LATENCY_OFFSET_CHANGED,
PA_CORE_HOOK_MAX
} pa_core_hook_t;
--
1.8.1
More information about the pulseaudio-discuss
mailing list