On Tue, 06 Mar 2018 11:04:35 +0100, Benjamin Berg wrote: > > Hi Peter, > > On Mon, 2018-02-26 at 08:46 +0000, Peter FP1 Zhang wrote: > > The following comments from our Accessory BU hardware team FYI. > > // > > Most of dock audio is converted from USB if it is connected by cable, > > but CS13 Mechnical dock is not. We need to know the specific dock > > model first. > > // > > I think that basically answers the question and we can conclude that > the 80s series has USB audio on the docking stations. > > CS13 is the previous generation docking (at the bottom of the laptops) > while the new models have a new docking connector at the side (which is > USB-C + 14 pins primarily for Ethernet incl. LEDs and likely some other > purpose). OK, I take the patch as is for now. thanks, Takashi > > Thanks, > Benjamin > > > -----Original Message----- > > From: Peter FP1 Zhang > > Sent: Wednesday, February 21, 2018 12:07 AM > > To: 'Takashi Iwai' > > Cc: Benjamin Berg; alsa-devel@xxxxxxxxxxxxxxxx; Jaroslav Kysela; linu > > x-kernel@xxxxxxxxxxxxxxx; kailang@xxxxxxxxxxx > > Subject: RE: [PATCH] ALSA: hda/realtek: Limit mic boost on T480 > > > > Thank you Takashi. > > I will also double confirm it with our Accessory BU guys about Ben's > > question when I back to my office. And will let you know if there was > > any different answer from them. > > > > > > Peter Zhang \ 张福平, PMP > > ThinkPad & ThinkStation Linux Solutions > > Tel: (+86) 181-1611-8005 | Lenovo Shanghai > > > > Linux for Those Who Do - http://www.lenovo.com/linux > > > > -----Original Message----- > > From: Takashi Iwai [mailto:tiwai@xxxxxxx] > > Sent: Tuesday, February 20, 2018 11:48 PM > > To: Peter FP1 Zhang > > Cc: Benjamin Berg; alsa-devel@xxxxxxxxxxxxxxxx; Jaroslav Kysela; linu > > x-kernel@xxxxxxxxxxxxxxx; kailang@xxxxxxxxxxx > > Subject: Re: [PATCH] ALSA: hda/realtek: Limit mic boost on T480 > > > > On Sat, 17 Feb 2018 14:54:02 +0100, > > Peter FP1 Zhang wrote: > > > > > > Hi Ben, > > > > > > My understanding is same as yours. Recently Realtek Audio expert > > > Kailang (on copy) submitted a patch for Dock as the attached, are > > > you asking the same thing? > > > Looks like the issue is related to Dock model, there are many > > > different Dock models, could you please let us know which specific > > > dock model you mean? Thanks. > > > > > > @Kailang, Do you have any suggestions or comments? > > > > FYI, because of this unclearness, the patch from Benjamin is still > > pending. I'm OK to apply it at first, then adjust later if the > > docking station quirk is required, too. Just let me know. > > > > > > thanks, > > > > Takashi > > > > > > > > Thanks, > > > > > > Peter Zhang \ 张福平, PMP > > > ThinkPad & ThinkStation Linux Solutions > > > Tel: (+86) 181-1611-8005 | Lenovo Shanghai > > > > > > Linux for Those Who Do - http://www.lenovo.com/linux > > > > > > > > > -----Original Message----- > > > From: Benjamin Berg [mailto:bberg@xxxxxxxxxx] > > > Sent: Wednesday, February 14, 2018 11:41 PM > > > To: Takashi Iwai > > > Cc: alsa-devel@xxxxxxxxxxxxxxxx; Jaroslav Kysela; > > > linux-kernel@xxxxxxxxxxxxxxx; Peter FP1 Zhang > > > Subject: Re: [PATCH] ALSA: hda/realtek: Limit mic boost on T480 > > > > > > -----BEGIN PGP SIGNED MESSAGE----- > > > Hash: SHA512 > > > > > > On Wed, 2018-02-14 at 14:00 +0100, Takashi Iwai wrote: > > > > On Wed, 14 Feb 2018 13:29:39 +0100, > > > > Benjamin Berg wrote: > > > > > > > > > > The internal mic boost on the T480 is too high. Fix this by > > > > > applying the ALC269_FIXUP_LIMIT_INT_MIC_BOOST fixup to the > > > > > machine > > > > > to limit the gain. > > > > > > > > > > Signed-off-by: Benjamin Berg <bberg@xxxxxxxxxx> > > > > > Tested-by: Benjamin Berg <bberg@xxxxxxxxxx> > > > > > > > > Applying this quirk itself is OK, but just wonder whether this > > > > model > > > > has a docking station port. Recently we fixed the dock issue, > > > > and > > > > it required to apply ALC298_FIXUP_TPT470_DOCK. > > > > > > I assumed that on the T480 and similar models the audio on the dock > > > is connected through USB (the dock is USB-C + further pins), but I > > > cannot confirm this myself right now as I only have a T480 > > > currently. > > > > > > I have now contacted Peter Zhang of Lenovo about this and also to > > > check if other 80 series models may need the same fixup. > > > > > > Benjamin > > > > > > > > > > > thanks, > > > > > > > > Takashi > > > > > > > > > --- > > > > > sound/pci/hda/patch_realtek.c | 1 + > > > > > 1 file changed, 1 insertion(+) > > > > > > > > > > diff --git a/sound/pci/hda/patch_realtek.c > > > > > b/sound/pci/hda/patch_realtek.c index > > > > > ce28f7ce64e6..8467ce3db767 > > > > > 100644 > > > > > --- a/sound/pci/hda/patch_realtek.c > > > > > +++ b/sound/pci/hda/patch_realtek.c > > > > > @@ -6510,6 +6510,7 @@ static const struct snd_pci_quirk > > > > > alc269_fixup_tbl[] = { > > > > > SND_PCI_QUIRK(0x17aa, 0x224b, "Thinkpad", > > > > > ALC298_FIXUP_TPT470_DOCK), > > > > > SND_PCI_QUIRK(0x17aa, 0x224c, "Thinkpad", > > > > > ALC298_FIXUP_TPT470_DOCK), > > > > > SND_PCI_QUIRK(0x17aa, 0x224d, "Thinkpad", > > > > > ALC298_FIXUP_TPT470_DOCK), > > > > > + SND_PCI_QUIRK(0x17aa, 0x225d, "Thinkpad T480", > > > > > ALC269_FIXUP_LIMIT_INT_MIC_BOOST), > > > > > SND_PCI_QUIRK(0x17aa, 0x30bb, "ThinkCentre AIO", > > > > > ALC233_FIXUP_LENOVO_LINE2_MIC_HOTKEY), > > > > > SND_PCI_QUIRK(0x17aa, 0x30e2, "ThinkCentre AIO", > > > > > ALC233_FIXUP_LENOVO_LINE2_MIC_HOTKEY), > > > > > SND_PCI_QUIRK(0x17aa, 0x310c, "ThinkCentre Station", > > > > > ALC294_FIXUP_LENOVO_MIC_LOCATION), > > > > > -- > > > > > 2.14.3 > > > > > > > > > > > > > > > > -----BEGIN PGP SIGNATURE----- > > > > > > iQIzBAEBCgAdFiEED2NO4vMS33W8E4AFq6ZWhpmFY3AFAlqEWKkACgkQq6ZWhpmF > > > Y3AEyQ/7B6l4Na1GLDf11P8OdzzcXOy6grjvrLYwq6WFA3M1DirlQsPwjSbnJ7V8 > > > znIX30EK6My9Ip6jyvJQIpiBlaX1FKJuJ3So9mjgUeOcCQMFkwnC1fMMyyWEAWLq > > > TI1s7F5w4OQwhQpwtlId4AIB7A2ZzQmP/9YqfIPDtIiNJqjd9aRruBJi8A8j3sB5 > > > slY1CYgCQhBUiAkUPisFrt+b4aFU5FlgINQR+6uOSKO0/IaSKq9rFwX76lOTOsBj > > > kHhywIF9Qef6CvUDaAOFWjQcTA/ooDPifUsfNzo6ZHnNZIks0Cov12JeO5UrxjiD > > > a3BuG6SZAVm4awfkTj5pIL0DuBAXYhTJd/hU3VMWdV0/kb0OeVONSahMzhSOCM30 > > > WdFKpWC0+n3TEkTWSG9QGwL6wbXiaLNZ0JRgQkcoJ9JqPVngFf9LIRBzqdMyzyCE > > > UzLu/oQ9XlXQrFx1IG5KS/RJJPUwpJKhXOygSpRZx/HhmaWpPVLCu9GNg+ytoVIL > > > QEg381jOTTPi+DX0k07WYK+hUEvB83cbtsm+EiEGMa4bxuhYQp2p1moGqOrzv2XK > > > iEN8LfF97KJ4GtqVWybnstxMxMdS0CwXBPxUZ3OY6C03S3sdVqk/QrEpE4i0+N8P > > > CcX3g+rEJVUkpsdm17Sf5u4DjeGH7HBPsDmHirrwvFdeOkTiO4E= > > > =NIZ6 > > > -----END PGP SIGNATURE----- > > > > > > [2 0000-alc298-thinkpad-dock.patch <application/octet-stream > > > (base64)>] > > > > -----BEGIN PGP SIGNATURE----- > > iQIzBAEBCgAdFiEED2NO4vMS33W8E4AFq6ZWhpmFY3AFAlqeZ7MACgkQq6ZWhpmF > Y3CUIRAAnZ40ojWpK/oQmBvZCo5tEuiLKuJtQYhsw3yOxs96Ud2u1W/pSx3ED1Re > qm0TNk9sj7oNPmVnUuSMN5JIj4erW/K3fQ/yM69Sen7LFgDYuSvVEEdhHQbPjpC0 > 4C9AHA/InFmecWFjHZ8OLC1DvkXPoHmlw3MpJ4delf1qYZgq6jSrpQFJd2ClnNln > wvinsIPo71lHLHB33aQd+Gii6C2GplW8V/j77qDVU3KAQQSSz9LDHitZ8oFxEk5U > kX+nsCN8LXrnBBNGNcVsKaKdJUEfBcelnxz/3I15NPLEroOA5kZnzArj6OBBnYvB > axpYlXKv1+eSdkq/qGq3H2Me7Ln38kNVYcu7wJKwsBxcMMwhXAoQRLwwr7tEKW+z > lA8leioxjeCR9ncqxbuUuqOAa6W/PRd6dH9Xp3UVaTg/EPC/kZfg9R2iY7I/ZQOT > 2rcLcsVxlLWg0BGAnV/2o3gRwXymHxKYT8R0MS3TFqVq+fZj9K+SXeueBid1qW78 > wHs9hHjFOO5pZprkOmLWbybV39qi9gEeBV+Q+bIyAlJiYXT2tBCe/t19+6ktCxNa > 8/BWeOvk+rRamtanwX+cXaYOVoh/Ney1nODJs70i58yrTpKM4RwQlhx515S7a+UQ > Hnx+xw8/wd9IPH67yc7D97iKSFHk5ObmeNB6IHUQhm7CyP6Yedw= > =+cS7 > -----END PGP SIGNATURE----- > _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel