<div dir="auto"><div>I don't know enough to review this patch, but I notice an inconsistency.<br><div class="gmail_extra"><br><div class="gmail_quote">On Sep 2, 2017 09:39, "Tanu Kaskinen" <<a href="mailto:tanuk@iki.fi">tanuk@iki.fi</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This control has been seen in the wild:<br>
<a href="https://lists.freedesktop.org/archives/pulseaudio-discuss/2017-August/028595.html" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>archives/pulseaudio-discuss/<wbr>2017-August/028595.html</a><br>
(The pastebin link in that mail might not work anymore, but the paste<br>
just shows that there's a Speaker Jack control).<br>
---<br>
 src/modules/alsa/mixer/paths/<wbr>analog-output-speaker.conf | 3 +++<br>
 1 file changed, 3 insertions(+)<br>
<br>
diff --git a/src/modules/alsa/mixer/<wbr>paths/analog-output-speaker.<wbr>conf b/src/modules/alsa/mixer/<wbr>paths/analog-output-speaker.<wbr>conf<br>
index 9f4dac414..6f9968e1f 100644<br>
--- a/src/modules/alsa/mixer/<wbr>paths/analog-output-speaker.<wbr>conf<br>
+++ b/src/modules/alsa/mixer/<wbr>paths/analog-output-speaker.<wbr>conf<br>
@@ -56,6 +56,9 @@ state.unplugged = unknown<br>
 state.plugged = no<br>
 state.unplugged = unknown<br>
<br>
+[Jack Speaker]<br>
+required-any = any<br>
+<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">The commit message and pastebin mention Speaker Jack, not Jack Speaker.</div><div dir="auto"><br></div><div dir="auto">Saludos</div></div>