Re: [PATCH] ALSA: hda - Fix headset mic detection problem for two Dell laptops

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

 



On Thu, 20 Oct 2016 08:03:33 +0200,
Hui Wang wrote:
> 
> They uses the codec ALC255, and have the different pin cfg definition
> from the ones in the existing pin quirk table. Now adding them into
> the table to fix the problem.
> 
> Cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Hui Wang <hui.wang@xxxxxxxxxxxxx>

Thanks, applied now.

But judging from the increasing number of the table entries, we may
rethink how to match the pattern in future.  The pincfg table was
introduced as a kind of fingerprint to match with the deployed
configuration.  Maybe the matching should be relaxed (e.g. doesn't
care about the assoc#), so that we can reduce the table size.


Takashi

> ---
>  sound/pci/hda/patch_realtek.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index 8483fc2..b582d57 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -5857,11 +5857,19 @@ enum {
>  		{0x1b, 0x02011020},
>  		{0x21, 0x0221101f}),
>  	SND_HDA_PIN_QUIRK(0x10ec0255, 0x1028, "Dell", ALC255_FIXUP_DELL1_MIC_NO_PRESENCE,
> +		{0x14, 0x90170110},
> +		{0x1b, 0x01011020},
> +		{0x21, 0x0221101f}),
> +	SND_HDA_PIN_QUIRK(0x10ec0255, 0x1028, "Dell", ALC255_FIXUP_DELL1_MIC_NO_PRESENCE,
>  		{0x14, 0x90170130},
>  		{0x1b, 0x01014020},
>  		{0x21, 0x0221103f}),
>  	SND_HDA_PIN_QUIRK(0x10ec0255, 0x1028, "Dell", ALC255_FIXUP_DELL1_MIC_NO_PRESENCE,
>  		{0x14, 0x90170130},
> +		{0x1b, 0x01011020},
> +		{0x21, 0x0221103f}),
> +	SND_HDA_PIN_QUIRK(0x10ec0255, 0x1028, "Dell", ALC255_FIXUP_DELL1_MIC_NO_PRESENCE,
> +		{0x14, 0x90170130},
>  		{0x1b, 0x02011020},
>  		{0x21, 0x0221103f}),
>  	SND_HDA_PIN_QUIRK(0x10ec0255, 0x1028, "Dell", ALC255_FIXUP_DELL1_MIC_NO_PRESENCE,
> -- 
> 1.9.1
> 
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]