From: Christian Kellner <christian@kellner.me>
Date: Thu, 9 Mar 2017 14:51:49 +0000 (+0100)
Subject: sink: Invert priorities to prefer external devices
X-Git-Tag: v10.99.1~67
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b53dd1f0df269d7e1b55e059b0a0d3d530b124b5;p=platform%2Fupstream%2Fpulseaudio.git

sink: Invert priorities to prefer external devices

Currently internal > speaker > headphone and pci > usb > bluetooth.
Invert both of these sets, with the reasoning that a headphone and
speakers are something that a user has actively attached and should
therefore get a higher priority. The same reasoning is applied for
the bus type, i.e. bluetooth and usb should be higher than pci,
because they most likely have been actively attached be a user.

BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=99222
---

diff --git a/src/pulsecore/sink.c b/src/pulsecore/sink.c
index 6011f45..fb16d57 100644
--- a/src/pulsecore/sink.c
+++ b/src/pulsecore/sink.c
@@ -3511,21 +3511,21 @@ unsigned pa_device_init_priority(pa_proplist *p) {
 
     if ((s = pa_proplist_gets(p, PA_PROP_DEVICE_FORM_FACTOR))) {
 
-        if (pa_streq(s, "internal"))
+        if (pa_streq(s, "headphone"))
             priority += 900;
         else if (pa_streq(s, "speaker"))
             priority += 500;
-        else if (pa_streq(s, "headphone"))
+        else if (pa_streq(s, "internal"))
             priority += 400;
     }
 
     if ((s = pa_proplist_gets(p, PA_PROP_DEVICE_BUS))) {
 
-        if (pa_streq(s, "pci"))
+        if (pa_streq(s, "bluetooth"))
             priority += 50;
         else if (pa_streq(s, "usb"))
             priority += 40;
-        else if (pa_streq(s, "bluetooth"))
+        else if (pa_streq(s, "pci"))
             priority += 30;
     }