Re: [PATCH] ALSA: hda/realtek - Enable jack detection for Intel CFL + ALC700

[Takashi Iwai <tiwai@suse.de>]

On Tue, 14 Nov 2017 04:00:08 +0100,
Guneshwor Singh wrote:
> 
> From: PeiSen Hou <pshou@realtek.com>
> 
> ALC700 has jack detection disabled by default in Intel Coffeelake
> Reference board. To enable it, this fixup is required.
> 
> Signed-off-by: PeiSen Hou <pshou@realtek.com>
> Signed-off-by: Guneshwor Singh <guneshwor.o.singh@intel.com>
> ---
>  sound/pci/hda/patch_realtek.c | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index 0ce71111b4e3..61ef08173b1c 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -5264,6 +5264,7 @@ enum {
>  	ALC233_FIXUP_LENOVO_MULTI_CODECS,
>  	ALC294_FIXUP_LENOVO_MIC_LOCATION,
>  	ALC700_FIXUP_INTEL_REFERENCE,
> +	ALC700_FIXUP_INTEL_CFL,
>  };
>  
>  static const struct hda_fixup alc269_fixups[] = {
> @@ -6074,6 +6075,21 @@ static const struct hda_fixup alc269_fixups[] = {
>  			{}
>  		}
>  	},
> +	[ALC700_FIXUP_INTEL_CFL] = {
> +		.type = HDA_FIXUP_VERBS,
> +		.v.verbs = (const struct hda_verb[]) {
> +			/* Enables internal speaker */
> +			{0x20, AC_VERB_SET_COEF_INDEX, 0x45},
> +			{0x20, AC_VERB_SET_PROC_COEF, 0x5289},
> +			{0x20, AC_VERB_SET_COEF_INDEX, 0x4a},
> +			{0x20, AC_VERB_SET_PROC_COEF, 0x201b},

I see only one bit here is different from the previous quirk for Intel
reference board.  Do we really need to keep both quirks separately?
Or maybe this new one is the correct verb in anyway?


thanks,

Takashi