Re: alc295_fixup_chromebook function

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

 



Hi Takashi,

This patch also changed model name as 'alc-chrome-book'.
It also tested by Intel Chrome audio team.

BR,
Kailang

-----Original Message-----
From: Takashi Iwai <tiwai@xxxxxxx> 
Sent: Monday, April 1, 2019 3:37 PM
To: Kailang <kailang@xxxxxxxxxxx>
Cc: (alsa-devel@xxxxxxxxxxxxxxxx) <alsa-devel@xxxxxxxxxxxxxxxx>
Subject: Re: alc295_fixup_chromebook function

On Mon, 01 Apr 2019 09:29:35 +0200,
Kailang wrote:
> 
> Hi Takashi,
> 
> You applied below patch.
> https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit
> /?h=for-next&id=c8a9afa632f0fd45731d3353525faf1fdb362c89
> 
> But I think ALC225_FIXUP_HEADSET_JACK will applicate for enable for headset button.
> It was not only use for Chrome OS.
> So, ALC225_FIXUP_HEADSET_JACK and alc295_fixup_chromebook separation were be better.
> 
> Intel test https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit/?h=for-next&id=10f5b1b85ed10a80d45bc2db450e65bd792efaad this patch.
> Mic JD was also not stable. (I don't know why. It passed with Ubuntu 
> OS.) It need to change to initial state would be better. (original was 
> HDA_FIXUP_ACT_PRE_PROBE)
> 
> +static void alc295_fixup_chromebook(struct hda_codec *codec,
> +				    const struct hda_fixup *fix, int action) {
> +
> +	switch (action) {
> +	case HDA_FIXUP_ACT_INIT:
> +		switch (codec->core.vendor_id) {
> +		case 0x10ec0295:
> +			alc_update_coef_idx(codec, 0x4a, 0x8000, 1 << 15); /* Reset HP JD */
> +			alc_update_coef_idx(codec, 0x4a, 0x8000, 0 << 15);
> +			break;
> +		case 0x10ec0236:
> +			alc_update_coef_idx(codec, 0x1b, 0x8000, 1 << 15); /* Reset HP JD */
> +			alc_update_coef_idx(codec, 0x1b, 0x8000, 0 << 15);
> +			break;
> +		}
> +		break;
> +	}
> +}
> 
> How could I create a fix patch for this?

The Chromebook quirk name was chosen so that the quirk implementation won't be used by any random user as a generic solution.  If it can be used for real solutions, feel free to rename / split the quirk for easy adaption.  I have no big preference, but would like to judge from the actual patches.


thanks,

Takashi

------Please consider the environment before printing this e-mail.

Attachment: 0001-headset-mic-jd.patch
Description: 0001-headset-mic-jd.patch

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux