[PATCH] alsa-mixer: recognize the "Speaker Jack" control

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Sep 3, 2017 at 11:10 AM, Tanu Kaskinen <tanuk at iki.fi> wrote:
> On Sun, 2017-09-03 at 10:26 -0300, Felipe Sateler wrote:
>> I don't know enough to review this patch, but I notice an inconsistency.
>>
>> On Sep 2, 2017 09:39, "Tanu Kaskinen" <tanuk at iki.fi> wrote:
>>
>> This control has been seen in the wild:
>> https://lists.freedesktop.org/archives/pulseaudio-discuss/
>> 2017-August/028595.html
>> (The pastebin link in that mail might not work anymore, but the paste
>> just shows that there's a Speaker Jack control).
>> ---
>>  src/modules/alsa/mixer/paths/analog-output-speaker.conf | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/src/modules/alsa/mixer/paths/analog-output-speaker.conf
>> b/src/modules/alsa/mixer/paths/analog-output-speaker.conf
>> index 9f4dac414..6f9968e1f 100644
>> --- a/src/modules/alsa/mixer/paths/analog-output-speaker.conf
>> +++ b/src/modules/alsa/mixer/paths/analog-output-speaker.conf
>> @@ -56,6 +56,9 @@ state.unplugged = unknown
>>  state.plugged = no
>>  state.unplugged = unknown
>>
>> +[Jack Speaker]
>> +required-any = any
>> +
>>
>>
>> The commit message and pastebin mention Speaker Jack, not Jack Speaker.
>
> This is because the configuration file uses this format for sections:
>
> [<type> <name>]
>
> Here the section type is "Jack" and the name is "Speaker".
>
> All jack controls in alsa have "Jack" at the end of their name (that's
> how PulseAudio recognizes that they are jack controls), so in this case
> the alsa control name is "Speaker Jack". We could write [Jack Speaker
> Jack] in the configuration file, but that would be ugly.

OK. Thanks for the explanation!

-- 

Saludos,
Felipe Sateler


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux