switch-on-port-available: Switch to headphones on unknown availability

Since not all users will have environments that asks what they plugged
in when their hardware supports TRRS inputs but don't have impedance
sensing, let's emulate our previous default behaviour of enabling the
headphone port at least.

This can likely be improved so users can configure the module to select
for the device they are most likely to plug in (so an option to enable
just the microphone, or headphones+headset-mic ports).

Fixes: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/issues/1028
This commit is contained in:
Arun Raghavan 2020-11-10 21:04:11 -05:00
parent e70d34ae18
commit 323195e305

View file

@ -234,6 +234,28 @@ static void switch_to_port(pa_device_port *port) {
if (pp.is_port_active) if (pp.is_port_active)
return; /* Already selected */ return; /* Already selected */
/* If a port availability became unknown, let's see if it's part of some
* availability group. If it is, it is likely to be a headphone jack that
* does not have impedance sensing to detect whether what was plugged in
* was a headphone, headset or microphone. In desktop environments that
* support it, this will trigger a user choice to select what kind of
* device was plugged in. However, let's switch to the headphone port at
* least, so that we have don't break functionality for setups that can't
* trigger this kind of interaction.
*
* We should make this configurable so that users can optionally override
* the default to a headset or mic.
*/
if (port->available == PA_AVAILABLE_UNKNOWN) {
/* Not part of a group of ports, so likely not a combination port */
if (!port->availability_group)
return;
/* For no we only switch the headphone port */
if (port->direction != PA_DIRECTION_OUTPUT)
return;
}
pa_log_debug("Trying to switch to port %s", port->name); pa_log_debug("Trying to switch to port %s", port->name);
if (!pp.is_preferred_profile_active) { if (!pp.is_preferred_profile_active) {
if (try_to_switch_profile(port) < 0) { if (try_to_switch_profile(port) < 0) {
@ -303,6 +325,7 @@ static pa_hook_result_t port_available_hook_callback(pa_core *c, pa_device_port
return PA_HOOK_OK; return PA_HOOK_OK;
switch (port->available) { switch (port->available) {
case PA_AVAILABLE_UNKNOWN:
case PA_AVAILABLE_YES: case PA_AVAILABLE_YES:
switch_to_port(port); switch_to_port(port);
break; break;