summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArun Raghavan <arun@asymptotic.io>2020-11-10 21:04:11 -0500
committerArun Raghavan <arun@asymptotic.io>2020-11-15 04:00:25 -0500
commit323195e305ce3bd0f183fc66a301b8ad6120e17f (patch)
tree351d0b6afe3dab9ae38eadb8ac826f7a5d94f592
parente70d34ae1882544a57d61ccbd690cf46ce090d0c (diff)
downloadpulseaudio-323195e305ce3bd0f183fc66a301b8ad6120e17f.tar.gz
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
-rw-r--r--src/modules/module-switch-on-port-available.c23
1 files changed, 23 insertions, 0 deletions
diff --git a/src/modules/module-switch-on-port-available.c b/src/modules/module-switch-on-port-available.c
index 2fa7e4d62..90a01c8fb 100644
--- a/src/modules/module-switch-on-port-available.c
+++ b/src/modules/module-switch-on-port-available.c
@@ -234,6 +234,28 @@ static void switch_to_port(pa_device_port *port) {
if (pp.is_port_active)
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);
if (!pp.is_preferred_profile_active) {
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;
switch (port->available) {
+ case PA_AVAILABLE_UNKNOWN:
case PA_AVAILABLE_YES:
switch_to_port(port);
break;