[alsa-devel] ChromeBook machine will lose MIC JD

[Kailang]

Thanks. Takashi.

-----Original Message-----
From: Takashi Iwai <tiwai at suse.de> 
Sent: Wednesday, March 13, 2019 6:51 PM
To: Jaroslav Kysela <perex at perex.cz>
Cc: Kailang <kailang at realtek.com>; (alsa-devel at alsa-project.org) <alsa-devel at alsa-project.org>
Subject: Re: [alsa-devel] ChromeBook machine will lose MIC JD

On Thu, 07 Mar 2019 20:49:22 +0100,
Jaroslav Kysela wrote:
> 
> Dne 07. 03. 19 v 9:11 Kailang napsal(a):
> > Hi Jaroslav,
> > 
> > I need to check with Intel chrome audio team for the name change.
> > But chrome OS used 'alc-sense-combo' for early patch file.
> 
> But with your patch, the ALC225_FIXUP_HEADSET_JACK and 
> ALC295_FIXUP_CHROME_BOOK do a different thing, so it's not a simple 
> replacement. Your chrome book fixup is not universal, so it should be 
> distinguished in my opinion.

Yeah, ideally this should be implemented in another model name.

But, in this particular case, the original quirk was provided specifically for Chromebook (despite of its generic model name "alc-sense-combo"), and since Chromebook is already using this string, it's better to keep the string as before.  That is, the risk of breakage for other users of this model name is much lower than the burden of changes of the fixed string for Chrome, I suppose.

So I'm going to take the patch as is -- as an exception.

> Also, please, inline your patches or use ;git send-email' command to 
> send your commit to this ML for a review.

Right.  But I thought it was some problem in Kailang's company network to use git-send-email.  But some other people from Realtek seem capable to send patches in the git way, so it can be doable, as it seems.


thanks,

Takashi

> 
> 					Thanks,
> 						Jaroslav
> 
> > It also need to add this patch.
> > 
> > Oh! Sorry!
> > I sent it with HTML mode.
> > I need to resend again for TXT mode. 
> > 
> > BR,
> > Kailang
> > 
> > -----Original Message-----
> > From: Jaroslav Kysela <perex at perex.cz>
> > Sent: Thursday, March 7, 2019 4:03 PM
> > To: Kailang <kailang at realtek.com>; Takashi Iwai (tiwai at suse.de) 
> > <tiwai at suse.de>
> > Cc: (alsa-devel at alsa-project.org) <alsa-devel at alsa-project.org>
> > Subject: Re: [alsa-devel] ChromeBook machine will lose MIC JD
> > 
> > Dne 07. 03. 19 v 8:56 Kailang napsal(a):
> >> Hi Takashi,
> >>
> >> Attach patch will solve the issue for Mic JD lose.
> >>
> > 
> > -	{.id = ALC225_FIXUP_HEADSET_JACK, .name = "alc-sense-combo"},
> > +	{.id = ALC295_FIXUP_CHROME_BOOK, .name = "alc-sense-combo"},
> > 
> > Define another name here like 'alc-chrome-book' and don't remove the previous model name 'alc-sense-combo'.
> > 
> > 						Jaroslav
> > 
> > --
> > Jaroslav Kysela <perex at perex.cz>
> > Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
> > 
> > ------Please consider the environment before printing this e-mail.
> > 
> 
> 
> --
> Jaroslav Kysela <perex at perex.cz>
> Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
>