linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo
@ 2021-10-01 13:31 Werner Sembach
  2021-10-01 13:31 ` [PATCH 1/2] ALSA: hda/realtek: Complete partial device name to avoid ambiguity Werner Sembach
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Werner Sembach @ 2021-10-01 13:31 UTC (permalink / raw)
  To: perex, tiwai, alsa-devel, linux-kernel

Add quirk for Clevo X170KM-G to fix internal speaker not working and rename
previously named "Clevo X170" into the more verbose "Clevo X170SM" to make
the devices differentiable.

Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
Cc: <stable@vger.kernel.org>



^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] ALSA: hda/realtek: Complete partial device name to avoid ambiguity
  2021-10-01 13:31 [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo Werner Sembach
@ 2021-10-01 13:31 ` Werner Sembach
  2021-10-01 13:31 ` [PATCH 2/2] ALSA: hda/realtek: Add quirk for Clevo X170KM-G Werner Sembach
  2021-10-04  7:35 ` [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo Takashi Iwai
  2 siblings, 0 replies; 4+ messages in thread
From: Werner Sembach @ 2021-10-01 13:31 UTC (permalink / raw)
  To: perex, tiwai, alsa-devel, linux-kernel

The string "Clevo X170" is not enough to unambiguously identify the correct
device.

Fixing it so another Clevo barebone name starting with "X170" can be added
without causing confusion.

Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
Cc: <stable@vger.kernel.org>
---
 sound/pci/hda/patch_realtek.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 8b7a389b6aed..706acbcff472 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -2537,7 +2537,7 @@ static const struct snd_pci_quirk alc882_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x1558, 0x67e1, "Clevo PB71[DE][CDF]", ALC1220_FIXUP_CLEVO_PB51ED_PINS),
 	SND_PCI_QUIRK(0x1558, 0x67e5, "Clevo PC70D[PRS](?:-D|-G)?", ALC1220_FIXUP_CLEVO_PB51ED_PINS),
 	SND_PCI_QUIRK(0x1558, 0x70d1, "Clevo PC70[ER][CDF]", ALC1220_FIXUP_CLEVO_PB51ED_PINS),
-	SND_PCI_QUIRK(0x1558, 0x7714, "Clevo X170", ALC1220_FIXUP_CLEVO_PB51ED_PINS),
+	SND_PCI_QUIRK(0x1558, 0x7714, "Clevo X170SM", ALC1220_FIXUP_CLEVO_PB51ED_PINS),
 	SND_PCI_QUIRK(0x1558, 0x9501, "Clevo P950HR", ALC1220_FIXUP_CLEVO_P950),
 	SND_PCI_QUIRK(0x1558, 0x9506, "Clevo P955HQ", ALC1220_FIXUP_CLEVO_P950),
 	SND_PCI_QUIRK(0x1558, 0x950a, "Clevo P955H[PR]", ALC1220_FIXUP_CLEVO_P950),
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] ALSA: hda/realtek: Add quirk for Clevo X170KM-G
  2021-10-01 13:31 [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo Werner Sembach
  2021-10-01 13:31 ` [PATCH 1/2] ALSA: hda/realtek: Complete partial device name to avoid ambiguity Werner Sembach
@ 2021-10-01 13:31 ` Werner Sembach
  2021-10-04  7:35 ` [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo Takashi Iwai
  2 siblings, 0 replies; 4+ messages in thread
From: Werner Sembach @ 2021-10-01 13:31 UTC (permalink / raw)
  To: perex, tiwai, alsa-devel, linux-kernel

This applies a SND_PCI_QUIRK(...) to the Clevo X170KM-G barebone. This
fixes the issue of the devices internal Speaker not working.

Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
Cc: <stable@vger.kernel.org>
---
 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 706acbcff472..6cb542905dc6 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -2538,6 +2538,7 @@ static const struct snd_pci_quirk alc882_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x1558, 0x67e5, "Clevo PC70D[PRS](?:-D|-G)?", ALC1220_FIXUP_CLEVO_PB51ED_PINS),
 	SND_PCI_QUIRK(0x1558, 0x70d1, "Clevo PC70[ER][CDF]", ALC1220_FIXUP_CLEVO_PB51ED_PINS),
 	SND_PCI_QUIRK(0x1558, 0x7714, "Clevo X170SM", ALC1220_FIXUP_CLEVO_PB51ED_PINS),
+	SND_PCI_QUIRK(0x1558, 0x7715, "Clevo X170KM-G", ALC1220_FIXUP_CLEVO_PB51ED),
 	SND_PCI_QUIRK(0x1558, 0x9501, "Clevo P950HR", ALC1220_FIXUP_CLEVO_P950),
 	SND_PCI_QUIRK(0x1558, 0x9506, "Clevo P955HQ", ALC1220_FIXUP_CLEVO_P950),
 	SND_PCI_QUIRK(0x1558, 0x950a, "Clevo P955H[PR]", ALC1220_FIXUP_CLEVO_P950),
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo
  2021-10-01 13:31 [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo Werner Sembach
  2021-10-01 13:31 ` [PATCH 1/2] ALSA: hda/realtek: Complete partial device name to avoid ambiguity Werner Sembach
  2021-10-01 13:31 ` [PATCH 2/2] ALSA: hda/realtek: Add quirk for Clevo X170KM-G Werner Sembach
@ 2021-10-04  7:35 ` Takashi Iwai
  2 siblings, 0 replies; 4+ messages in thread
From: Takashi Iwai @ 2021-10-04  7:35 UTC (permalink / raw)
  To: Werner Sembach; +Cc: perex, tiwai, alsa-devel, linux-kernel

On Fri, 01 Oct 2021 15:31:09 +0200,
Werner Sembach wrote:
> 
> Add quirk for Clevo X170KM-G to fix internal speaker not working and rename
> previously named "Clevo X170" into the more verbose "Clevo X170SM" to make
> the devices differentiable.
> 
> Signed-off-by: Werner Sembach <wse@tuxedocomputers.com>
> Cc: <stable@vger.kernel.org>

Applied both patches now.  Thanks.


Takashi

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-10-04  7:35 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-01 13:31 [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo Werner Sembach
2021-10-01 13:31 ` [PATCH 1/2] ALSA: hda/realtek: Complete partial device name to avoid ambiguity Werner Sembach
2021-10-01 13:31 ` [PATCH 2/2] ALSA: hda/realtek: Add quirk for Clevo X170KM-G Werner Sembach
2021-10-04  7:35 ` [PATCH 0/2] ALSA: hda/realtek: Fix name and add quirk for new Clevo Takashi Iwai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).