module-switch-on-port-available didn't do anything when a port changes its status if the card didn't have any sinks or sources. This was to avoid bad things during card initialization, but the if condition also prevented any profile switches away from the "off" profile, because the card has no sinks or sources when the "off" profile is active. pa_card nowadays has the "linked" flag that module-switch-on-port-available could have checked instead, but since it doesn't make sense to emit port status change events before the card has been initialized, I added the check in pa_device_port_set_available() instead. BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=101794 --- src/modules/module-switch-on-port-available.c | 5 ----- src/pulsecore/device-port.c | 2 +- 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/src/modules/module-switch-on-port-available.c b/src/modules/module-switch-on-port-available.c index b9a0f3b3a..4020987fd 100644 --- a/src/modules/module-switch-on-port-available.c +++ b/src/modules/module-switch-on-port-available.c @@ -281,11 +281,6 @@ static pa_hook_result_t port_available_hook_callback(pa_core *c, pa_device_port return PA_HOOK_OK; } - if (pa_idxset_size(port->card->sinks) == 0 && pa_idxset_size(port->card->sources) == 0) - /* This card is not initialized yet. We'll handle it in - sink_new / source_new callbacks later. */ - return PA_HOOK_OK; - switch (port->available) { case PA_AVAILABLE_YES: switch_to_port(port); diff --git a/src/pulsecore/device-port.c b/src/pulsecore/device-port.c index 76a7e80a1..5cf4ac639 100644 --- a/src/pulsecore/device-port.c +++ b/src/pulsecore/device-port.c @@ -92,7 +92,7 @@ void pa_device_port_set_available(pa_device_port *p, pa_available_t status) { * before the card object has been created. The card object should probably * be created before port objects, and then p->card could be non-NULL for * the whole lifecycle of pa_device_port. */ - if (p->card) { + if (p->card && p->card->linked) { /* A sink or source whose active port is unavailable can't be the * default sink/source, so port availability changes may affect the * default sink/source choice. */ -- 2.13.2
