[alsa-devel] [PATCH 2/2] ALSA: hda - Add models for Dell headset jacks

[David Henningsson]

On 06/17/2013 09:49 AM, Takashi Iwai wrote:
> At Tue, 11 Jun 2013 11:06:49 +0200,
> David Henningsson wrote:
>>
>> These headset jacks keep coming in on more and more platforms, and
>> it's possible I don't catch them all. Make it easier to test and
>> verify by making models.
>>
>> Signed-off-by: David Henningsson <david.henningsson at canonical.com>
>
> The patch does more than it says; it reformats and modifies
> HD-Audio-Models.txt.  If it's intended to be merged to 3.10 kernel,
> please make things as minimal as possible, then prepare another patch
> for cleaning up the whole document.

Given Linus's latest "don't send me non-critical stuff" comment [1], I 
was expecting you to defer this patch to 3.11 anyway. If you can merge 
the first patch (the one that adds three headset IDs) to 3.10, that 
would be appreciated though.

Also, I'm not sure how to split this patch, all reformat/modifications 
are just because of the documentation of the added models.

>
>
> thanks,
>
> Takashi
>
>> ---
>>   Documentation/sound/alsa/HD-Audio-Models.txt |   41 ++++++++++++++------------
>>   sound/pci/hda/patch_realtek.c                |    3 ++
>>   2 files changed, 25 insertions(+), 19 deletions(-)
>>
>> diff --git a/Documentation/sound/alsa/HD-Audio-Models.txt b/Documentation/sound/alsa/HD-Audio-Models.txt
>> index bb8b0dc..9a0927c 100644
>> --- a/Documentation/sound/alsa/HD-Audio-Models.txt
>> +++ b/Documentation/sound/alsa/HD-Audio-Models.txt
>> @@ -21,27 +21,30 @@ ALC267/268
>>   ==========
>>     inv-dmic	Inverted internal mic workaround
>>
>> -ALC269/270/275/276/280/282
>> +ALC269/270/275/276/28x/29x
>>   ======
>> -  laptop-amic	Laptops with analog-mic input
>> -  laptop-dmic	Laptops with digital-mic input
>> -  alc269-dmic	Enable ALC269(VA) digital mic workaround
>> -  alc271-dmic	Enable ALC271X digital mic workaround
>> -  inv-dmic	Inverted internal mic workaround
>> -  lenovo-dock   Enables docking station I/O for some Lenovos
>> -
>> -ALC662/663/272
>> +  laptop-amic		Laptops with analog-mic input
>> +  laptop-dmic		Laptops with digital-mic input
>> +  alc269-dmic		Enable ALC269(VA) digital mic workaround
>> +  alc271-dmic		Enable ALC271X digital mic workaround
>> +  inv-dmic		Inverted internal mic workaround
>> +  lenovo-dock		Enables docking station I/O for some Lenovos
>> +  dell-headset-multi	Headset jack, which can also be used as mic-in
>> +  dell-headset-dock	Headset jack (without mic-in), and also dock I/O
>> +
>> +ALC662/663/668/272
>>   ==============
>> -  mario		Chromebook mario model fixup
>> -  asus-mode1	ASUS
>> -  asus-mode2	ASUS
>> -  asus-mode3	ASUS
>> -  asus-mode4	ASUS
>> -  asus-mode5	ASUS
>> -  asus-mode6	ASUS
>> -  asus-mode7	ASUS
>> -  asus-mode8	ASUS
>> -  inv-dmic	Inverted internal mic workaround
>> +  mario			Chromebook mario model fixup
>> +  asus-mode1		ASUS
>> +  asus-mode2		ASUS
>> +  asus-mode3		ASUS
>> +  asus-mode4		ASUS
>> +  asus-mode5		ASUS
>> +  asus-mode6		ASUS
>> +  asus-mode7		ASUS
>> +  asus-mode8		ASUS
>> +  inv-dmic		Inverted internal mic workaround
>> +  dell-headset-multi	Headset jack, which can also be used as mic-in
>>
>>   ALC680
>>   ======
>> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
>> index af9e71b..403010c 100644
>> --- a/sound/pci/hda/patch_realtek.c
>> +++ b/sound/pci/hda/patch_realtek.c
>> @@ -3599,6 +3599,8 @@ static const struct hda_model_fixup alc269_fixup_models[] = {
>>   	{.id = ALC269_FIXUP_INV_DMIC, .name = "inv-dmic"},
>>   	{.id = ALC269_FIXUP_LENOVO_DOCK, .name = "lenovo-dock"},
>>   	{.id = ALC269_FIXUP_HP_GPIO_LED, .name = "hp-gpio-led"},
>> +	{.id = ALC269_FIXUP_DELL1_MIC_NO_PRESENCE, .name = "dell-headset-multi"},
>> +	{.id = ALC269_FIXUP_DELL2_MIC_NO_PRESENCE, .name = "dell-headset-dock"},
>>   	{}
>>   };
>>
>> @@ -4278,6 +4280,7 @@ static const struct hda_model_fixup alc662_fixup_models[] = {
>>   	{.id = ALC662_FIXUP_ASUS_MODE7, .name = "asus-mode7"},
>>   	{.id = ALC662_FIXUP_ASUS_MODE8, .name = "asus-mode8"},
>>   	{.id = ALC662_FIXUP_INV_DMIC, .name = "inv-dmic"},
>> +	{.id = ALC668_FIXUP_DELL_MIC_NO_PRESENCE, .name = "dell-headset-multi"},
>>   	{}
>>   };
>>
>> --
>> 1.7.9.5
>>
>



-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic

[1] https://lkml.org/lkml/2013/6/8/108