| Message ID | 20200826084014.211217-1-adrien.crivelli@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ALSA: hda/realtek: Add quirk for Samsung Galaxy Book Ion NT950XCJ-X716A | expand |
On Wed, 26 Aug 2020 10:40:14 +0200, Adrien Crivelli wrote: > > The Galaxy Book Ion NT950XCJ-X716A (15 inches) uses the same ALC298 > codec as other Samsung laptops which have the no headphone sound bug. I > confirmed on my own hardware that this fixes the bug. > > This also correct the model name for the 13 inches version. It was > incorrectly referenced as NT950XCJ-X716A in commit e17f02d05. But it > should have been NP930XCJ-K01US. > > BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=207423 > Signed-off-by: Adrien Crivelli <adrien.crivelli@gmail.com> Thanks, applied. Takashi
Great, thanks! On Wed, 26 Aug 2020 at 17:47, Takashi Iwai <tiwai@suse.de> wrote: > On Wed, 26 Aug 2020 10:40:14 +0200, > Adrien Crivelli wrote: > > > > The Galaxy Book Ion NT950XCJ-X716A (15 inches) uses the same ALC298 > > codec as other Samsung laptops which have the no headphone sound bug. I > > confirmed on my own hardware that this fixes the bug. > > > > This also correct the model name for the 13 inches version. It was > > incorrectly referenced as NT950XCJ-X716A in commit e17f02d05. But it > > should have been NP930XCJ-K01US. > > > > BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=207423 > > Signed-off-by: Adrien Crivelli <adrien.crivelli@gmail.com> > > Thanks, applied. > > > Takashi >
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index a1fa983d2..98789691a 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -7695,7 +7695,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x144d, 0xc169, "Samsung Notebook 9 Pen (NP930SBE-K01US)", ALC298_FIXUP_SAMSUNG_HEADPHONE_VERY_QUIET), SND_PCI_QUIRK(0x144d, 0xc176, "Samsung Notebook 9 Pro (NP930MBE-K04US)", ALC298_FIXUP_SAMSUNG_HEADPHONE_VERY_QUIET), SND_PCI_QUIRK(0x144d, 0xc189, "Samsung Galaxy Flex Book (NT950QCG-X716)", ALC298_FIXUP_SAMSUNG_HEADPHONE_VERY_QUIET), - SND_PCI_QUIRK(0x144d, 0xc18a, "Samsung Galaxy Book Ion (NT950XCJ-X716A)", ALC298_FIXUP_SAMSUNG_HEADPHONE_VERY_QUIET), + SND_PCI_QUIRK(0x144d, 0xc18a, "Samsung Galaxy Book Ion (NP930XCJ-K01US)", ALC298_FIXUP_SAMSUNG_HEADPHONE_VERY_QUIET), + SND_PCI_QUIRK(0x144d, 0xc830, "Samsung Galaxy Book Ion (NT950XCJ-X716A)", ALC298_FIXUP_SAMSUNG_HEADPHONE_VERY_QUIET), SND_PCI_QUIRK(0x144d, 0xc740, "Samsung Ativ book 8 (NP870Z5G)", ALC269_FIXUP_ATIV_BOOK_8), SND_PCI_QUIRK(0x144d, 0xc812, "Samsung Notebook Pen S (NT950SBE-X58)", ALC298_FIXUP_SAMSUNG_HEADPHONE_VERY_QUIET), SND_PCI_QUIRK(0x1458, 0xfa53, "Gigabyte BXBT-2807", ALC283_FIXUP_HEADSET_MIC),
The Galaxy Book Ion NT950XCJ-X716A (15 inches) uses the same ALC298 codec as other Samsung laptops which have the no headphone sound bug. I confirmed on my own hardware that this fixes the bug. This also correct the model name for the 13 inches version. It was incorrectly referenced as NT950XCJ-X716A in commit e17f02d05. But it should have been NP930XCJ-K01US. BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=207423 Signed-off-by: Adrien Crivelli <adrien.crivelli@gmail.com> --- sound/pci/hda/patch_realtek.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)